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 6 pull requests #121458

Closed
wants to merge 17 commits into from

Commits on Feb 3, 2024

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

Commits on Feb 21, 2024

  1. Add more tests

    oli-obk committed Feb 21, 2024
    Configuration menu
    Copy the full SHA
    31478cd View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    66bd645 View commit details
    Browse the repository at this point in the history
  3. Fix typo in serialized.rs

    accomodate -> accommodate
    eltociear authored Feb 21, 2024
    Configuration menu
    Copy the full SHA
    00234f0 View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2024

  1. Fix panic when compiling Rocket.

    `Rustc::emit_diagnostic` reconstructs a diagnostic passed in from the
    macro machinery. Currently it uses the type `DiagnosticBuilder<'_,
    ErrorGuaranteed>`, which is incorrect, because the diagnostic might be a
    warning. And if it is a warning, because of the `ErrorGuaranteed` we end
    up calling into `emit_producing_error_guaranteed` and the assertion
    within that function (correctly) fails because the level is not an error
    level.
    
    The fix is simple: change the type to `DiagnosticBuilder<'_, ()>`. Using
    `()` works no matter what the diagnostic level is, and we don't need an
    `ErrorGuaranteed` here.
    
    The panic was reported in rust-lang#120576.
    nnethercote committed Feb 22, 2024
    Configuration menu
    Copy the full SHA
    326b44e View commit details
    Browse the repository at this point in the history
  2. Make some IntoDiagnostic impls generic.

    PR rust-lang#119097 made the decision to make all `IntoDiagnostic` impls generic,
    because this allowed a bunch of nice cleanups. But four hand-written
    impls were unintentionally overlooked. This commit makes them generic.
    nnethercote committed Feb 22, 2024
    Configuration menu
    Copy the full SHA
    02423a5 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    c276af2 View commit details
    Browse the repository at this point in the history
  4. DefId to LocalDefId

    lcnr committed Feb 22, 2024
    Configuration menu
    Copy the full SHA
    7921ce3 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    9e016a8 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    e3021eb View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    e4622e0 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#120598 - compiler-errors:no-rigid-check, r=…

    …lcnr
    
    No need to `validate_alias_bound_self_from_param_env` in `assemble_alias_bound_candidates`
    
    We already fully normalize the self type before we reach `assemble_alias_bound_candidates`, so there's no reason to double check that a projection is truly rigid by checking param-env bounds.
    
    I think this is also blocked on us making sure to always normalize opaques: rust-lang#120549.
    
    r? lcnr
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    490ee5a View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#121386 - oli-obk:no_higher_ranked_opaques, …

    …r=lcnr
    
    test that we do not support higher-ranked regions in opaque type inference
    
    We already do all the right checks in `check_opaque_type_parameter_valid`, and we have done so since at least 2 years.
    
    I collected the tests from rust-lang#116935 and rust-lang#100503 and added some more
    
    cc rust-lang#96146
    
    r? `@lcnr`
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    24e8155 View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#121401 - eltociear:patch-25, r=nnethercote

    Fix typo in serialized.rs
    
    accomodate -> accommodate
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    8bf05b1 View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#121427 - nnethercote:fix-Rocket, r=oli-obk

    Fix panic when compiling `Rocket`.
    
    This panic was reported [here](rust-lang#120576 (comment)).
    
    r? `@oli-obk`
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    f9aeaa6 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#121439 - jrudolph:patch-1, r=bjorn3

    Fix typo in metadata.rs doc comment
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    25aac34 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#121441 - lcnr:typesystem-cleanup, r=compile…

    …r-errors
    
    `DefId`  to `LocalDefId`
    matthiaskrgr authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    20e862e View commit details
    Browse the repository at this point in the history