mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revise test slightly so that dyn in macro invocation *must* be parsed as keyword in test.

Browse Source 
Back-story: After reflection this morning, I realized that the
previous form of this test would allow the macro invocation to treat
the `dyn` input as a raw-identifier rather than a keyword, and since
the input was discarded by that version of the macro, the test would
pass despite the detail that the input `dyn` should not have been
parsed as a raw-identifier.

This revision fixes that oversight, by actually *using* the macro
input to construct a `Box<dyn Trait>` type.
This commit is contained in:
Felix S. Klock II 2019-03-28 12:49:50 +01:00
parent 1f63a52ca2
commit f043d2da65
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/test/ui/dyn-keyword/dyn-2015-edition-keyword-ident-lint.fixed
View File

@ -45,7 +45,7 @@ mod macro_defn {
//~| WARN was previously accepted
// Note that we do not lint nor fix occurrences under macros
($dyn:ident) => { Box<dyn Trait> }
($dyn:tt) => { (Box<dyn Trait>, Box<$dyn Trait>) }
}
pub fn r#dyn() -> ::outer_mod::r#dyn::r#dyn {
@ -72,7 +72,7 @@ mod macro_defn {
dyn
)
{
Box::new(10)
(Box::new(1), Box::new(2))
}
}

4
src/test/ui/dyn-keyword/dyn-2015-edition-keyword-ident-lint.rs
View File

@ -45,7 +45,7 @@ mod macro_defn {
//~| WARN was previously accepted
// Note that we do not lint nor fix occurrences under macros
($dyn:ident) => { Box<dyn Trait> }
($dyn:tt) => { (Box<dyn Trait>, Box<$dyn Trait>) }
}
pub fn dyn() -> ::outer_mod::dyn::dyn {
@ -72,7 +72,7 @@ mod macro_defn {
dyn
)
{
Box::new(10)
(Box::new(1), Box::new(2))
}
}