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 12 pull requests #89158

Merged
merged 26 commits into from
Sep 22, 2021
Merged

Rollup of 12 pull requests #89158

merged 26 commits into from
Sep 22, 2021

Commits on Sep 10, 2021

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

Commits on Sep 17, 2021

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

Commits on Sep 18, 2021

  1. EscapeDefault: change range field to Range<u8>, reducing struct s…

    …ize 24 -> 6 bytes
    klensy committed Sep 18, 2021
    Configuration menu
    Copy the full SHA
    cccd6e0 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    7c8f4f7 View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2021

  1. [bootstrap] Improve the error message when ninja is not found to li…

    …nk to installation instructions.
    
    Signed-off-by: Daira Hopwood <daira@jacaranda.org>
    daira committed Sep 19, 2021
    Configuration menu
    Copy the full SHA
    a627b4f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    23d6437 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2021

  1. Fix a technicality regarding the size of C's char type

    Specifically, ISO/IEC 9899:2018 — better known as "C18" — (and at least
    C11, C99 and C89) do not specify the size of `byte` in bits.
    Section 3.6 defines "byte" as "addressable unit of data storage" while
    section 6.2.5 ("Types") only defines "char" as "large enough to store
    any member of the basic execution set" giving it a lower bound of 7 bit
    (since there are 96 characters in the basic execution set).
    With section 6.5.3.4 paragraph 4 "When sizeof is applied to an operant
    that has type char […] the result is 1" you could read this as the size
    of `char` in bits being defined as exactly the same as the number of
    bits in a byte but it's also valid to read that as an exception.
    
    In general implementations take `char` as the smallest unit of
    addressable memory, which for modern byte-addressed architectures is
    overwhelmingly 8 bits to the point of this convention being completely
    cemented into just about all of our software.
    
    So is any of this actually relevant at all? I hope not. I sincerely hope
    that this never, ever comes up.
    But if for some reason a poor rustacean is having to interface with C
    code running on a Cray X1 that in 2003 is still doing word-addressed
    memory with 64-bit words and they trust the docs here blindly it will
    blow up in her face. And I'll be truly sorry for her to have to deal
    with … all of that.
    dequbed committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    23c608f View commit details
    Browse the repository at this point in the history
  2. no ensure

    BoxyUwU committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    5fdb9e4 View commit details
    Browse the repository at this point in the history
  3. ⬆️ rust-analyzer

    lnicola committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    514b8ba View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    402ebc7 View commit details
    Browse the repository at this point in the history

