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

Closed
wants to merge 74 commits into from
Closed

Conversation

TaKO8Ki
Copy link
Member

@TaKO8Ki TaKO8Ki commented Aug 16, 2022

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

HKalbasi and others added 30 commits July 28, 2022 22:38
Previously added a blank _ => {} for match statements
fix: make `concat!` work with char

Fixes rust-lang#12921

- I avoided making `unquote_str()` take char literals as well because it's depended on by another function `parse_string()` that's only supposed to take strings.
- Even with this patch, we don't output `\0` as `\u{0}` which rust-lang#12921 pointed out ~~, but we're not actually responsible for serializing it but rowan is~~. They are functionally equivalent and I don't think it'd cause any confusion, but we *could* try escaping them before serialization (for reference, `rustc -Zunpretty=expanded`, which `cargo expand` uses under the hood, [makes use of `str::escape_default()`](https://github.com/rust-lang/rust/blob/3830ecaa8db798d2727cbdfa4ddf314ff938f268/compiler/rustc_ast/src/util/literal.rs#L161).
Add fixups for incomplete in proc-macros

Partially implements rust-lang/rust-analyzer#12777.

Added support for for loops and match statements.

I couldn't do paths like `crate::foo::` as I wasn't able to add `SyntheticTokens` to the end of `foo::`, they always ended up after `crate::`

This is my first contribution so please don't be shy about letting me know if I've done anything wrong!
Generate rust type from json

fix rust-lang#10118

Should this be a diagnostic? I made it a diagnostic because of issue label.
…Veykril

internal: Document CLI flag stability
[code] make toggleInlayHints understand {off,on}UntilPressed

fixes rust-lang#12964
…r=Veykril

Parse range patterns in struct and slice without trailing comma

Resolves rust-lang#12935

This patch includes the support for range patterns in slices, which is unstable (tracked in rust-lang#67264). If it's not desired I can remove it.
Corrected order of printing op and `=`

Fixes rust-lang/rust-analyzer#12971 where `Display` impl for assignment operators does `=+` instead of `+=`
Remove imports that are also in edition 2021's prelude

small cleanup
Fix pattern field completions not working for unions
Recover from missing ellipsis in record literals for path expressions
Veykril and others added 17 commits August 13, 2022 20:49
…kril

Pad empty diagnostic messages in relatedInformation as well

Follw up to rust-lang/rust-analyzer#13016
Take into account renamed extension id when launching
minor: Remove redundant --pre-release flag from publish
There is some redundancy here, but the extra assertions make it easier
to keep track of relative things, e.g. `ExprKind` is the biggest part
of `Expr`.
nonexistant -> nonexistent
…acrum

unwind: don't build dependency when building for Miri

This is basically re-submitting rust-lang#94813.

In that PR there was a suggestion to instead have bootstrap set a `RUST_CHECK` env var and use that rather than doing something Miri-specific. However, such an env var would mean that when switching between `./x.py check` and `./x.py build`, the build script gets re-run each time, which doesn't seem good. So I think for now checking for Miri probably causes fewer problems.

r? `@Mark-Simulacrum`
… r=spastorino

Ast and parser tweaks

r? ``@spastorino``
compiletest: fix typo in runtest.rs

nonexistant -> nonexistent
…r=lnicola

⬆️ rust-analyzer

r? ``@ghost``
Support 128-bit atomics on all aarch64 targets

Some aarch64 targets currently set `max_atomic_width` to 64, but aarch64 always supports 128-bit atomics.

r? ``@Amanieu``
@rustbot rustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Aug 16, 2022
@TaKO8Ki
Copy link
Member Author

TaKO8Ki commented Aug 16, 2022

@bors r+ rollup=never p=5

@bors
Copy link
Contributor

bors commented Aug 16, 2022

📌 Commit 7e100c1 has been approved by TaKO8Ki

It is now in the queue for this repository.

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Aug 16, 2022
@matthiaskrgr
Copy link
Member

This overlaps with #100626 🤔

@TaKO8Ki TaKO8Ki closed this Aug 16, 2022
@TaKO8Ki TaKO8Ki reopened this Aug 16, 2022
@TaKO8Ki
Copy link
Member Author

TaKO8Ki commented Aug 16, 2022

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 16, 2022
@TaKO8Ki TaKO8Ki closed this Aug 16, 2022
@TaKO8Ki TaKO8Ki deleted the rollup-snnmshj branch August 16, 2022 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.