]> git.lizzy.rs Git - rust.git/commitdiff
Fully integrate token collection for additional AST structs
authorAaron Hill <aa1ronham@gmail.com>
Sat, 22 Aug 2020 02:11:41 +0000 (22:11 -0400)
committerAaron Hill <aa1ronham@gmail.com>
Thu, 10 Sep 2020 21:58:14 +0000 (17:58 -0400)
This commit contains miscellaneous changes that don't fit into any of
the other commits in this PR

clippy_lints/src/enum_variants.rs
clippy_lints/src/manual_non_exhaustive.rs
clippy_lints/src/single_component_path_imports.rs
clippy_lints/src/utils/ast_utils.rs

index a9294a87f15d060d23878f0ff1a15705713d8395..67a463538568e9adf619ce3add880796f21c81c4 100644 (file)
@@ -285,7 +285,7 @@ fn check_item(&mut self, cx: &EarlyContext<'_>, item: &Item) {
                             );
                         }
                     }
-                    if item.vis.node.is_pub() {
+                    if item.vis.kind.is_pub() {
                         let matching = partial_match(mod_camel, &item_camel);
                         let rmatching = partial_rmatch(mod_camel, &item_camel);
                         let nchars = mod_camel.chars().count();
@@ -316,7 +316,7 @@ fn check_item(&mut self, cx: &EarlyContext<'_>, item: &Item) {
             }
         }
         if let ItemKind::Enum(ref def, _) = item.kind {
-            let lint = match item.vis.node {
+            let lint = match item.vis.kind {
                 VisibilityKind::Public => PUB_ENUM_VARIANT_NAMES,
                 _ => ENUM_VARIANT_NAMES,
             };
index 4e49bdbdd21bd02a8b04c50f3bfd1ed991eced46..9c623821fdddc3cee97deec82fc03398d43d299e 100644 (file)
@@ -122,7 +122,7 @@ fn is_doc_hidden(attr: &Attribute) -> bool {
 
 fn check_manual_non_exhaustive_struct(cx: &EarlyContext<'_>, item: &Item, data: &VariantData) {
     fn is_private(field: &StructField) -> bool {
-        matches!(field.vis.node, VisibilityKind::Inherited)
+        matches!(field.vis.kind, VisibilityKind::Inherited)
     }
 
     fn is_non_exhaustive_marker(field: &StructField) -> bool {
@@ -141,7 +141,7 @@ fn find_header_span(cx: &EarlyContext<'_>, item: &Item, data: &VariantData) -> S
 
     let fields = data.fields();
     let private_fields = fields.iter().filter(|f| is_private(f)).count();
-    let public_fields = fields.iter().filter(|f| f.vis.node.is_pub()).count();
+    let public_fields = fields.iter().filter(|f| f.vis.kind.is_pub()).count();
 
     if_chain! {
         if private_fields == 1 && public_fields >= 1 && public_fields == fields.len() - 1;
index 58bfd0bc553f425161c172dd35bd0d309fd12a1f..35b38eca14d1be6293e729d4ce3c370d0fef0a8a 100644 (file)
@@ -41,7 +41,7 @@ fn check_item(&mut self, cx: &EarlyContext<'_>, item: &Item) {
         if_chain! {
             if !in_macro(item.span);
             if cx.sess.opts.edition == Edition::Edition2018;
-            if !item.vis.node.is_pub();
+            if !item.vis.kind.is_pub();
             if let ItemKind::Use(use_tree) = &item.kind;
             if let segments = &use_tree.prefix.segments;
             if segments.len() == 1;
index 3c3f8b26e3ac1a4a4b83974e7d9b7d20fabce2c9..4f0474c62cb6f23244b4e53dfaeefa24c314acdf 100644 (file)
@@ -392,7 +392,7 @@ pub fn eq_defaultness(l: Defaultness, r: Defaultness) -> bool {
 
 pub fn eq_vis(l: &Visibility, r: &Visibility) -> bool {
     use VisibilityKind::*;
-    match (&l.node, &r.node) {
+    match (&l.kind, &r.kind) {
         (Public, Public) | (Inherited, Inherited) | (Crate(_), Crate(_)) => true,
         (Restricted { path: l, .. }, Restricted { path: r, .. }) => eq_path(l, r),
         _ => false,