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 11 pull requests #77589

Closed
wants to merge 26 commits into from
Closed

Rollup of 11 pull requests #77589

wants to merge 26 commits into from

Commits on Sep 5, 2020

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

Commits on Sep 21, 2020

  1. Reduce boilerplate with the matches! macro

    Replaces simple bool `match`es of the form
    
        match $expr {
            $pattern => true
            _ => false
        }
    
    and their inverse with invocations of the matches! macro.
    LingMan committed Sep 21, 2020
    Configuration menu
    Copy the full SHA
    a6ff925 View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2020

  1. rewrite old test so that its attributes are consistent with what we w…

    …ant in the language.
    
    (Note that the fact this test existed is a slight sign that we may need a crater
    run on this bugfix...)
    pnkfelix authored and Mark-Simulacrum committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    b4e77d2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9601724 View commit details
    Browse the repository at this point in the history
  3. Inline "eof" methods

    tamird committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    f78a7ad View commit details
    Browse the repository at this point in the history
  4. Prevent forbid from being ignored if overriden at the same level.

    That is, this changes `#[forbid(foo)] #[allow(foo)]` from allowing foo to
    forbidding foo.
    pnkfelix authored and Mark-Simulacrum committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    afa2a67 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    5ab1967 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    62f7712 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    b205436 View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2020

  1. Configuration menu
    Copy the full SHA
    afe83d4 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    5388eb4 View commit details
    Browse the repository at this point in the history
  3. inliner: use caller param_env

    lcnr committed Oct 5, 2020
    Configuration menu
    Copy the full SHA
    daf48b8 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    b1ce619 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    9704911 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    35192ff View commit details
    Browse the repository at this point in the history
  7. Rollup merge of #76388 - poliorcetics:system-time-document-panic, r=K…

    …odrAus
    
    Add a note about the panic behavior of math operations on time objects
    
    Fixes #71226.
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    7c2dd01 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of #76995 - LingMan:middle_matches, r=varkor

    Reduce boilerplate with the matches! macro
    
    Replaces simple bool `match`es of the form
    
        match $expr {
            $pattern => true
            _ => false
        }
    
    and their inverse with invocations of the matches! macro.
    
    Limited to rustc_middle for now to get my feet wet.
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    886e030 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of #77228 - GuillaumeGomez:maybeuninit-examples, r=pickfire

    Add missing examples for MaybeUninit
    
    r? @Dylan-DPC
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    01d45ec View commit details
    Browse the repository at this point in the history
  10. Rollup merge of #77528 - tamird:avoid-cast-net-parser, r=dtolnay

    Avoid unchecked casts in net parser
    
    Once this and #77426 are in, I'll send another PR adding scope id parsing.
    
    r? @dtolnay
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    5b97a70 View commit details
    Browse the repository at this point in the history
  11. Rollup merge of #77534 - Mark-Simulacrum:issue-70819-disallow-overrid…

    …e-forbid-in-same-scope, r=petrochenkov
    
    Disallow overriding forbid in same scope
    
    Rebased #73379.
    
    Fixes #70819.
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    8f33841 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    2e185c7 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of #77558 - thomcc:defaults-toml-extension, r=jyn514

    Rename bootstrap/defaults/{config.toml.PROFILE => config.PROFILE.toml}
    
    This allows these files to have okay syntax highlighting in editors, and helps avoid nagging from editors which want to suggest that I install a plugin for `*.library` files to view the `config.toml.library` or whatever.
    
    It's a very minor change.
    
    r?@jyn514
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    5b56541 View commit details
    Browse the repository at this point in the history
  14. Rollup merge of #77560 - rschoon:fix-litkind-rc-bytebuf, r=lcnr

    Fix LitKind's byte buffer to use refcounted slice
    
    While working on adding a new lint for clippy (see rust-lang/rust-clippy#6044) for avoiding shared ownership of "mutable buffer" types (such as using `Rc<Vec<T>>` instead of `Rc<[T]>`), I noticed a type exported from rustc_ast and used by clippy gets caught by the lint. This PR fixes the exported type.
    
    This PR includes the actual change to clippy too, but I will open a PR directly against clippy for that part (although it will currently fail to build there).
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    04d9ae4 View commit details
    Browse the repository at this point in the history
  15. Rollup merge of #77568 - lcnr:mir-inline-def-id, r=ecstatic-morse

    inliner: use caller param_env
    
    We used the callee param env instead of the caller param env by accident in #77430, this PR fixes that and caches it in the `Inliner` struct.
    
    fixes #77564
    
    r? @ecstatic-morse
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    6d1cc0d View commit details
    Browse the repository at this point in the history
  16. Rollup merge of #77571 - pickfire:patch-6, r=cramertj

    Use matches! for core::char methods
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    6fbda95 View commit details
    Browse the repository at this point in the history
  17. Rollup merge of #77587 - ehuss:unicode-escape-span, r=ecstatic-morse

    Fix span for unicode escape suggestion.
    
    If a unicode escape is missing the curly braces, the suggested fix is to add the curly braces, but the span for the fix was incorrect. It was not covering the `\u`, but the suggested text includes the `\u`, causing the resulting fix to be `"\u\u{1234}"`. This changes it so that the span includes the `\u`. An alternate fix would be to remove `\u` from the suggested fix, but I think the error message reads better if the entire escape is included.
    jonas-schievink authored Oct 5, 2020
    Configuration menu
    Copy the full SHA
    7af4959 View commit details
    Browse the repository at this point in the history