[PATCH] D71686: Fix false positive in magic number checker

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 22 09:29:23 PST 2019


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM aside from some minor nits.



================
Comment at: clang-tools-extra/test/clang-tidy/checkers/readability-magic-numbers-todo.cpp:2
+// RUN: %check_clang_tidy %s readability-magic-numbers %t
+// RUN: --
+// XFAIL: *
----------------
You might as well move this onto the previous line; it just looks strange here.


================
Comment at: clang-tools-extra/test/clang-tidy/checkers/readability-magic-numbers.cpp:219
+
+// prove that using enumerations values don't produce warnings
+enum class Letter : unsigned {
----------------
prove -> Prove

and add a full stop to the end of the comment.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71686/new/

https://reviews.llvm.org/D71686





More information about the cfe-commits mailing list