diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-12-21 16:57:49 +0000 |
---|---|---|
committer | GitHub <[email protected]> | 2020-12-21 16:57:49 +0000 |
commit | c8a73fe655f7bf9778deeec4c8b4b541e0af398b (patch) | |
tree | f6f9a11b950e001505a3c711f59d34620a037684 /crates/assists/src/handlers | |
parent | 71c8073aa1e2cd3101a031a4f10e460013d4df74 (diff) | |
parent | 2e7abf83844ef0c7d807262a682941f2aef9d3d9 (diff) |
Merge #6982
6982: Remove parentheses when inverting `!(cond)` r=matklad a=Jesse-Bakker
Followup to #6894
When inverting a composite condition twice, the parentheses were left. This also removes those unnecessary parentheses when applying the invert-if assist.
Co-authored-by: Jesse Bakker <[email protected]>
Diffstat (limited to 'crates/assists/src/handlers')
-rw-r--r-- | crates/assists/src/handlers/invert_if.rs | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/crates/assists/src/handlers/invert_if.rs b/crates/assists/src/handlers/invert_if.rs index 91e2f5c8c..f9c33b3f7 100644 --- a/crates/assists/src/handlers/invert_if.rs +++ b/crates/assists/src/handlers/invert_if.rs | |||
@@ -78,6 +78,15 @@ mod tests { | |||
78 | } | 78 | } |
79 | 79 | ||
80 | #[test] | 80 | #[test] |
81 | fn invert_if_remove_not_parentheses() { | ||
82 | check_assist( | ||
83 | invert_if, | ||
84 | "fn f() { i<|>f !(x == 3 || x == 4 || x == 5) { 3 * 2 } else { 1 } }", | ||
85 | "fn f() { if x == 3 || x == 4 || x == 5 { 1 } else { 3 * 2 } }", | ||
86 | ) | ||
87 | } | ||
88 | |||
89 | #[test] | ||
81 | fn invert_if_remove_inequality() { | 90 | fn invert_if_remove_inequality() { |
82 | check_assist( | 91 | check_assist( |
83 | invert_if, | 92 | invert_if, |