[PATCH] D37014: [clang-tidy] Add a checker to remove useless intermediate variables before return statements with comparisons

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 22 13:20:05 PDT 2017


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:60
 
+- New `readability-useless-intermediate-var
+  <http://clang.llvm.org/extra/clang-tidy/checks/readability-useless-intermediate-var.html>`_ check
----------------
Please place new checks in alphabetical order.


================
Comment at: docs/ReleaseNotes.rst:63
+
+  This new checker detects useless intermediate variables before return
+  statements that return the result of a simple comparison. This checker also
----------------
JonasToth wrote:
> maybe "useless" should be replaced with "unnecessary". but thats only my opinion.
Checker -> check.
Please enclose return (in statement context) in ``.
Same for other places.


Repository:
  rL LLVM

https://reviews.llvm.org/D37014





More information about the cfe-commits mailing list