Skip to content

Reduce the amount of unstable features used in libproc_macro #95308

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 6 commits into from
Apr 9, 2022

Conversation

bjorn3
Copy link
Member

@bjorn3 bjorn3 commented Mar 25, 2022

This makes it easier to adapt the source for stable when copying it into rust-analyzer to load rustc compiled proc macros.

}

impl<'a, A, R> !Sync for Closure<'a, A, R> {}
impl<'a, A, R> !Send for Closure<'a, A, R> {}
struct Env;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rust-analyzer copies turn Env into a regular struct too, but don't replace &mut Env with *mut Env, which is UB according to stacked borrows.

@bjorn3
Copy link
Member Author

bjorn3 commented Mar 25, 2022

Looks like rustbot and gha are borked.

r? @Mark-Simulacrum as suggested by github

@bjorn3 bjorn3 force-pushed the more_stable_proc_macro branch from b06dbfe to 6702a9b Compare March 25, 2022 16:24
@bjorn3 bjorn3 force-pushed the more_stable_proc_macro branch from 6702a9b to 6eab980 Compare March 25, 2022 18:04
@bjorn3 bjorn3 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 26, 2022
@Mark-Simulacrum Mark-Simulacrum 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-review Status: Awaiting review from the assignee but also interested parties. labels Apr 6, 2022
@bjorn3 bjorn3 added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 6, 2022
@Mark-Simulacrum
Copy link
Member

@bors r+

@bjorn3
Copy link
Member Author

bjorn3 commented Apr 9, 2022

Bors didn't respond.

@bors r=Mark-Simulacrum

@bjorn3 bjorn3 closed this Apr 9, 2022
@bjorn3 bjorn3 reopened this Apr 9, 2022
@bjorn3
Copy link
Member Author

bjorn3 commented Apr 9, 2022

@bors r=Mark-Simulacrum

@bors
Copy link
Collaborator

bors commented Apr 9, 2022

📌 Commit 7eda975 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 9, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 9, 2022
Rollup of 6 pull requests

Successful merges:

 - rust-lang#95308 (Reduce the amount of unstable features used in libproc_macro)
 - rust-lang#95676 (Update RLS)
 - rust-lang#95769 (Hide cross-crate `#[doc(hidden)]` associated items in trait impls)
 - rust-lang#95785 (interpret: err instead of ICE on size mismatches in to_bits_or_ptr_internal)
 - rust-lang#95802 (fix unused constant warning on some Windows targets)
 - rust-lang#95810 (Use `format-args-capture` and remove unnecessary nested blocks)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit e232cb4 into rust-lang:master Apr 9, 2022
@rustbot rustbot added this to the 1.62.0 milestone Apr 9, 2022
@bjorn3 bjorn3 deleted the more_stable_proc_macro branch April 9, 2022 13:40
@bjorn3 bjorn3 mentioned this pull request May 13, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants