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

Use opt_def_id() over def_id()

Browse Source
This commit is contained in:
Yuki Okushi 2020-03-01 15:15:29 +09:00
parent d3c79346a3
commit f9c8c49c0b
3 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/librustc_save_analysis/lib.rs
View File

@ -816,7 +816,7 @@ pub fn get_macro_use_data(&self, span: Span) -> Option<MacroRef> {
fn lookup_def_id(&self, ref_id: NodeId) -> Option<DefId> {
match self.get_path_res(ref_id) {
Res::PrimTy(_) | Res::SelfTy(..) | Res::Err => None,
def => Some(def.def_id()),
def => def.opt_def_id(),
}
}

2
src/test/ui/attributes/register-attr-tool-import.rs
View File

@ -1,4 +1,6 @@
// edition:2018
// compile-flags: -Zsave-analysis
// ~^ Also regression test for #69588
#![feature(register_attr)]
#![feature(register_tool)]

12
src/test/ui/attributes/register-attr-tool-import.stderr
View File

@ -1,35 +1,35 @@
error: cannot use an explicitly registered attribute through an import
--> $DIR/register-attr-tool-import.rs:12:3
--> $DIR/register-attr-tool-import.rs:14:3
|
LL | #[renamed_attr]
| ^^^^^^^^^^^^
|
note: the explicitly registered attribute imported here
--> $DIR/register-attr-tool-import.rs:9:5
--> $DIR/register-attr-tool-import.rs:11:5
|
LL | use attr as renamed_attr; // OK
| ^^^^^^^^^^^^^^^^^^^^
error: cannot use a tool module through an import
--> $DIR/register-attr-tool-import.rs:13:3
--> $DIR/register-attr-tool-import.rs:15:3
|
LL | #[renamed_tool::attr]
| ^^^^^^^^^^^^
|
note: the tool module imported here
--> $DIR/register-attr-tool-import.rs:10:5
--> $DIR/register-attr-tool-import.rs:12:5
|
LL | use tool as renamed_tool; // OK
| ^^^^^^^^^^^^^^^^^^^^
error: cannot use a tool module through an import
--> $DIR/register-attr-tool-import.rs:13:3
--> $DIR/register-attr-tool-import.rs:15:3
|
LL | #[renamed_tool::attr]
| ^^^^^^^^^^^^
|
note: the tool module imported here
--> $DIR/register-attr-tool-import.rs:10:5
--> $DIR/register-attr-tool-import.rs:12:5
|
LL | use tool as renamed_tool; // OK
| ^^^^^^^^^^^^^^^^^^^^