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

Merged
merged 32 commits into from
Dec 11, 2020
Merged

Rollup of 11 pull requests #79927

merged 32 commits into from
Dec 11, 2020

Commits on Sep 21, 2020

  1. Edit documentation for std::{f32,f64}::mul_add.

    Makes it more clear that a performance improvement is not guaranteed
    when using FMA, even when the target architecture supports it natively.
    fu5ha committed Sep 21, 2020
    Configuration menu
    Copy the full SHA
    72a5cbb View commit details
    Browse the repository at this point in the history

Commits on Oct 13, 2020

  1. generalize warning

    fu5ha committed Oct 13, 2020
    Configuration menu
    Copy the full SHA
    a6d98d8 View commit details
    Browse the repository at this point in the history

Commits on Dec 3, 2020

  1. Make the kernel_copy tests more robust/concurrent.

    These tests write to the same filenames in /tmp and in some cases these
    files don't get cleaned up properly. This caused issues for us when
    different users run the tests on the same system, e.g.:
    
    ```
    ---- sys::unix::kernel_copy::tests::bench_file_to_file_copy stdout ----
    thread 'sys::unix::kernel_copy::tests::bench_file_to_file_copy' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }', library/std/src/sys/unix/kernel_copy/tests.rs:71:10
    ---- sys::unix::kernel_copy::tests::bench_file_to_socket_copy stdout ----
    thread 'sys::unix::kernel_copy::tests::bench_file_to_socket_copy' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }', library/std/src/sys/unix/kernel_copy/tests.rs:100:10
    ```
    
    Use `std::sys_common::io__test::tmpdir()` to solve this.
    vext01 committed Dec 3, 2020
    Configuration menu
    Copy the full SHA
    87c1fdb View commit details
    Browse the repository at this point in the history

Commits on Dec 4, 2020

  1. Refine E0212 error message

    sasurau4 committed Dec 4, 2020
    Configuration menu
    Copy the full SHA
    36363e5 View commit details
    Browse the repository at this point in the history
  2. Add long explanation for E0212

    Update compiler/rustc_error_codes/src/error_codes/E0212.md
    
    Co-authored-by: Joshua Nelson <[email protected]>
    sasurau4 and Joshua Nelson committed Dec 4, 2020
    Configuration menu
    Copy the full SHA
    a1e94cd View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    87c6216 View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2020

  1. Configuration menu
    Copy the full SHA
    12db222 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    d6baf38 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7bd47bd View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    85e9ea0 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    d2de69d View commit details
    Browse the repository at this point in the history

Commits on Dec 8, 2020

  1. Review feedback for collect_intra_doc_links.rs

    * Add assertion value is defined.
    * Simplify comment.
    * Fix bad change in err message.
    Eric-Arellano committed Dec 8, 2020
    Configuration menu
    Copy the full SHA
    f68cc68 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a3174de View commit details
    Browse the repository at this point in the history
  3. Review feedback

    * Use a match statement.
    * Clarify why we can't use `file_stem()`.
    * Error if the `:` is missing for Tidy error codes, rather than no-oping.
    Eric-Arellano committed Dec 8, 2020
    Configuration menu
    Copy the full SHA
    989edf4 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2020

  1. Clarify the 'default is only allowed on...' error

    Code like
    
        impl Foo {
            default fn foo() {}
        }
    
    will trigger the error
    
        error: `default` is only allowed on items in `impl` definitions
         --> src/lib.rs:5:5
          |
        5 |     default fn foo() {}
          |     -------^^^^^^^^^
          |     |
          |     `default` because of this
    
    but that's very confusing! I *did* put it on an item in an impl!
    
    So this commit changes the message to
    
        error: `default` is only allowed on items in trait impls
         --> src/lib.rs:5:5
          |
        5 |     default fn foo() {}
          |     -------^^^^^^^^^
          |     |
          |     `default` because of this
    camelid committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    4e21942 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    33ae62c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    56d9784 View commit details
    Browse the repository at this point in the history

