Skip to content

Commit ec77e33

Browse files
committed
Make lint work on all members of ast::Item_
1 parent 2530516 commit ec77e33

File tree

5 files changed

+50
-5
lines changed

5 files changed

+50
-5
lines changed

clippy_lints/src/attrs.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ fn is_relevant_item(tcx: TyCtxt, item: &Item) -> bool {
203203
if let ItemFn(_, _, _, _, _, eid) = item.node {
204204
is_relevant_expr(tcx, tcx.body_tables(eid), &tcx.hir.body(eid).value)
205205
} else {
206-
false
206+
true
207207
}
208208
}
209209

tests/ui/empty_line_after_outer_attribute.rs

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,29 @@ fn with_one_newline() { assert!(true) }
2424

2525
fn with_two_newlines() { assert!(true) }
2626

27+
28+
// This should produce a warning
29+
#[crate_type = "lib"]
30+
31+
enum Baz {
32+
One,
33+
Two
34+
}
35+
36+
// This should produce a warning
37+
#[crate_type = "lib"]
38+
39+
struct Foo {
40+
one: isize,
41+
two: isize
42+
}
43+
44+
// This should produce a warning
45+
#[crate_type = "lib"]
46+
47+
mod foo {
48+
}
49+
2750
// This should not produce a warning
2851
#[allow(non_camel_case_types)]
2952
#[allow(missing_docs)]

tests/ui/empty_line_after_outer_attribute.stderr

Lines changed: 25 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,29 @@ error: Found an empty line after an outer attribute. Perhaps you forgot to add a
2626
25 | | fn with_two_newlines() { assert!(true) }
2727
| |_
2828

29-
error: aborting due to 3 previous errors
29+
error: Found an empty line after an outer attribute. Perhaps you forgot to add a '!' to make it an inner attribute?
30+
--> $DIR/empty_line_after_outer_attribute.rs:29:1
31+
|
32+
29 | / #[crate_type = "lib"]
33+
30 | |
34+
31 | | enum Baz {
35+
| |_
36+
37+
error: Found an empty line after an outer attribute. Perhaps you forgot to add a '!' to make it an inner attribute?
38+
--> $DIR/empty_line_after_outer_attribute.rs:37:1
39+
|
40+
37 | / #[crate_type = "lib"]
41+
38 | |
42+
39 | | struct Foo {
43+
| |_
44+
45+
error: Found an empty line after an outer attribute. Perhaps you forgot to add a '!' to make it an inner attribute?
46+
--> $DIR/empty_line_after_outer_attribute.rs:45:1
47+
|
48+
45 | / #[crate_type = "lib"]
49+
46 | |
50+
47 | | mod foo {
51+
| |_
52+
53+
error: aborting due to 6 previous errors
3054

tests/ui/inline_fn_without_body.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ trait Foo {
1111
#[inline(always)]fn always_inline();
1212

1313
#[inline(never)]
14-
1514
fn never_inline();
1615

1716
#[inline]

tests/ui/inline_fn_without_body.stderr

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ error: use of `#[inline]` on trait method `never_inline` which has no body
1919
|
2020
13 | #[inline(never)]
2121
| _____-^^^^^^^^^^^^^^^
22-
14 | |
23-
15 | | fn never_inline();
22+
14 | | fn never_inline();
2423
| |____- help: remove
2524

2625
error: aborting due to 3 previous errors

0 commit comments

Comments
 (0)