[PATCH] D54349: [clang-tidy] new check 'readability-redundant-pp'

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 9 14:02:41 PST 2018


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:73
+
+  Finds potentially redundant preprocessor usage.
+
----------------
preprocessor directives? Same in documentation.


================
Comment at: docs/clang-tidy/checks/readability-redundant-pp.rst:8
+
+* `#ifdef` .. `#endif` pairs which are nested inside an outer pair with the
+  same condition.
----------------
JonasToth wrote:
> This needs more explanation, please add `.. code-block:: c++` sections for the cases that demonstrate the issue.
Please use ``. Same below.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54349





More information about the cfe-commits mailing list