Eugene.Zelenko added inline comments.
================ Comment at: clang-tidy/readability/MagicNumbersCheck.h:42 + return false; + }; + bool isSyntheticValue(const clang::SourceManager *SourceManager, ---------------- Unnecessary semicolon. Please also add empty line after. ================ Comment at: docs/clang-tidy/checks/readability-magic-numbers.rst:55 + +By default only 0, 1 and -1 integer values are accepted without a warning. +This can be overriden with the 'IgnoredIntegerValues' option. In addition, ---------------- Please enclose numbers in `. Same for 0.0. ================ Comment at: docs/clang-tidy/checks/readability-magic-numbers.rst:56 +By default only 0, 1 and -1 integer values are accepted without a warning. +This can be overriden with the 'IgnoredIntegerValues' option. In addition, +the 0.0 floating point value is accepted. There is no configuration for ---------------- Please use `. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49114 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits