Implement use_self for tuple structs

Author: detrumi

clippy_lints/src/use_self.rs

@@ -7,6 +7,7 @@
 // option. This file may not be copied, modified, or distributed
 // except according to those terms.
 
+use crate::rustc::hir::def::{CtorKind, Def};
 use crate::rustc::hir::intravisit::{walk_path, walk_ty, NestedVisitorMap, Visitor};
 use crate::rustc::hir::*;
 use crate::rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
@@ -30,8 +31,7 @@ use if_chain::if_chain;
 /// - False positive when using associated types (#2843)
 /// - False positives in some situations when using generics (#3410)
 /// - False positive when type from outer function can't be used (#3463)
-/// - Does not diagnose tuple structs (#3498)
-/// - Does not trigger in lifetimed struct
+/// - Does not trigger in lifetimed structs
 ///
 /// **Example:**
 /// ```rust
@@ -232,10 +232,15 @@ struct UseSelfVisitor<'a, 'tcx: 'a> {
 
 impl<'a, 'tcx> Visitor<'tcx> for UseSelfVisitor<'a, 'tcx> {
     fn visit_path(&mut self, path: &'tcx Path, _id: HirId) {
-        if self.item_path.def == path.def && path.segments.last().expect(SEGMENTS_MSG).ident.name != SelfUpper.name() {
-            span_use_self_lint(self.cx, path);
+        if path.segments.last().expect(SEGMENTS_MSG).ident.name != SelfUpper.name() {
+            if self.item_path.def == path.def {
+                span_use_self_lint(self.cx, path);
+            } else if let Def::StructCtor(ctor_did, CtorKind::Fn) = path.def {
+                if self.item_path.def.opt_def_id() == self.cx.tcx.parent_def_id(ctor_did) {
+                    span_use_self_lint(self.cx, path);
+                }
+            }
         }
-
         walk_path(self, path);
     }
 

tests/ui/use_self.rs

@@ -216,6 +216,16 @@ mod existential {
     }
 }
 
+mod tuple_structs {
+    pub struct TS(i32);
+
+    impl TS {
+        pub fn ts() -> Self {
+            TS(0)
+        }
+    }
+}
+
 mod issue3410 {
 
     struct A;

tests/ui/use_self.stderr

@@ -126,5 +126,11 @@ error: unnecessary structure name repetition
 LL |         fn bad(foos: &[Self]) -> impl Iterator<Item = &Foo> {
    |                                                        ^^^ help: use the applicable keyword: `Self`
 
-error: aborting due to 21 previous errors
+error: unnecessary structure name repetition
+   --> $DIR/use_self.rs:224:13
+    |
+224 |             TS(0)
+    |             ^^ help: use the applicable keyword: `Self`
+
+error: aborting due to 22 previous errors