Commits on Sep 21, 2021

  1. Impl Error for FromSecsError without foreign type

    Using it through the crate-local path in `std` means that it shouldn't make an "Implementations on Foreign Types" section in the `std::error::Error` docs.
    mbartlett21 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    33766ae View commit details
    Browse the repository at this point in the history
  2. Fix match for placeholder region

    Ankit Chandawala committed Sep 21, 2021
    Configuration menu
    Copy the full SHA
    d4ee698 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    e4faf17 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    999888c View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#88795 - FabianWolff:issue-88684, r=wesleywiser

    Print a note if a character literal contains a variation selector
    
    Fixes rust-lang#88684.
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    c2cdba4 View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#89015 - klensy:escape-def, r=Mark-Simulacrum

    core::ascii::escape_default: reduce struct size
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    051168b View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#89078 - camsteffen:map-ref, r=cjgillot

    Cleanup: Remove needless reference in ParentHirIterator
    
    It forces an intermediate binding of `Map` which is a Copy type.
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    9f50c87 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#89086 - WaffleLapkin:stabilize_iter_map_whi…

    …le, r=kennytm
    
    Stabilize `Iterator::map_while`
    
    Per the FCP: rust-lang#68537 (comment)
    
    This PR stabilizes `Iterator::map_while` and `iter::MapWhile` in Rust 1.57.
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    d7de8d2 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#89096 - daira:improve-ninja-error-message, …

    …r=jyn514
    
    [bootstrap] Improve the error message when `ninja` is not found to link to installation instructions
    
    fixes rust-lang#89091
    
    Signed-off-by: Daira Hopwood <daira@jacaranda.org>
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    a8633eb View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#89113 - BoxyUwU:incr-comp-thir-act, r=lcnr

    dont `.ensure()` the `thir_abstract_const` query call in `mir_build`
    
    might fix an ICE seen in rust-lang#89022 (note: this PR does not close that issue) about attempting to read stolen thir. I couldn't repro the ICE but this `.ensure` seems sus anyway.
    
    r? `@lcnr`
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    ecfdadc View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#89114 - dequbed:c-char, r=yaahc

    Fixes a technicality regarding the size of C's `char` type
    
    Specifically, ISO/IEC 9899:2018 — better known as "C18" — (and at least
    C11, C99 and C89) do not specify the size of `byte` in bits.
    Section 3.6 defines "byte" as "addressable unit of data storage" while
    section 6.2.5 ("Types") only defines "char" as "large enough to store
    any member of the basic execution set" giving it a lower bound of 7 bit
    (since there are 96 characters in the basic execution set).
    With section 6.5.3.4 paragraph 4 "When sizeof is applied to an operant
    that has type char […] the result is 1" you could read this as the size
    of `char` in bits being defined as exactly the same as the number of
    bits in a byte but it's also valid to read that as an exception.
    
    In general implementations take `char` as the smallest unit of
    addressable memory, which for modern byte-addressed architectures is
    overwhelmingly 8 bits to the point of this convention being completely
    cemented into just about all of our software.
    
    So is any of this actually relevant at all? I hope not. I sincerely hope
    that this never, ever comes up.
    But if for some reason a poor rustacean is having to interface with C
    code running on a Cray X1 that in 2003 is still doing word-addressed
    memory with 64-bit chars and they trust the docs here blindly it will
    blow up in her face. And I'll be truly sorry for her to have to deal
    with … all of that.
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    8a6e9cf View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#89115 - lnicola:rust-analyzer-2021-09-20, r…

    …=Mark-Simulacrum
    
    ⬆️ rust-analyzer
    
    `@bors` r+ rollup
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    74cdd64 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#89126 - FabianWolff:issue-89088, r=petroche…

    …nkov
    
    Fix ICE when `indirect_structural_match` is allowed
    
    Fixes rust-lang#89088. The ICE is caused by `delay_good_path_bug()`, which is called (indirectly) from a `format!()` macro invocation. I have moved the macro invocation into the `decorate` closure of `struct_span_lint_hir()`, so that the macro is only invoked if the lint is not allowed (i.e., causes at least a warning, and thus prevents `delay_good_path_bug()` from firing).
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    8d95bb2 View commit details
    Browse the repository at this point in the history
  14. Rollup merge of rust-lang#89141 - mbartlett21:patch-2, r=kennytm

    Impl `Error` for `FromSecsError` without foreign type
    
    Using it through the crate-local path in `std` means that it shouldn't make an "Implementations on Foreign Types" section in the `std::error::Error` docs.
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    17c9a22 View commit details
    Browse the repository at this point in the history
  15. Rollup merge of rust-lang#89142 - nerandell:master, r=jackh726

    Fix match for placeholder region
    
    cc rust-lang#89118
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    aca790b View commit details
    Browse the repository at this point in the history
  16. Rollup merge of rust-lang#89147 - b-naber:refs_in_check_const_value_e…

    …q, r=oli-obk
    
    add case for checking const refs in check_const_value_eq
    
    Previously in `check_const_value_eq` we destructured `ConstValue::ByRef` instances, this didn't account for `ty::Ref`s however, which led to an ICE.
    
    Fixes rust-lang#88876
    Fixes rust-lang#88384
    
    r? `@oli-obk`
    the8472 authored Sep 21, 2021
    Configuration menu
    Copy the full SHA
    a3e6c19 View commit details
    Browse the repository at this point in the history