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 10 pull requests #121896

Closed
wants to merge 13 commits into from

Commits on Mar 2, 2024

  1. Configuration menu
    Copy the full SHA
    624f9d3 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ec5e2dc View commit details
    Browse the repository at this point in the history
  3. The ordinary lowering of thir::ExprKind::Let is unreachable

    After desugaring, `let` expressions should only appear inside `if` expressions
    or `match` guards, possibly nested within a let-chain. In both cases they are
    specifically handled by the lowerings of those expressions, so this case is
    currently unreachable.
    Zalathar committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    972d8da View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#109263 - squell:master, r=cuviper

    fix typo in documentation for std::fs::Permissions
    
    Please check and re-check this PR carefully to see if I got this right.
    
    But by my logic, if the `read_only` function returns `true`, I would not expect be able to write to the file (it being read only); so this text is meant to clarify that `read_only` being `false` doesn't mean *you* can actually write to the file, just that "in general" someone is able to.
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    56ba5af View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#120684 - carschandler:patch-1, r=nnethercote

    Update E0716.md for clarity
    
    When reading through this, I got slightly hung up thinking the `let` it was referring to was the `let tmp` on line 25, which was confusing considering the comment states that the temporary is freed at the end of the block. I think adding this clarification could potentially help some beginners like myself without being overly verbose.
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    4e8af6c View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#121759 - RalfJung:addr_of, r=the8472

    attempt to further clarify addr_of docs
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    e63d49f View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#121815 - nnethercote:mv-gather_comments, r=…

    …est31
    
    Move `gather_comments`.
    
    To the module where it is used, so it doesn't have to be `pub`.
    
    r? ````@est31````
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    5080da9 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#121835 - nnethercote:mv-HandleStore, r=bjorn3

    Move `HandleStore` into `server.rs`.
    
    This just moves the server-relevant parts of handles into `server.rs`. It introduces a new higher-order macro `with_api_handle_types` to avoid some duplication.
    
    This fixes two `FIXME` comments, and makes things clearer, by not having server code in `client.rs`.
    
    r? ````@bjorn3````
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    b0de7d0 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#121847 - shamatar:btreemap_fix_implicits, r…

    …=cuviper
    
    Remove hidden use of Global
    
    Fixes rust-lang#121797
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    9996a4a View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#121861 - tbu-:pr_floating_point_exact_examp…

    …les, r=workingjubilee
    
    Use the guaranteed precision of a couple of float functions in docs
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    a57e0fc View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#121875 - estebank:e0277-drive-by, r=compile…

    …r-errors
    
     Account for unmet T: !Copy in E0277 message
    
    ```
    error[E0277]: the trait bound `T: !Copy` is not satisfied
      --> $DIR/simple.rs:10:16
       |
    LL |     not_copy::<T>();
       |                ^ the trait bound `T: !Copy` is not satisfied
    ```
    instead of the current
    
    ```
    error[E0277]: the trait bound `T: !Copy` is not satisfied
      --> $DIR/simple.rs:10:16
       |
    LL |     not_copy::<T>();
       |                ^ the trait `!Copy` is not implemented for `T`
    ```
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    5321efb View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#121888 - cppcoffee:style, r=Nilstrieb

    style library/core/src/error.rs
    
    Add an extra blank line for clarity in distinguishing implementations.
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    1415c24 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#121892 - Zalathar:expr-kind-let, r=Nadrieril

    The ordinary lowering of `thir::ExprKind::Let` is unreachable
    
    After desugaring, `let` expressions should only appear inside `if` expressions or `match` guards, possibly nested within a let-chain. In both cases they are specifically handled by the lowerings of those expressions, so this case is currently unreachable.
    
    ---
    
    Context: https://rust-lang.zulipchat.com/#narrow/stream/182449-t-compiler.2Fhelp/topic/Lowering.20of.20.60thir.3A.3AExprKind.3A.3ALet.60.20is.20unreachable
    
    My conclusions are partly based on the observation that stubbing out this match arm doesn't cause any test failures. So either this really is unreachable, or it can be reached in some obscure circumstances that our test suite doesn't cover.
    
    If we end up needing this code (or something like it) for an implementation of rust-lang/rfcs#3573, it should be easy enough to pull it back out of version control history.
    
    I looked into having the `if`/`match` lowerings call back into `expr_into_dest`, but from what I can tell that won't work well, because there are extra scoping considerations that require some awareness of the enclosing if/match.
    
    r? `@Nadrieril`
    matthiaskrgr committed Mar 2, 2024
    Configuration menu
    Copy the full SHA
    3de2576 View commit details
    Browse the repository at this point in the history