diff options
| author | bors <bors@rust-lang.org> | 2020-11-28 21:37:05 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2020-11-28 21:37:05 +0000 |
| commit | e42ba4829c02e8308ae142b5a2fd5efb6ccf0a7b (patch) | |
| tree | 5d4de448c320fa1fd3dd4ab17abf99eafc610974 | |
| parent | d75bc868caee77f1b29763f8ba5a00494ed52f6b (diff) | |
| parent | cb6a654b75ca095aee9ebbe5cf05fc48df5b9b50 (diff) | |
| download | rust-e42ba4829c02e8308ae142b5a2fd5efb6ccf0a7b.tar.gz rust-e42ba4829c02e8308ae142b5a2fd5efb6ccf0a7b.zip | |
Auto merge of #6390 - pro-grammer1:master, r=ebroto
Added known problem to comparison_chain docs changelog: Added documentation to comparison_chain that explains a possible performance penalty, according to issue #5354 This is my first PR, I hope everything has been done correctly. Fixes #5354
| -rw-r--r-- | clippy_lints/src/comparison_chain.rs | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clippy_lints/src/comparison_chain.rs b/clippy_lints/src/comparison_chain.rs index 99f161a0510..ae1143b2c50 100644 --- a/clippy_lints/src/comparison_chain.rs +++ b/clippy_lints/src/comparison_chain.rs @@ -12,7 +12,8 @@ declare_clippy_lint! { /// **Why is this bad?** `if` is not guaranteed to be exhaustive and conditionals can get /// repetitive /// - /// **Known problems:** None. + /// **Known problems:** The match statement may be slower due to the compiler + /// not inlining the call to cmp. See issue #5354 /// /// **Example:** /// ```rust,ignore |