Commits on Dec 10, 2020

  1. Configuration menu
    Copy the full SHA
    2363a20 View commit details
    Browse the repository at this point in the history
  2. Update const-fn doc in unstable-book

    Update src/doc/unstable-book/src/language-features/const-fn.md
    
    Co-authored-by: Ivan Tham <[email protected]>
    sasurau4 and pickfire committed Dec 10, 2020
    Configuration menu
    Copy the full SHA
    caab16f View commit details
    Browse the repository at this point in the history
  3. Use Symbol for inline asm register class names

    This takes care of one "FIXME":
    // FIXME: use direct symbol comparison for register class names
    
    Instead of using string literals, this uses Symbol for register
    class names.
    Arlie Davis committed Dec 10, 2020
    Configuration menu
    Copy the full SHA
    40ed0f6 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    f7306b1 View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2020

  1. Rollup merge of rust-lang#77027 - termhn:mul_add_doc_change, r=m-ou-se

    Improve documentation for `std::{f32,f64}::mul_add`
    
    Makes it more clear that performance improvement is not guaranteed when using FMA, even when the target architecture supports it natively.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    1b4ffe4 View commit details
    Browse the repository at this point in the history
  2. Rollup merge of rust-lang#79375 - vext01:kernel-copy-temps, r=bjorn3

    Make the kernel_copy tests more robust/concurrent.
    
    These tests write to the same filenames in /tmp and in some cases these files don't get cleaned up properly. This caused issues for us when different users run the tests on the same system, e.g.:
    
    ```
    ---- sys::unix::kernel_copy::tests::bench_file_to_file_copy stdout ----
    thread 'sys::unix::kernel_copy::tests::bench_file_to_file_copy' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }', library/std/src/sys/unix/kernel_copy/tests.rs:71:10
    ---- sys::unix::kernel_copy::tests::bench_file_to_socket_copy stdout ----
    thread 'sys::unix::kernel_copy::tests::bench_file_to_socket_copy' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }', library/std/src/sys/unix/kernel_copy/tests.rs💯10
    ```
    
    Use `std::sys_common::io__test::tmpdir()` to solve this.
    
    CC ``@the8472.``
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    a8c19e1 View commit details
    Browse the repository at this point in the history
  3. Rollup merge of rust-lang#79639 - sasurau4:feature/add-long-explanati…

    …on-E0212, r=GuillaumeGomez
    
    Add long explanation for E0212
    
    Helps with rust-lang#61137
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    f3a3fc9 View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#79698 - m-ou-se:libs-tracking-issue-templat…

    …e, r=KodrAus
    
    Add tracking issue template for library features.
    
    This adds a issue template for a library tracking issue.
    
    There's already a template for tracking issues, but it's mostly geared towards compiler/language features. A separate template makes it a bit easier to make sure it matches with the process we use for library changes.
    
    Main differences:
    - Added a note about how small library features can be added without RFC, and removed the parts that assume there's an RFC.
    - Merged the 'Steps' and 'History' sections: Library features are often small enough that there's no multiple steps planned ahead of time.
    - Removed the section about avoiding large discussions and opening separate issues for problems with the feature. Library features are usually focussed enough that the discussion about a feature is best kept together in the tracking issue.
    - Removed links to the rustc-dev-guide, which are specific to changes in the compiler and language.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    8b9a59c View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#79809 - Eric-Arellano:split-once, r=matklad

    Dogfood `str_split_once()`
    
    Part of rust-lang#74773.
    
    Beyond increased clarity, this fixes some instances of a common confusion with how `splitn(2)` behaves: the first element will always be `Some()`, regardless of the delimiter, and even if the value is empty.
    
    Given this code:
    
    ```rust
    fn main() {
        let val = "...";
        let mut iter = val.splitn(2, '=');
        println!("Input: {:?}, first: {:?}, second: {:?}", val, iter.next(), iter.next());
    }
    ```
    
    We get:
    
    ```
    Input: "no_delimiter", first: Some("no_delimiter"), second: None
    Input: "k=v", first: Some("k"), second: Some("v")
    Input: "=", first: Some(""), second: Some("")
    ```
    
    Using `str_split_once()` makes more clear what happens when the delimiter is not found.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    17ec4b8 View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#79851 - camelid:better-error-for-default-fn…

    …, r=davidtwco
    
    Clarify the 'default is only allowed on...' error
    
    Code like
    
        impl Foo {
            default fn foo() {}
        }
    
    will trigger the error
    
        error: `default` is only allowed on items in `impl` definitions
         --> src/lib.rs:5:5
          |
        5 |     default fn foo() {}
          |     -------^^^^^^^^^
          |     |
          |     `default` because of this
    
    but that's very confusing! I *did* put it on an item in an impl!
    
    So this commit changes the message to
    
        error: `default` is only allowed on items in trait impls
         --> src/lib.rs:5:5
          |
        5 |     default fn foo() {}
          |     -------^^^^^^^^^
          |     |
          |     `default` because of this
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    dc90573 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#79858 - sasurau4:doc/update-unstable-book-c…

    …onst-fn, r=oli-obk
    
    Update const-fn doc in unstable-book
    
    Fix rust-lang#79691
    
    I couldn't find suitable examples. It seems that `const_fn` feature-gate used only following place. https://github.com/rust-lang/rust/blob/810324d1f31eb8d75e8f0044df720652986ef133/compiler/rustc_ast_passes/src/feature_gate.rs#L560-L562
    
    And example like following emits [E0379](https://doc.rust-lang.org/error-index.html#E0379).
    
    ```rust
    #![feature(const_fn)]
    
    trait Foo {
        const fn bar() -> Self;
    }
    ```
    
    Any other suitable example exists, please let me know.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    8709ac8 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#79860 - rust-lang:frewsxcv-patch-2, r=jyn514

    Clarify that String::split_at takes a byte index.
    
    To someone skimming through the `String` docs and only reads the first line, the person could interpret "index" to be "char index". Later on in the docs it clarifies, but by adding "byte" it removes that ambiguity.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    c0cc910 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#79871 - Pratyush:patch-1, r=joshtriplett

    Fix small typo in `wrapping_shl` documentation
    
    Fixes a small typo in the documentation.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    c94345e View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#79896 - GuillaumeGomez:more-elements-focus,…

    … r=Manishearth
    
    Make search results tab and help button focusable with keyboard
    
    Fixes rust-lang#79859.
    
    I replaced the element with `button` tag, which allows to focus them (and "click" on them using "enter") using only the keyboard.
    
    cc ``@sersorrel``
    
    r? ``@Manishearth``
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    3a46a6b View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#79917 - sivadeilra:asm_symbols, r=petrochenkov

    Use Symbol for inline asm register class names
    
    This takes care of one "FIXME":
    // FIXME: use direct symbol comparison for register class names
    
    Instead of using string literals, this uses Symbol for register
    class names.
    
    This is part of work I am doing to improve how Symbol interning works.
    tmandry authored Dec 11, 2020
    Configuration menu
    Copy the full SHA
    0327b5d View commit details
    Browse the repository at this point in the history