Tweak output of import suggestions

estebank · estebank · commit 5de8e6edfcc7 · 2024-06-13T20:22:21.000Z
When both `std::` and `core::` items are available, only suggest the `std::` ones. We ensure that in `no_std` crates we suggest `core::` items. Ensure that the list of items suggested to be imported are always in the order of local crate items, `std`/`core` items and finally foreign crate items. Tweak wording of import suggestion: if there are multiple items but they are all of the same kind, we use the kind name and not the generic "items". Fix #83564.
diff --git a/compiler/rustc_resolve/src/diagnostics.rs b/compiler/rustc_resolve/src/diagnostics.rs
@@ -2783,33 +2783,65 @@ fn show_candidates(
     // by iterating through a hash map, so make sure they are ordered:
     for path_strings in [&mut accessible_path_strings, &mut inaccessible_path_strings] {
         path_strings.sort_by(|a, b| a.0.cmp(&b.0));
+        path_strings.dedup_by(|a, b| a.0 == b.0);
         let core_path_strings =
             path_strings.extract_if(|p| p.0.starts_with("core::")).collect::<Vec<_>>();
-        path_strings.extend(core_path_strings);
-        path_strings.dedup_by(|a, b| a.0 == b.0);
+        let std_path_strings =
+            path_strings.extract_if(|p| p.0.starts_with("std::")).collect::<Vec<_>>();
+        let foreign_crate_path_strings =
+            path_strings.extract_if(|p| !p.0.starts_with("crate::")).collect::<Vec<_>>();
+
+        // We list the `crate` local paths first.
+        // Then we list the `std`/`core` paths.
+        if std_path_strings.len() == core_path_strings.len() {
+            // Do not list `core::` paths if we are already listing the `std::` ones.
+            path_strings.extend(std_path_strings);
+        } else {
+            path_strings.extend(std_path_strings);
+            path_strings.extend(core_path_strings);
+        }
+        // List all paths from foreign crates last.
+        path_strings.extend(foreign_crate_path_strings);
     }
-    accessible_path_strings.sort();
 
     if !accessible_path_strings.is_empty() {
-        let (determiner, kind, name, through) =
+        let (determiner, kind, s, name, through) =
             if let [(name, descr, _, _, via_import)] = &accessible_path_strings[..] {
                 (
                     "this",
                     *descr,
+                    "",
                     format!(" `{name}`"),
                     if *via_import { " through its public re-export" } else { "" },
                 )
             } else {
-                ("one of these", "items", String::new(), "")
+                // Get the unique item kinds and if there's only one, we use the right kind name
+                // instead of the more generic "items".
+                let mut kinds = accessible_path_strings
+                    .iter()
+                    .map(|(_, descr, _, _, _)| *descr)
+                    .collect::<FxHashSet<&str>>()
+                    .into_iter();
+                let kind = if let Some(kind) = kinds.next()
+                    && let None = kinds.next()
+                {
+                    kind
+                } else {
+                    "item"
+                };
+                let s = if kind.ends_with('s') { "es" } else { "s" };
+
+                ("one of these", kind, s, String::new(), "")
             };
 
         let instead = if let Instead::Yes = instead { " instead" } else { "" };
         let mut msg = if let DiagMode::Pattern = mode {
             format!(
-                "if you meant to match on {kind}{instead}{name}, use the full path in the pattern",
+                "if you meant to match on {kind}{s}{instead}{name}, use the full path in the \
+                 pattern",
             )
         } else {
-            format!("consider importing {determiner} {kind}{through}{instead}")
+            format!("consider importing {determiner} {kind}{s}{through}{instead}")
         };
 
         for note in accessible_path_strings.iter().flat_map(|cand| cand.3.as_ref()) {
diff --git a/src/tools/clippy/tests/ui/crashes/ice-6252.stderr b/src/tools/clippy/tests/ui/crashes/ice-6252.stderr
@@ -4,9 +4,7 @@ error[E0412]: cannot find type `PhantomData` in this scope
 LL |     _n: PhantomData,
    |         ^^^^^^^^^^^ not found in this scope
    |
-help: consider importing one of these items
-   |
-LL + use core::marker::PhantomData;
+help: consider importing this struct
    |
 LL + use std::marker::PhantomData;
    |
diff --git a/tests/ui/const-generics/issues/issue-82956.stderr b/tests/ui/const-generics/issues/issue-82956.stderr
@@ -4,7 +4,7 @@ error[E0433]: failed to resolve: use of undeclared type `IntoIter`
 LL |         let mut iter = IntoIter::new(self);
    |                        ^^^^^^^^ use of undeclared type `IntoIter`
    |
-help: consider importing one of these items
+help: consider importing one of these structs
    |
 LL + use std::array::IntoIter;
    |
diff --git a/tests/ui/consts/const_refs_to_static-ice-121413.rs b/tests/ui/consts/const_refs_to_static-ice-121413.rs
@@ -5,12 +5,16 @@
 // ignore-tidy-linelength
 #![feature(const_refs_to_static)]
 const REF_INTERIOR_MUT: &usize = {
+    //~^ HELP consider importing this struct
     static FOO: Sync = AtomicUsize::new(0);
     //~^ ERROR failed to resolve: use of undeclared type `AtomicUsize`
     //~| WARN trait objects without an explicit `dyn` are deprecated
     //~| ERROR the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
     //~| ERROR the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
     //~| WARN this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021!
+    //~| HELP if this is an object-safe trait, use `dyn`
+    //~| HELP the trait `Sized` is not implemented for `(dyn Sync + 'static)`
+    //~| HELP the trait `Sized` is not implemented for `(dyn Sync + 'static)`
     unsafe { &*(&FOO as *const _ as *const usize) }
 };
 pub fn main() {}
diff --git a/tests/ui/consts/const_refs_to_static-ice-121413.stderr b/tests/ui/consts/const_refs_to_static-ice-121413.stderr
@@ -1,5 +1,5 @@
 error[E0433]: failed to resolve: use of undeclared type `AtomicUsize`
-  --> $DIR/const_refs_to_static-ice-121413.rs:8:24
+  --> $DIR/const_refs_to_static-ice-121413.rs:9:24
    |
 LL |     static FOO: Sync = AtomicUsize::new(0);
    |                        ^^^^^^^^^^^ use of undeclared type `AtomicUsize`
@@ -10,7 +10,7 @@ LL + use std::sync::atomic::AtomicUsize;
    |
 
 warning: trait objects without an explicit `dyn` are deprecated
-  --> $DIR/const_refs_to_static-ice-121413.rs:8:17
+  --> $DIR/const_refs_to_static-ice-121413.rs:9:17
    |
 LL |     static FOO: Sync = AtomicUsize::new(0);
    |                 ^^^^
@@ -24,15 +24,15 @@ LL |     static FOO: dyn Sync = AtomicUsize::new(0);
    |                 +++
 
 error[E0277]: the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
-  --> $DIR/const_refs_to_static-ice-121413.rs:8:17
+  --> $DIR/const_refs_to_static-ice-121413.rs:9:17
    |
 LL |     static FOO: Sync = AtomicUsize::new(0);
    |                 ^^^^ doesn't have a size known at compile-time
    |
    = help: the trait `Sized` is not implemented for `(dyn Sync + 'static)`
 
 error[E0277]: the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
-  --> $DIR/const_refs_to_static-ice-121413.rs:8:24
+  --> $DIR/const_refs_to_static-ice-121413.rs:9:24
    |
 LL |     static FOO: Sync = AtomicUsize::new(0);
    |                        ^^^^^^^^^^^^^^^^^^^ doesn't have a size known at compile-time
diff --git a/tests/ui/did_you_mean/issue-56028-there-is-an-enum-variant.stderr b/tests/ui/did_you_mean/issue-56028-there-is-an-enum-variant.stderr
@@ -22,7 +22,7 @@ error[E0425]: cannot find value `Set` in this scope
 LL | fn setup() -> Set { Set }
    |                     ^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these unit variants
    |
 LL + use AffixHeart::Set;
    |
diff --git a/tests/ui/hygiene/extern-prelude-from-opaque-fail-2018.stderr b/tests/ui/hygiene/extern-prelude-from-opaque-fail-2018.stderr
@@ -24,8 +24,7 @@ LL |         fn f() { my_core::mem::drop(0); }
 LL | a!();
    | ---- in this macro invocation
    |
-   = help: consider importing one of these items:
-           core::mem
+   = help: consider importing this module:
            std::mem
    = note: this error originates in the macro `a` (in Nightly builds, run with -Z macro-backtrace for more info)
 
@@ -35,9 +34,7 @@ error[E0433]: failed to resolve: use of undeclared crate or module `my_core`
 LL |     fn f() { my_core::mem::drop(0); }
    |              ^^^^^^^ use of undeclared crate or module `my_core`
    |
-help: consider importing one of these items
-   |
-LL +     use core::mem;
+help: consider importing this module
    |
 LL +     use std::mem;
    |
diff --git a/tests/ui/imports/cycle-import-in-std-1.stderr b/tests/ui/imports/cycle-import-in-std-1.stderr
@@ -4,8 +4,7 @@ error[E0432]: unresolved import `ops`
 LL | use ops::{self as std};
    |           ^^^^^^^^^^^ no external crate `ops`
    |
-   = help: consider importing one of these items instead:
-           core::ops
+   = help: consider importing this module instead:
            std::ops
 
 error: aborting due to 1 previous error
diff --git a/tests/ui/imports/cycle-import-in-std-2.stderr b/tests/ui/imports/cycle-import-in-std-2.stderr
@@ -4,8 +4,7 @@ error[E0432]: unresolved import `ops`
 LL | use ops::{self as std};
    |           ^^^^^^^^^^^ no external crate `ops`
    |
-   = help: consider importing one of these items instead:
-           core::ops
+   = help: consider importing this module instead:
            std::ops
 
 error: aborting due to 1 previous error
diff --git a/tests/ui/imports/import-alias-issue-121168.edition2018.stderr b/tests/ui/imports/import-alias-issue-121168.edition2018.stderr
@@ -4,7 +4,7 @@ error[E0412]: cannot find type `Foo` in this scope
 LL |     let _: Foo<i32> = todo!();
    |            ^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these structs
    |
 LL + use crate::nice_crate_name::Foo;
    |
diff --git a/tests/ui/imports/import-alias-issue-121168.edition2021.stderr b/tests/ui/imports/import-alias-issue-121168.edition2021.stderr
@@ -4,7 +4,7 @@ error[E0412]: cannot find type `Foo` in this scope
 LL |     let _: Foo<i32> = todo!();
    |            ^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these structs
    |
 LL + use crate::nice_crate_name::Foo;
    |
diff --git a/tests/ui/imports/issue-56125.stderr b/tests/ui/imports/issue-56125.stderr
@@ -4,16 +4,16 @@ error[E0432]: unresolved import `empty::issue_56125`
 LL |     use empty::issue_56125;
    |         ^^^^^^^^^^^^^^^^^^ no `issue_56125` in `m3::empty`
    |
-help: consider importing one of these items instead
+help: consider importing one of these modules instead
    |
+LL |     use crate::m3::last_segment::issue_56125;
+   |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+LL |     use crate::m3::non_last_segment::non_last_segment::issue_56125;
+   |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 LL |     use ::issue_56125::issue_56125;
    |         ~~~~~~~~~~~~~~~~~~~~~~~~~~
 LL |     use ::issue_56125::last_segment::issue_56125;
    |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-LL |     use ::issue_56125::non_last_segment::non_last_segment::issue_56125;
-   |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-LL |     use crate::m3::last_segment::issue_56125;
-   |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      and 1 other candidate
 
 error[E0659]: `issue_56125` is ambiguous
diff --git a/tests/ui/lint/use_suggestion_json.stderr b/tests/ui/lint/use_suggestion_json.stderr
@@ -95,7 +95,7 @@ mod foo {
   ],
   "children": [
     {
-      "message": "consider importing one of these items",
+      "message": "consider importing one of these structs",
       "code": null,
       "level": "help",
       "spans": [
@@ -386,7 +386,7 @@ mod foo {
 \u001b[0m\u001b[1m\u001b[38;5;12mLL\u001b[0m\u001b[0m \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;12m|\u001b[0m\u001b[0m \u001b[0m\u001b[0m    let x: Iter;\u001b[0m
 \u001b[0m   \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;12m|\u001b[0m\u001b[0m            \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;9m^^^^\u001b[0m\u001b[0m \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;9mnot found in this scope\u001b[0m
 \u001b[0m   \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;12m|\u001b[0m
-\u001b[0m\u001b[1m\u001b[38;5;14mhelp\u001b[0m\u001b[0m: consider importing one of these items\u001b[0m
+\u001b[0m\u001b[1m\u001b[38;5;14mhelp\u001b[0m\u001b[0m: consider importing one of these structs\u001b[0m
 \u001b[0m   \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;12m|\u001b[0m
 \u001b[0m\u001b[1m\u001b[38;5;12mLL\u001b[0m\u001b[0m \u001b[0m\u001b[0m\u001b[38;5;10m+ use std::collections::binary_heap::Iter;\u001b[0m
 \u001b[0m   \u001b[0m\u001b[0m\u001b[1m\u001b[38;5;12m|\u001b[0m
diff --git a/tests/ui/namespace/namespace-mix.stderr b/tests/ui/namespace/namespace-mix.stderr
@@ -12,7 +12,7 @@ help: a tuple struct with a similar name exists
    |
 LL |     check(m1::TS);
    |               ~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use m2::S;
    |
@@ -40,7 +40,7 @@ help: a tuple struct with a similar name exists
    |
 LL |     check(xm1::TS);
    |                ~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use m2::S;
    |
@@ -66,7 +66,7 @@ help: a tuple variant with a similar name exists
    |
 LL |     check(m7::TV);
    |               ~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use m8::V;
    |
@@ -94,7 +94,7 @@ help: a tuple variant with a similar name exists
    |
 LL |     check(xm7::TV);
    |                ~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use m8::V;
    |
diff --git a/tests/ui/resolve/issue-16058.stderr b/tests/ui/resolve/issue-16058.stderr
@@ -4,7 +4,7 @@ error[E0574]: expected struct, variant or union type, found enum `Result`
 LL |         Result {
    |         ^^^^^^ not a struct, variant or union type
    |
-help: consider importing one of these items instead
+help: consider importing one of these type aliases instead
    |
 LL + use std::fmt::Result;
    |
diff --git a/tests/ui/resolve/issue-21221-1.stderr b/tests/ui/resolve/issue-21221-1.stderr
@@ -4,29 +4,29 @@ error[E0405]: cannot find trait `Mul` in this scope
 LL | impl Mul for Foo {
    |      ^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these traits
+   |
+LL + use std::ops::Mul;
    |
 LL + use mul1::Mul;
    |
 LL + use mul2::Mul;
    |
-LL + use std::ops::Mul;
-   |
 
 error[E0412]: cannot find type `Mul` in this scope
   --> $DIR/issue-21221-1.rs:58:16
    |
 LL | fn getMul() -> Mul {
    |                ^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these traits
+   |
+LL + use std::ops::Mul;
    |
 LL + use mul1::Mul;
    |
 LL + use mul2::Mul;
    |
-LL + use std::ops::Mul;
-   |
 
 error[E0405]: cannot find trait `ThisTraitReallyDoesntExistInAnyModuleReally` in this scope
   --> $DIR/issue-21221-1.rs:63:6
diff --git a/tests/ui/resolve/issue-21221-2.stderr b/tests/ui/resolve/issue-21221-2.stderr
@@ -4,7 +4,7 @@ error[E0405]: cannot find trait `T` in this scope
 LL | impl T for Foo { }
    |      ^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these traits
    |
 LL + use baz::T;
    |
diff --git a/tests/ui/resolve/issue-50599.stderr b/tests/ui/resolve/issue-50599.stderr
@@ -4,7 +4,7 @@ error[E0425]: cannot find value `LOG10_2` in module `std::f64`
 LL |     const M: usize = (f64::from(N) * std::f64::LOG10_2) as usize;
    |                                                ^^^^^^^ not found in `std::f64`
    |
-help: consider importing one of these items
+help: consider importing one of these constants
    |
 LL + use std::f128::consts::LOG10_2;
    |
diff --git a/tests/ui/resolve/issue-73427.stderr b/tests/ui/resolve/issue-73427.stderr
@@ -105,7 +105,7 @@ help: the following enum variant is available
    |
 LL |     (E::TupleWithFields(/* fields */)).foo();
    |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use std::f128::consts::E;
    |
diff --git a/tests/ui/resolve/privacy-enum-ctor.stderr b/tests/ui/resolve/privacy-enum-ctor.stderr
@@ -82,7 +82,7 @@ help: a function with a similar name exists
    |
 LL |     let _: E = m::f;
    |                   ~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use std::f128::consts::E;
    |
@@ -123,7 +123,7 @@ help: alternatively, the following enum variant is available
    |
 LL |     let _: E = (E::Fn(/* fields */));
    |                ~~~~~~~~~~~~~~~~~~~~~
-help: consider importing one of these items instead
+help: consider importing one of these constants instead
    |
 LL + use std::f128::consts::E;
    |
diff --git a/tests/ui/rfcs/rfc-2126-extern-absolute-paths/not-allowed.stderr b/tests/ui/rfcs/rfc-2126-extern-absolute-paths/not-allowed.stderr
@@ -4,10 +4,8 @@ error[E0432]: unresolved import `alloc`
 LL | use alloc;
    |     ^^^^^ no external crate `alloc`
    |
-help: consider importing one of these items instead
+help: consider importing this module instead
    |
-LL | use core::alloc;
-   |     ~~~~~~~~~~~
 LL | use std::alloc;
    |     ~~~~~~~~~~
 
diff --git a/tests/ui/rust-2018/issue-52202-use-suggestions.stderr b/tests/ui/rust-2018/issue-52202-use-suggestions.stderr
@@ -4,7 +4,7 @@ error[E0422]: cannot find struct, variant or union type `Drain` in this scope
 LL |     let _d = Drain {};
    |              ^^^^^ not found in this scope
    |
-help: consider importing one of these items
+help: consider importing one of these structs
    |
 LL + use crate::plumbing::Drain;
    |
diff --git a/tests/ui/suggestions/core-std-import-order-issue-83564.no_std.fixed b/tests/ui/suggestions/core-std-import-order-issue-83564.no_std.fixed
diff --git a/tests/ui/suggestions/core-std-import-order-issue-83564.no_std.stderr b/tests/ui/suggestions/core-std-import-order-issue-83564.no_std.stderr
diff --git a/tests/ui/suggestions/core-std-import-order-issue-83564.rs b/tests/ui/suggestions/core-std-import-order-issue-83564.rs
diff --git a/tests/ui/suggestions/core-std-import-order-issue-83564.std.fixed b/tests/ui/suggestions/core-std-import-order-issue-83564.std.fixed
diff --git a/tests/ui/suggestions/core-std-import-order-issue-83564.std.stderr b/tests/ui/suggestions/core-std-import-order-issue-83564.std.stderr
diff --git a/tests/ui/suggestions/crate-or-module-typo.stderr b/tests/ui/suggestions/crate-or-module-typo.stderr
diff --git a/tests/ui/suggestions/suggest-tryinto-edition-change.rs b/tests/ui/suggestions/suggest-tryinto-edition-change.rs
diff --git a/tests/ui/suggestions/suggest-tryinto-edition-change.stderr b/tests/ui/suggestions/suggest-tryinto-edition-change.stderr

Original file line number	Diff line number	Diff line change
@@ -4,9 +4,7 @@ error[E0412]: cannot find type `PhantomData` in this scope
`4`	`4`	`LL \| _n: PhantomData,`
`5`	`5`	`\| ^^^^^^^^^^^ not found in this scope`
`6`	`6`	`\|`
`7`		`-help: consider importing one of these items`
`8`		`- \|`
`9`		`-LL + use core::marker::PhantomData;`
	`7`	`+help: consider importing this struct`
`10`	`8`	`\|`
`11`	`9`	`LL + use std::marker::PhantomData;`
`12`	`10`	`\|`
Original file line number	Diff line number	Diff line change
@@ -4,7 +4,7 @@ error[E0433]: failed to resolve: use of undeclared type `IntoIter`
`4`	`4`	`LL \| let mut iter = IntoIter::new(self);`
`5`	`5`	\| ^^^^^^^^ use of undeclared type `IntoIter`
`6`	`6`	`\|`
`7`		`-help: consider importing one of these items`
	`7`	`+help: consider importing one of these structs`
`8`	`8`	`\|`
`9`	`9`	`LL + use std::array::IntoIter;`
`10`	`10`	`\|`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	error[E0433]: failed to resolve: use of undeclared type `AtomicUsize`
`2`		`- --> $DIR/const_refs_to_static-ice-121413.rs:8:24`
	`2`	`+ --> $DIR/const_refs_to_static-ice-121413.rs:9:24`
`3`	`3`	`\|`
`4`	`4`	`LL \| static FOO: Sync = AtomicUsize::new(0);`
`5`	`5`	\| ^^^^^^^^^^^ use of undeclared type `AtomicUsize`
`@@ -10,7 +10,7 @@ LL + use std::sync::atomic::AtomicUsize;`
`10`	`10`	`\|`
`11`	`11`
`12`	`12`	warning: trait objects without an explicit `dyn` are deprecated
`13`		`- --> $DIR/const_refs_to_static-ice-121413.rs:8:17`
	`13`	`+ --> $DIR/const_refs_to_static-ice-121413.rs:9:17`
`14`	`14`	`\|`
`15`	`15`	`LL \| static FOO: Sync = AtomicUsize::new(0);`
`16`	`16`	`\| ^^^^`
`@@ -24,15 +24,15 @@ LL \| static FOO: dyn Sync = AtomicUsize::new(0);`
`24`	`24`	`\| +++`
`25`	`25`
`26`	`26`	error[E0277]: the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
`27`		`- --> $DIR/const_refs_to_static-ice-121413.rs:8:17`
	`27`	`+ --> $DIR/const_refs_to_static-ice-121413.rs:9:17`
`28`	`28`	`\|`
`29`	`29`	`LL \| static FOO: Sync = AtomicUsize::new(0);`
`30`	`30`	`\| ^^^^ doesn't have a size known at compile-time`
`31`	`31`	`\|`
`32`	`32`	= help: the trait `Sized` is not implemented for `(dyn Sync + 'static)`
`33`	`33`
`34`	`34`	error[E0277]: the size for values of type `(dyn Sync + 'static)` cannot be known at compilation time
`35`		`- --> $DIR/const_refs_to_static-ice-121413.rs:8:24`
	`35`	`+ --> $DIR/const_refs_to_static-ice-121413.rs:9:24`
`36`	`36`	`\|`
`37`	`37`	`LL \| static FOO: Sync = AtomicUsize::new(0);`
`38`	`38`	`\| ^^^^^^^^^^^^^^^^^^^ doesn't have a size known at compile-time`
Original file line number	Diff line number	Diff line change
@@ -22,7 +22,7 @@ error[E0425]: cannot find value `Set` in this scope
`22`	`22`	`LL \| fn setup() -> Set { Set }`
`23`	`23`	`\| ^^^ not found in this scope`
`24`	`24`	`\|`
`25`		`-help: consider importing one of these items`
	`25`	`+help: consider importing one of these unit variants`
`26`	`26`	`\|`
`27`	`27`	`LL + use AffixHeart::Set;`
`28`	`28`	`\|`
Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,7 @@ LL \| fn f() { my_core::mem::drop(0); }`
`24`	`24`	`LL \| a!();`
`25`	`25`	`\| ---- in this macro invocation`
`26`	`26`	`\|`
`27`		`- = help: consider importing one of these items:`
`28`		`- core::mem`
	`27`	`+ = help: consider importing this module:`
`29`	`28`	`std::mem`
`30`	`29`	= note: this error originates in the macro `a` (in Nightly builds, run with -Z macro-backtrace for more info)
`31`	`30`
@@ -35,9 +34,7 @@ error[E0433]: failed to resolve: use of undeclared crate or module `my_core`
`35`	`34`	`LL \| fn f() { my_core::mem::drop(0); }`
`36`	`35`	\| ^^^^^^^ use of undeclared crate or module `my_core`
`37`	`36`	`\|`
`38`		`-help: consider importing one of these items`
`39`		`- \|`
`40`		`-LL + use core::mem;`
	`37`	`+help: consider importing this module`
`41`	`38`	`\|`
`42`	`39`	`LL + use std::mem;`
`43`	`40`	`\|`
Original file line number	Diff line number	Diff line change
@@ -4,8 +4,7 @@ error[E0432]: unresolved import `ops`
`4`	`4`	`LL \| use ops::{self as std};`
`5`	`5`	\| ^^^^^^^^^^^ no external crate `ops`
`6`	`6`	`\|`
`7`		`- = help: consider importing one of these items instead:`
`8`		`- core::ops`
	`7`	`+ = help: consider importing this module instead:`
`9`	`8`	`std::ops`
`10`	`9`
`11`	`10`	`error: aborting due to 1 previous error`
Original file line number	Diff line number	Diff line change
@@ -4,7 +4,7 @@ error[E0412]: cannot find type `Foo` in this scope
`4`	`4`	`LL \| let _: Foo<i32> = todo!();`
`5`	`5`	`\| ^^^ not found in this scope`
`6`	`6`	`\|`
`7`		`-help: consider importing one of these items`
	`7`	`+help: consider importing one of these structs`
`8`	`8`	`\|`
`9`	`9`	`LL + use crate::nice_crate_name::Foo;`
`10`	`10`	`\|`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ help: a tuple struct with a similar name exists`
`12`	`12`	`\|`
`13`	`13`	`LL \| check(m1::TS);`
`14`	`14`	`\| ~~`
`15`		`-help: consider importing one of these items instead`
	`15`	`+help: consider importing one of these constants instead`
`16`	`16`	`\|`
`17`	`17`	`LL + use m2::S;`
`18`	`18`	`\|`
`@@ -40,7 +40,7 @@ help: a tuple struct with a similar name exists`
`40`	`40`	`\|`
`41`	`41`	`LL \| check(xm1::TS);`
`42`	`42`	`\| ~~`
`43`		`-help: consider importing one of these items instead`
	`43`	`+help: consider importing one of these constants instead`
`44`	`44`	`\|`
`45`	`45`	`LL + use m2::S;`
`46`	`46`	`\|`
`@@ -66,7 +66,7 @@ help: a tuple variant with a similar name exists`
`66`	`66`	`\|`
`67`	`67`	`LL \| check(m7::TV);`
`68`	`68`	`\| ~~`
`69`		`-help: consider importing one of these items instead`
	`69`	`+help: consider importing one of these constants instead`
`70`	`70`	`\|`
`71`	`71`	`LL + use m8::V;`
`72`	`72`	`\|`
`@@ -94,7 +94,7 @@ help: a tuple variant with a similar name exists`
`94`	`94`	`\|`
`95`	`95`	`LL \| check(xm7::TV);`
`96`	`96`	`\| ~~`
`97`		`-help: consider importing one of these items instead`
	`97`	`+help: consider importing one of these constants instead`
`98`	`98`	`\|`
`99`	`99`	`LL + use m8::V;`
`100`	`100`	`\|`
Original file line number	Diff line number	Diff line change
@@ -4,7 +4,7 @@ error[E0574]: expected struct, variant or union type, found enum `Result`
`4`	`4`	`LL \| Result {`
`5`	`5`	`\| ^^^^^^ not a struct, variant or union type`
`6`	`6`	`\|`
`7`		`-help: consider importing one of these items instead`
	`7`	`+help: consider importing one of these type aliases instead`
`8`	`8`	`\|`
`9`	`9`	`LL + use std::fmt::Result;`
`10`	`10`	`\|`
Original file line number	Diff line number	Diff line change
@@ -4,29 +4,29 @@ error[E0405]: cannot find trait `Mul` in this scope
`4`	`4`	`LL \| impl Mul for Foo {`
`5`	`5`	`\| ^^^ not found in this scope`
`6`	`6`	`\|`
`7`		`-help: consider importing one of these items`
	`7`	`+help: consider importing one of these traits`
	`8`	`+ \|`
	`9`	`+LL + use std::ops::Mul;`
`8`	`10`	`\|`
`9`	`11`	`LL + use mul1::Mul;`
`10`	`12`	`\|`
`11`	`13`	`LL + use mul2::Mul;`
`12`	`14`	`\|`
`13`		`-LL + use std::ops::Mul;`
`14`		`- \|`
`15`	`15`
`16`	`16`	error[E0412]: cannot find type `Mul` in this scope
`17`	`17`	`--> $DIR/issue-21221-1.rs:58:16`
`18`	`18`	`\|`
`19`	`19`	`LL \| fn getMul() -> Mul {`
`20`	`20`	`\| ^^^ not found in this scope`
`21`	`21`	`\|`
`22`		`-help: consider importing one of these items`
	`22`	`+help: consider importing one of these traits`
	`23`	`+ \|`
	`24`	`+LL + use std::ops::Mul;`
`23`	`25`	`\|`
`24`	`26`	`LL + use mul1::Mul;`
`25`	`27`	`\|`
`26`	`28`	`LL + use mul2::Mul;`
`27`	`29`	`\|`
`28`		`-LL + use std::ops::Mul;`
`29`		`- \|`
`30`	`30`
`31`	`31`	error[E0405]: cannot find trait `ThisTraitReallyDoesntExistInAnyModuleReally` in this scope
`32`	`32`	`--> $DIR/issue-21221-1.rs:63:6`