Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollup of 7 pull requests #91914

Closed
wants to merge 30 commits into from

Commits on Dec 13, 2021

  1. When .await is called on a non-Future expression, suggest removal

    Keep track of the origin of a `T: Future` obligation when caused by an
    `.await` expression.
    
    Address rust-lang#66731.
    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    7227a87 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    75b6275 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    caf0c1b View commit details
    Browse the repository at this point in the history
  4. Further silence ? errors

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    81a3b90 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    79749d6 View commit details
    Browse the repository at this point in the history
  6. Remove unnecessary argument

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    4f2b1c0 View commit details
    Browse the repository at this point in the history
  7. Fix mistake

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    d45e030 View commit details
    Browse the repository at this point in the history
  8. Keep info on pre-desugaring expression for better "incorrect .await

    …" suggestion
    
    Keep the `HirId` of `.await`ed expressions so in the case of a `fn` call
    on on a sync `fn`, we can suggest maybe turning it into an `async fn`.
    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    f640438 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    8888d0d View commit details
    Browse the repository at this point in the history
  10. Fix rebase and clippy tests

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    b825b0f View commit details
    Browse the repository at this point in the history
  11. tidy fix

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    9ecb141 View commit details
    Browse the repository at this point in the history
  12. Simplify diagnostic logic

    The spans no longer overlap, so we no longer need to specialize the
    output depending on whether they would.
    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    d59f74a View commit details
    Browse the repository at this point in the history
  13. review comments

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    64dea33 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    1a7f2d5 View commit details
    Browse the repository at this point in the history
  15. fix clippy tests

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    64f88e8 View commit details
    Browse the repository at this point in the history
  16. fix coverage report test

    estebank committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    f2fc84f View commit details
    Browse the repository at this point in the history
  17. Revert "Set MACOSX_DEPLOYMENT_TARGET env var to default for linking i…

    …f not set."
    
    This reverts commit b376f56, which is
    the main part of rust-lang#90499, because it turns out that this causes a good
    amount of breakage in crates relying on the old behavior.
    
    Fixes rust-lang#91372.
    hkratz committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    3011154 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2021

  1. Configuration menu
    Copy the full SHA
    ae21dd0 View commit details
    Browse the repository at this point in the history
  2. Remove in_band_lifetimes from rustc_trait_selection

    Sylvan Bowdler committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    1ea756b View commit details
    Browse the repository at this point in the history
  3. made compiler happy

    lameferret committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    90aa8fb View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    97e844a View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    a586e7d View commit details
    Browse the repository at this point in the history
  6. ⬆️ rust-analyzer

    lnicola committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    c771970 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#90939 - estebank:wg-af-polish, r=tmandry

    Tweak errors coming from `for`-loop, `?` and `.await` desugaring
    
     * Suggest removal of `.await` on non-`Future` expression
     * Keep track of obligations introduced by desugaring
     * Remove span pointing at method for obligation errors coming from desugaring
     * Point at called local sync `fn` and suggest making it `async`
    
    ```
    error[E0277]: `()` is not a future
      --> $DIR/unnecessary-await.rs:9:10
       |
    LL |     boo().await;
       |     -----^^^^^^ `()` is not a future
       |     |
       |     this call returns `()`
       |
       = help: the trait `Future` is not implemented for `()`
    help: do not `.await` the expression
       |
    LL -     boo().await;
    LL +     boo();
       |
    help: alternatively, consider making `fn boo` asynchronous
       |
    LL | async fn boo () {}
       | +++++
    ```
    
    Fix rust-lang#66731.
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    2666841 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#91870 - rusticstuff:macosx_min_version_reve…

    …rt, r=Mark-Simulacrum
    
    Revert setting a default for the MACOSX_DEPLOYMENT_TARGET env var for linking
    
    This reverts commit b376f56, which is the main part of rust-lang#90499, because it turns out that this causes a good amount of breakage in crates relying on the old behavior. In particular `winit`, `coreaudio` and crates that depend on them are affected. Fixes rust-lang#91372.
    
    Background:
    Before rust-lang#90499 the behavior was the following: If MACOSX_DEPLOYMENT_TARGET is not set,  we pass the minimum supported OS version to LLVM but not to the linker. The linker default depends on the Xcode version and the version of the OS it is running on. That caused one known problem in libcurl with the most recent Xcode versions. rust-lang#90499 passed the minumum supported version (10.7 for Macos x86-64) to the linker instead. This has shown to be problematic because some crates such as winit, coreaudio implicitly expect a newer minimum OS version. The libcurl issue has been fixed independently (see alexcrichton/curl-rust#417), so a revert should not really be problematic.
    
    Eventually we should probably mimic clang's behavior and fall back to the default of the currently configured Macos SDK for both the LLVM min os target version and MACOSX_DEPLOYMENT_TARGET for linking. That would entail looking at the `Version` property of the `SDKSettings.json` in the currently configured SDK.
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    421c5f1 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#91880 - matthiaskrgr:clippy_perf_dec, r=jyn514

    fix clippy::single_char_pattern perf findings
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    ed84782 View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#91882 - Patrick-Poitras:remove-in-band-life…

    …times-from-rustc-typeck, r=jackh726
    
    Remove `in_band_lifetimes` from `rustc_typeck`
    
    Joining in on the effort to remove the `in_band_lifetimes` features, as described in issue rust-lang#91867.
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    20106a3 View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#91904 - SylvanB:remove_in_band_lifetimes_ru…

    …stc_trait_selection, r=petrochenkov
    
    Remove `in_band_lifetimes` from `rustc_trait_selection`
    
    Another one for rust-lang#91867
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    087f3b7 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#91906 - anuvratsingh:remove_in_band_lifetim…

    …es_library_proc_macro, r=petrochenkov
    
    Removed `in_band_lifetimes` from `library\proc_macro`
    
    Issue [rust-lang#91867](rust-lang#91867)
    
    This is my first try, I followed the instructions given. Fixed all the errors that were thrown while compiling.
    Compiled with stage 0,1, and 2 all of them compiled successfully.
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    74a8661 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#91909 - lnicola:rust-analyzer-2021-12-14, r…

    …=lnicola
    
    ⬆️ rust-analyzer
    
    r? ``@ghost``
    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    d910cc1 View commit details
    Browse the repository at this point in the history