about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorVadim Petrochenkov <vadim.petrochenkov@gmail.com>2020-02-15 12:38:34 +0300
committerVadim Petrochenkov <vadim.petrochenkov@gmail.com>2020-02-15 12:38:34 +0300
commitbcd7e2b38b3eb26004bf94a63814341329233491 (patch)
tree4cc18f670798dc2654e7e85eed7fb599311c810c /src
parent19288ddfd6b3448c2c221d75610bff722a6582e8 (diff)
downloadrust-bcd7e2b38b3eb26004bf94a63814341329233491.tar.gz
rust-bcd7e2b38b3eb26004bf94a63814341329233491.zip
rustc_lint: Move `unused_doc_comments` from pre-expansion to early lints
Diffstat (limited to 'src')
-rw-r--r--src/librustc_lint/lib.rs3
-rw-r--r--src/test/ui/useless-comment.rs4
-rw-r--r--src/test/ui/useless-comment.stderr31
3 files changed, 10 insertions, 28 deletions
diff --git a/src/librustc_lint/lib.rs b/src/librustc_lint/lib.rs
index 2204e104803..d3670056c1a 100644
--- a/src/librustc_lint/lib.rs
+++ b/src/librustc_lint/lib.rs
@@ -94,7 +94,7 @@ fn lint_mod(tcx: TyCtxt<'_>, module_def_id: DefId) {
 
 macro_rules! pre_expansion_lint_passes {
     ($macro:path, $args:tt) => {
-        $macro!($args, [KeywordIdents: KeywordIdents, UnusedDocComment: UnusedDocComment,]);
+        $macro!($args, [KeywordIdents: KeywordIdents,]);
     };
 }
 
@@ -114,6 +114,7 @@ macro_rules! early_lint_passes {
                 NonAsciiIdents: NonAsciiIdents,
                 IncompleteFeatures: IncompleteFeatures,
                 RedundantSemicolon: RedundantSemicolon,
+                UnusedDocComment: UnusedDocComment,
             ]
         );
     };
diff --git a/src/test/ui/useless-comment.rs b/src/test/ui/useless-comment.rs
index 7d2e5ab6f2b..f19c97e0102 100644
--- a/src/test/ui/useless-comment.rs
+++ b/src/test/ui/useless-comment.rs
@@ -6,7 +6,7 @@ macro_rules! mac {
     () => {}
 }
 
-/// foo //~ ERROR unused doc comment
+/// foo //FIXME ERROR unused doc comment
 mac!();
 
 fn foo() {
@@ -29,7 +29,7 @@ fn foo() {
     #[doc = "bar"] //~ ERROR unused doc comment
     3;
 
-    /// bar //~ ERROR unused doc comment
+    /// bar //FIXME ERROR unused doc comment
     mac!();
 
     let x = /** comment */ 47; //~ ERROR unused doc comment
diff --git a/src/test/ui/useless-comment.stderr b/src/test/ui/useless-comment.stderr
index e5e4290d0e1..beb2a09f10e 100644
--- a/src/test/ui/useless-comment.stderr
+++ b/src/test/ui/useless-comment.stderr
@@ -1,25 +1,16 @@
 error: unused doc comment
-  --> $DIR/useless-comment.rs:9:1
+  --> $DIR/useless-comment.rs:13:5
    |
-LL | /// foo
-   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-LL | mac!();
-   | ------- rustdoc does not generate documentation for macro expansions
+LL |     /// a
+   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+LL |     let x = 12;
+   |     ----------- rustdoc does not generate documentation for statements
    |
 note: the lint level is defined here
   --> $DIR/useless-comment.rs:3:9
    |
 LL | #![deny(unused_doc_comments)]
    |         ^^^^^^^^^^^^^^^^^^^
-   = help: to document an item produced by a macro, the macro must produce the documentation as part of its expansion
-
-error: unused doc comment
-  --> $DIR/useless-comment.rs:13:5
-   |
-LL |     /// a
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-LL |     let x = 12;
-   |     ----------- rustdoc does not generate documentation for statements
 
 error: unused doc comment
   --> $DIR/useless-comment.rs:16:5
@@ -69,16 +60,6 @@ LL |     3;
    |     - rustdoc does not generate documentation for expressions
 
 error: unused doc comment
-  --> $DIR/useless-comment.rs:32:5
-   |
-LL |     /// bar
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-LL |     mac!();
-   |     ------- rustdoc does not generate documentation for macro expansions
-   |
-   = help: to document an item produced by a macro, the macro must produce the documentation as part of its expansion
-
-error: unused doc comment
   --> $DIR/useless-comment.rs:35:13
    |
 LL |     let x = /** comment */ 47;
@@ -94,5 +75,5 @@ LL | |
 LL | |     }
    | |_____- rustdoc does not generate documentation for expressions
 
-error: aborting due to 10 previous errors
+error: aborting due to 8 previous errors