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

Reply via email to