-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
use TokenStream rather than &[TokenTree] for built-in macros #64041
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
is this a good idea in the first place? If it's good, I'll go and fix all other macros |
I think we'll need something like this for #63689. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
I see a 👍 from @matthewjasper , so I'll move forward with this :) |
ready for review @rustbot modify labels to: -S-waiting-on-author, +S-waiting-on-review |
Error: Parsing label command in comment failed: ... labels to|error: a label delta at >|: -S-waiti... Please let |
This comment has been minimized.
This comment has been minimized.
@rustbot modify labels to -S-waiting-on-author, +S-waiting-on-review Yeah, the Edit: And, pushed a commit with a fix and will try and deploy in an hour or so. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
625a27c
to
610451f
Compare
addressed the comments. Note that this changes (unstable) compiler plugin API. @rustbot modify labels: +S-waiting-on-review, -S-waiting-on-author |
@bors r+ |
📌 Commit 610451f9246b82cdb8bb220219ed9d8674343124 has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
That way, we don't loose the jointness info
610451f
to
6136495
Compare
@bors r=petrochenkov |
📌 Commit 6136495 has been approved by |
…nkov use TokenStream rather than &[TokenTree] for built-in macros That way, we don't loose the jointness info
Rollup of 10 pull requests Successful merges: - #63166 (Add Result::cloned{,_err} and Result::copied{,_err}) - #63930 (Account for doc comments coming from proc macros without spans) - #63985 (Stabilize pin_into_inner in 1.39.0) - #64023 (libstd fuchsia fixes) - #64030 (Fix unlock ordering in SGX synchronization primitives) - #64041 (use TokenStream rather than &[TokenTree] for built-in macros) - #64043 (Add some more tests for underscore imports) - #64092 (Update xLTO compatibility table in rustc book.) - #64120 (Move path parsing earlier) - #64123 (Added warning around code with reference to uninit bytes) Failed merges: r? @ghost
…nkov use TokenStream rather than &[TokenTree] for built-in macros That way, we don't loose the jointness info
…nkov use TokenStream rather than &[TokenTree] for built-in macros That way, we don't loose the jointness info
…nkov use TokenStream rather than &[TokenTree] for built-in macros That way, we don't loose the jointness info
Rollup of 11 pull requests Successful merges: - #62848 (Use unicode-xid crate instead of libcore) - #63774 (Fix `window.hashchange is not a function`) - #63930 (Account for doc comments coming from proc macros without spans) - #64003 (place: Passing `align` = `layout.align.abi`, when also passing `layout`) - #64030 (Fix unlock ordering in SGX synchronization primitives) - #64041 (use TokenStream rather than &[TokenTree] for built-in macros) - #64051 (Add x86_64-linux-kernel target) - #64063 (Fix const_err with `-(-0.0)`) - #64083 (Point at appropriate arm on type error on if/else/match with one non-! arm) - #64100 (Fix const eval bug breaking run-pass tests in Miri) - #64157 (Opaque type locations in error message for clarity.) Failed merges: r? @ghost
Fix test for changes in plugin API. Updates for rust-lang/rust#64041
That way, we don't loose the jointness info