[PATCH] D38688: [clang-tidy] Misc redundant expressions checker updated for macros

Barancsuk Lilla via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 2 08:48:53 PDT 2017


barancsuk added inline comments.


================
Comment at: clang-tidy/misc/RedundantExpressionCheck.cpp:41
+static const llvm::StringSet<> KnownBannedMacroNames = {"EAGAIN", "EWOULDBLOCK",
+                                                      "SIGCLD", "SIGCHLD"};
 
----------------
xazax.hun wrote:
> Is this block clang formatted? The indentation of the second line looks strange. 
It was, but the second line shifted somehow. I fixed it in the updated diff.


https://reviews.llvm.org/D38688





More information about the cfe-commits mailing list