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 #124277

Merged
merged 14 commits into from
Apr 23, 2024
Merged

Rollup of 7 pull requests #124277

merged 14 commits into from
Apr 23, 2024

Commits on Apr 20, 2024

  1. Configuration menu
    Copy the full SHA
    dd01f75 View commit details
    Browse the repository at this point in the history

Commits on Apr 21, 2024

  1. Configuration menu
    Copy the full SHA
    875f0c2 View commit details
    Browse the repository at this point in the history

Commits on Apr 22, 2024

  1. Configuration menu
    Copy the full SHA
    06cd79b View commit details
    Browse the repository at this point in the history
  2. Use DefiningOpaqueTypes::Yes, as the InferCtxt we use has no opaq…

    …ue types it may define
    oli-obk committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    6bff7f4 View commit details
    Browse the repository at this point in the history
  3. Improve handling of expr->field errors

    The current message for "`->` used for field access" is the following:
    
    ```rust
    error: expected one of `!`, `.`, `::`, `;`, `?`, `{`, `}`, or an operator, found `->`
     --> src/main.rs:2:6
      |
    2 |     a->b;
      |      ^^ expected one of 8 possible tokens
    ```
    
    (playground link[1])
    
    This PR tries to address this by adding a dedicated error message and recovery. The proposed error message is:
    
    ```
    error: `->` used for field access or method call
     --> ./tiny_test.rs:2:6
      |
    2 |     a->b;
      |      ^^ help: try using `.` instead
      |
      = help: the `.` operator will dereference the value if needed
    ```
    
    (feel free to bikeshed it as much as necessary)
    
    [1]: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=7f8b6f4433aa7866124123575456f54e
    
    Signed-off-by: Sasha Pourcelot <sasha.pourcelot@protonmail.com>
    scrabsha committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    98332c1 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    7789874 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    df437a2 View commit details
    Browse the repository at this point in the history

Commits on Apr 23, 2024

  1. Rollup merge of rust-lang#123680 - compiler-errors:gen-kw, r=Nadrieril

    Deny gen keyword in `edition_2024_compat` lints
    
    Splits the `keyword_idents` lint into two -- `keyword_idents_2018` and `keyword_idents_2024` -- since each corresponds to a future-compat warning in a different edition. Group these together into a new `keyword_idents` lint group, and add the latter to the `rust_2024_compatibility` so that `gen` is ready for the 2024 edition.
    
    cc `@traviscross` `@ehuss`
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    d5cfc5c View commit details
    Browse the repository at this point in the history
  2. Rollup merge of rust-lang#124057 - gurry:124031-ice-layout-errored, r…

    …=compiler-errors
    
    Fix ICE when ADT tail has type error
    
    Fixes rust-lang#124031
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    8859631 View commit details
    Browse the repository at this point in the history
  3. Rollup merge of rust-lang#124168 - oli-obk:define_opaque_types12, r=lcnr

    Use `DefiningOpaqueTypes::Yes` in rustdoc, where the `InferCtxt` is guaranteed to have no opaque types it can define
    
    r? `@lcnr`
    
    I manually checked there it's always `tcx.infer_ctxt().build()`
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    326cd5c View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#124197 - GuillaumeGomez:move-duplicated-cod…

    …e, r=notriddle
    
    Move duplicated code in functions in `tests/rustdoc-gui/notable-trait.goml`
    
    It also allowed me to add some new common test like:
    
    ```
            assert-position: (
                "//*[`@class='tooltip` popover']",
                {"x": |popover_x|}
            )
    ```
    
    r? `@notriddle`
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    e9b0c91 View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#124200 - scrabsha:sasha/->, r=compiler-erro…

    …rs,fmease
    
    Improve handling of expr->field errors
    
    The current message for "`->` used for field access" is the following:
    
    ```rust
    error: expected one of `!`, `.`, `::`, `;`, `?`, `{`, `}`, or an operator, found `->`
     --> src/main.rs:2:6
      |
    2 |     a->b;
      |      ^^ expected one of 8 possible tokens
    ```
    
    ([playground link](https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=7f8b6f4433aa7866124123575456f54e))
    
    This PR tries to address this by adding a dedicated error message and recovery. The proposed error message is:
    
    ```
    error: `->` used for field access or method call
     --> ./tiny_test.rs:2:6
      |
    2 |     a->b;
      |      ^^ help: try using `.` instead
      |
      = help: the `.` operator will dereference the value if needed
    ```
    
    (feel free to bikeshed it as much as necessary)
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    5800e2a View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#124220 - RalfJung:interpret-wrong-vtable, r…

    …=oli-obk
    
    Miri: detect wrong vtables in wide pointers
    
    Fixes rust-lang/miri#3497.
    Needed to catch the UB that rust-lang#123572 will start exploiting.
    
    r? `@oli-obk`
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    8039488 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#124266 - RalfJung:no-answer, r=joboet

    remove an unused type from the reentrant lock tests
    
    At least it seems unused. This was added back in 45aa6c8 together with a test related to poisoning; when the test got removed, it seems like it was forgotten to also remove this type.
    matthiaskrgr committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    819b4d5 View commit details
    Browse the repository at this point in the history