diff options
| author | Guillaume Gomez <guillaume1.gomez@gmail.com> | 2017-09-24 14:01:49 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-09-24 14:01:49 +0200 |
| commit | 8a25ec019ed033436cfd2835b19b80914297936f (patch) | |
| tree | a31529e2e511a7d582ed098fd85cae47c6602ddf /src/libsyntax | |
| parent | 6f9078745e882b3cdedf058479d9d6d936c6750c (diff) | |
| parent | 6734d39b49dd00fe5ba1ea875a3e03f9eca23ac0 (diff) | |
| download | rust-8a25ec019ed033436cfd2835b19b80914297936f.tar.gz rust-8a25ec019ed033436cfd2835b19b80914297936f.zip | |
Rollup merge of #44103 - zackmdavis:cmp_op_must_use, r=arielb1
add comparison operators to must-use lint (under `fn_must_use` feature) Although RFC 1940 is about annotating functions with `#[must_use]`, a key part of the motivation was linting unused equality operators. (See https://github.com/rust-lang/rfcs/pull/1812#issuecomment-265695898—it seems to have not been clear to discussants at the time that marking the comparison methods as `must_use` would not give us the lints on comparison operators, at least in (what the present author understood as) the most straightforward implementation, as landed in #43728 (3645b062).) To rectify the situation, we here lint unused comparison operators as part of the unused-must-use lint (feature gated by the `fn_must_use` feature flag, which now arguably becomes a slight (tolerable in the opinion of the present author) misnomer). This is in the matter of #43302. cc @crumblingstatue
Diffstat (limited to 'src/libsyntax')
| -rw-r--r-- | src/libsyntax/feature_gate.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libsyntax/feature_gate.rs b/src/libsyntax/feature_gate.rs index 1fef382c83a..5c730aaa8d0 100644 --- a/src/libsyntax/feature_gate.rs +++ b/src/libsyntax/feature_gate.rs @@ -380,7 +380,7 @@ declare_features! ( // #[doc(masked)] (active, doc_masked, "1.21.0", None), - // allow `#[must_use]` on functions (RFC 1940) + // allow `#[must_use]` on functions and comparison operators (RFC 1940) (active, fn_must_use, "1.21.0", Some(43302)), // allow '|' at beginning of match arms (RFC 1925) |