Skip to content

Commit 404e262

Browse files
committed
single_match: Clarify the don't lint test case
1 parent a5a07e5 commit 404e262

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

tests/ui/single_match.rs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,9 @@ fn ranges() {
152152
}
153153
let x = (Some(E::V), Some(42));
154154

155-
// don't lint
155+
// Don't lint, because the `E` enum can be extended with additional fields later. Thus, the
156+
// proposed replacement to `if let Some(E::V)` may hide non-exhaustive warnings that appeared
157+
// because of `match` construction.
156158
match x {
157159
(Some(E::V), _) => {},
158160
(None, _) => {},
@@ -176,19 +178,19 @@ fn ranges() {
176178
(..) => {},
177179
}
178180

179-
// don't lint
181+
// Don't lint, see above.
180182
match (Some(E::V), Some(E::V), Some(E::V)) {
181183
(.., Some(E::V), _) => {},
182184
(.., None, _) => {},
183185
}
184186

185-
// don't lint
187+
// Don't lint, see above.
186188
match (Some(E::V), Some(E::V), Some(E::V)) {
187189
(Some(E::V), ..) => {},
188190
(None, ..) => {},
189191
}
190192

191-
// don't lint
193+
// Don't lint, see above.
192194
match (Some(E::V), Some(E::V), Some(E::V)) {
193195
(_, Some(E::V), ..) => {},
194196
(_, None, ..) => {},

0 commit comments

Comments
 (0)