[PATCH] D22507: Clang-tidy - Enum misuse check

Peter Szecsi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 25 16:19:24 PST 2016


szepet updated this revision to Diff 82488.
szepet marked 6 inline comments as done.
szepet added a comment.

The requested changes have been made.
Some more refactor on the Case2 since it is the same as the LHS/RHS case. Moved more common statements out of the branch (Case2-3) for better readabilty. (And less code duplication.)


https://reviews.llvm.org/D22507

Files:
  clang-tidy/misc/CMakeLists.txt
  clang-tidy/misc/MiscTidyModule.cpp
  clang-tidy/misc/SuspiciousEnumUsageCheck.cpp
  clang-tidy/misc/SuspiciousEnumUsageCheck.h
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/misc-suspicious-enum-usage.rst
  test/clang-tidy/misc-suspicious-enum-usage-strict.cpp
  test/clang-tidy/misc-suspicious-enum-usage.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22507.82488.patch
Type: text/x-patch
Size: 18712 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161226/6c53328f/attachment-0001.bin>


More information about the cfe-commits mailing list