[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