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 5 pull requests #84368

Closed
wants to merge 16 commits into from

Commits on Mar 25, 2021

  1. Configuration menu
    Copy the full SHA
    30fc601 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    321aace View commit details
    Browse the repository at this point in the history
  3. Updating test stderr files

    sunjay committed Mar 25, 2021
    Configuration menu
    Copy the full SHA
    7faaf39 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    789186d View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    2acd8eb View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    3e34eb8 View commit details
    Browse the repository at this point in the history

Commits on Apr 1, 2021

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

Commits on Apr 20, 2021

  1. rustdoc: use details tag for trait implementors

    This switches from JS-generated toggles to using the HTML <details> tag
    for expanding and collapsing entries in the "Implementors" section.
    jsha committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    569096c View commit details
    Browse the repository at this point in the history
  2. ⬆️ rust-analyzer

    lnicola committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    e85f19b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    8cc918a View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#83004 - sunjay:field-never-read-issue-81658…

    …, r=pnkfelix
    
    Improve diagnostic for when field is never read
    
    Related to (but does not close) rust-lang#81658
    
    This completes the first step of `@pnkfelix's` [mentoring instructions](rust-lang#81658 (comment)) but does not actually improve the diagnostics (yet!). The two tests are heavily reduced versions of code from the original bug report.
    
    I've confirmed that the reduced `field-used-in-ffi` test [fails on nightly](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=f0862c89ddca028c55c20a5ed05e679a) but [passes on stable](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=f0862c89ddca028c55c20a5ed05e679a). This confirms that the regression is reproduced correctly. The `drop-only-field` test is a case that `@pnkfelix` mentioned in his mentoring instructions. It is not a regression, but will come in handy when we make the diagnostic smarter by looking at whether the field type implements `Drop`.
    
    Per the [rustc-dev-guide](https://rustc-dev-guide.rust-lang.org/tests/adding.html), each test includes a comment summarizing what it is about.
    GuillaumeGomez committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    7f561d8 View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#83729 - JohnTitor:issue-43913, r=estebank

    Add a suggestion when using a type alias instead of trait alias
    
    Fixes rust-lang#43913
    
    r? `@estebank`
    GuillaumeGomez committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    4e43d20 View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#84320 - jsha:details-implementors, r=Manish…

    …earth,Nemo157,GuillaumeGomez
    
    Use details tag for trait implementors.
    
    Part of rust-lang#83332 and following on from rust-lang#83337 and rust-lang#83355.
    
    This removes one category of JS-generated toggles (implementors), and replaces them with a `<details>` tag. This simplifies the JS, and fixes some bugs where things that were supposed to be hidden by the toggle were not hidden. Compare https://hoffman-andrews.com/rust/details-implementors/std/io/trait.Read.html#impl-Read vs https://doc.rust-lang.org/nightly/std/io/trait.Read.html#implementors.
    
    This introduces a `left: -23px` to put the toggle in the correct place, matching the current style for `.collapse-toggle`.
    
    It's worth noting this introduces a slight behavior change: since the entire line is now a `<summary>`, any part of the line is clickable. So for instance, in `impl Read for File`, clicking `impl` or `for` will collapse / expand the docs. Clicking `Read` or `File` still links to the appropriate documentation as before.
    GuillaumeGomez committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    c8e7052 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#84359 - lnicola:rust-analyzer-2021-04-20, r…

    …=jonas-schievink
    
    ⬆️ rust-analyzer
    GuillaumeGomez committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    d6a2b03 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#84365 - vext01:improve-lto-docstrings, r=pe…

    …trochenkov
    
    Improve the docstrings of the `Lto` struct.
    
    This change is the result of [this zulip discussion](https://rust-lang.zulipchat.com/#narrow/stream/182449-t-compiler.2Fhelp/topic/Making.20sense.20of.20LTO.20modes.20in.20rustc).
    
    Hopefully it makes things a little clearer. What do you think?
    GuillaumeGomez committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    b58e1f2 View commit details
    Browse the repository at this point in the history