different paths to the same struct

Author: ada4a

tests/ui/f1.fixed

@@ -1,11 +1,14 @@
 #![allow(clippy::disallowed_names)]
 #![warn(clippy::map_identity)]
 
-#[derive(Clone, Copy)]
-struct Foo {
-    foo: u8,
-    bar: u8,
+mod foo {
+    #[derive(Clone, Copy)]
+    pub struct Foo {
+        pub foo: u8,
+        pub bar: u8,
+    }
 }
+use foo::Foo;
 
 struct Bar {
     foo: u8,
@@ -18,6 +21,10 @@ fn main() {
     let _ = x.into_iter();
     //~^ map_identity
 
+    // still lint when different paths are used for the same struct
+    let _ = x.into_iter();
+    //~^ map_identity
+
     // don't lint: same fields but different structs
     let _ = x.into_iter().map(|Foo { foo, bar }| Bar { foo, bar });
 

tests/ui/f1.rs

@@ -1,11 +1,14 @@
 #![allow(clippy::disallowed_names)]
 #![warn(clippy::map_identity)]
 
-#[derive(Clone, Copy)]
-struct Foo {
-    foo: u8,
-    bar: u8,
+mod foo {
+    #[derive(Clone, Copy)]
+    pub struct Foo {
+        pub foo: u8,
+        pub bar: u8,
+    }
 }
+use foo::Foo;
 
 struct Bar {
     foo: u8,
@@ -18,6 +21,10 @@ fn main() {
     let _ = x.into_iter().map(|Foo { foo, bar }| Foo { foo, bar });
     //~^ map_identity
 
+    // still lint when different paths are used for the same struct
+    let _ = x.into_iter().map(|Foo { foo, bar }| foo::Foo { foo, bar });
+    //~^ map_identity
+
     // don't lint: same fields but different structs
     let _ = x.into_iter().map(|Foo { foo, bar }| Bar { foo, bar });
 

tests/ui/f1.stderr

@@ -1,5 +1,5 @@
 error: unnecessary map of the identity function
-  --> tests/ui/f1.rs:18:26
+  --> tests/ui/f1.rs:21:26
    |
 LL |     let _ = x.into_iter().map(|Foo { foo, bar }| Foo { foo, bar });
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the call to `map`
@@ -8,16 +8,22 @@ LL |     let _ = x.into_iter().map(|Foo { foo, bar }| Foo { foo, bar });
    = help: to override `-D warnings` add `#[allow(clippy::map_identity)]`
 
 error: unnecessary map of the identity function
-  --> tests/ui/f1.rs:26:26
+  --> tests/ui/f1.rs:25:26
+   |
+LL |     let _ = x.into_iter().map(|Foo { foo, bar }| foo::Foo { foo, bar });
+   |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the call to `map`
+
+error: unnecessary map of the identity function
+  --> tests/ui/f1.rs:33:26
    |
 LL |     let _ = x.into_iter().map(|Foo { foo, bar }| Foo { foo: foo, bar: bar });
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the call to `map`
 
 error: unnecessary map of the identity function
-  --> tests/ui/f1.rs:30:26
+  --> tests/ui/f1.rs:37:26
    |
 LL |     let _ = x.into_iter().map(|Foo { foo, bar }| Foo { bar, foo });
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the call to `map`
 
-error: aborting due to 3 previous errors
+error: aborting due to 4 previous errors