[PATCH] D49114: [clang-tidy] Add a check for "magic numbers"
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 18 13:34:36 PDT 2018
lebedev.ri added inline comments.
================
Comment at: clang-tidy/readability/MagicNumbersCheck.cpp:23
+
+const char DefaultIgnoredValues[] = "0;1;2;10;100;";
+
----------------
aaron.ballman wrote:
> Why 2, 10, and 100?
These really should be a config option.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49114
More information about the cfe-commits
mailing list