Get diesel_derives2 working with hygiene turned on #1521
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When `proc_macro2` has the nightly feature enabled, a lot of the rules around hygiene change. Most of the changes needed appear to be due to bugs in Rust (e.g. to access a tuple struct field, the span of the dot has to be resolved specially, but not for a named struct field), but the span changes for table names in particular is definitely required. The change to hygiene rules randomly broke our const workaround (the ability to `use` items defined in an expression like that was potentially unintended to begin with). However, there's no reason it needs to be a const, we can just have it be a module with no additional problems. I've added a few UI tests for the failure cases that are actually affected by this change (there is more work to be done on this, of course). Note that we can't actually run those tests yet, as they require a nightly compiler, and we won't build on nightly until tomorrow. However, I've left the tests in place so that the changes in output can be seen. The error for `table_name` isn't quite there, but due to a bug in Rust it's the best we can do. Right now it looks like this: ``` 1 | #[table_name = "users"] | ^ Use of undeclared type or module `users` ``` instead of like this: ``` 1 | #[table_name = "users"] | ^^^^^^^ Use of undeclared type or module `users` ```
|
/cc @dtolnay |
|
Note: Build is going to be red until tomorrow. Nothing I can do about that, but this should be fine to review |
|
What happens tomorrow? |
|
rust-lang/rust#47738 will be in tomorrow's nightly. |
dtolnay
reviewed
Feb 2, 2018
| use proc_macro2::{Spacing, TokenNode, TokenTree}; | ||
|
|
||
| // https://github.com/rust-lang/rust/issues/47312 | ||
| tokens.append(TokenTree { |
There was a problem hiding this comment.
quote_spanned! can make this more concise but it is up to you whether you prefer the explicit way or the concise way.
quote_spanned!(call_site=> .)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When
proc_macro2has the nightly feature enabled, a lot of the rulesaround hygiene change. Most of the changes needed appear to be due to
bugs in Rust (e.g. to access a tuple struct field, the span of the dot has to
be resolved specially, but not for a named struct field), but the span
changes for table names in particular is definitely required.
The change to hygiene rules randomly broke our const workaround (the
ability to
useitems defined in an expression like that waspotentially unintended to begin with). However, there's no reason it
needs to be a const, we can just have it be a module with no additional
problems.
I've added a few UI tests for the failure cases that are actually
affected by this change (there is more work to be done on this, of
course). Note that we can't actually run those tests yet, as they
require a nightly compiler, and we won't build on nightly until
tomorrow. However, I've left the tests in place so that the changes in
output can be seen.
The error for
table_nameisn't quite there, but due to a bug in Rustit's the best we can do. Right now it looks like this:
instead of like this: