Skip to content

Commit

Permalink
allow let_unit_value
Browse files Browse the repository at this point in the history
  • Loading branch information
FoseFx committed Apr 19, 2022
1 parent 0d692c2 commit 98d8b5e
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 8 deletions.
1 change: 1 addition & 0 deletions tests/ui/trim_split_whitespace.fixed
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
// run-rustfix
#![warn(clippy::trim_split_whitespace)]
#![allow(clippy::let_unit_value)]

struct Custom();
impl Custom {
Expand Down
1 change: 1 addition & 0 deletions tests/ui/trim_split_whitespace.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
// run-rustfix
#![warn(clippy::trim_split_whitespace)]
#![allow(clippy::let_unit_value)]

struct Custom();
impl Custom {
Expand Down
16 changes: 8 additions & 8 deletions tests/ui/trim_split_whitespace.stderr
Original file line number Diff line number Diff line change
@@ -1,49 +1,49 @@
error: found call to `str::trim` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:61:23
--> $DIR/trim_split_whitespace.rs:62:23
|
LL | let _ = " A B C ".trim().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim()`: `split_whitespace()`
|
= note: `-D clippy::trim-split-whitespace` implied by `-D warnings`

error: found call to `str::trim_start` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:62:23
--> $DIR/trim_split_whitespace.rs:63:23
|
LL | let _ = " A B C ".trim_start().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim_start()`: `split_whitespace()`

error: found call to `str::trim_end` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:63:23
--> $DIR/trim_split_whitespace.rs:64:23
|
LL | let _ = " A B C ".trim_end().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim_end()`: `split_whitespace()`

error: found call to `str::trim` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:66:37
--> $DIR/trim_split_whitespace.rs:67:37
|
LL | let _ = (" A B C ").to_string().trim().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim()`: `split_whitespace()`

error: found call to `str::trim_start` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:67:37
--> $DIR/trim_split_whitespace.rs:68:37
|
LL | let _ = (" A B C ").to_string().trim_start().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim_start()`: `split_whitespace()`

error: found call to `str::trim_end` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:68:37
--> $DIR/trim_split_whitespace.rs:69:37
|
LL | let _ = (" A B C ").to_string().trim_end().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim_end()`: `split_whitespace()`

error: found call to `str::trim` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:75:15
--> $DIR/trim_split_whitespace.rs:76:15
|
LL | let _ = s.trim().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim()`: `split_whitespace()`

error: found call to `str::trim` before `str::split_whitespace`
--> $DIR/trim_split_whitespace.rs:83:15
--> $DIR/trim_split_whitespace.rs:84:15
|
LL | let _ = s.trim().split_whitespace(); // should trigger lint
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `trim()`: `split_whitespace()`
Expand Down

0 comments on commit 98d8b5e

Please sign in to comment.