tests/ui/macros: add annotations for reference rules#153482
tests/ui/macros: add annotations for reference rules#153482DanielEScherzer wants to merge 1 commit intorust-lang:mainfrom
Conversation
rustbot
added
S-waiting-on-review
| }} | ||
| m1!(); | ||
|
|
||
| foo!(); //~ ERROR `foo` is ambiguous |
There was a problem hiding this comment.
It seems like this is testing names.resolution.expansion.macros.ambiguity.more-expanded-vs-outer.
| macro_rules! m1 { () => { | ||
| macro_rules! foo { () => {} } | ||
|
|
||
| #[macro_use] //~ ERROR `macro_two` is already in scope |
There was a problem hiding this comment.
Filed rust-lang/reference#2207, as I couldn't find a rule for this specifically.
| @@ -1,3 +1,4 @@ | |||
| //@ reference: attributes.derive.behavior | |||
There was a problem hiding this comment.
I feel like this test is more closely showing that macro_rules macros are in a different namespace than attributes or derive macros. There is names.namespaces.sub-namespaces.intro which describes the distinction between attribute macros and fn-like macros.
There isn't such a clear-cut description for derive macros, and I'm wondering if the sub-namespace section needs some clarification to include that.
|
☔ The latest upstream changes (presumably #154384) made this pull request unmergeable. Please resolve the merge conflicts. |
3 participants
r? ehuss
@rustbot label +A-docs