[PATCH] D27621: [clang-tidy] check to find declarations declaring more than one name

Firat Kasmis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 08:40:32 PST 2016


firolino updated this revision to Diff 81591.
firolino added a comment.

- small improvements
- re-added lost handling for:

  bool defPre = false,
  #ifdef IS_ENABLED
         defTest = false;
  #else
         defTest = true;
  #endif

- added support for new test cases:

  int /* :: */ S::*pp2 = &S::a, var1 = 0;
  void /*(*/ ( /*(*/ *f3)(int), (*g3)(int, float);
  /* *& */ int /* *& */ ** /* *& */ pp,*xx;


https://reviews.llvm.org/D27621

Files:
  clang-tidy/readability/CMakeLists.txt
  clang-tidy/readability/OneNamePerDeclarationCheck.cpp
  clang-tidy/readability/OneNamePerDeclarationCheck.h
  clang-tidy/readability/ReadabilityTidyModule.cpp
  clang-tidy/utils/LexerUtils.cpp
  clang-tidy/utils/LexerUtils.h
  docs/ReleaseNotes.rst
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/readability-one-name-per-declaration.rst
  test/clang-tidy/readability-one-name-per-declaration-complex.cpp
  test/clang-tidy/readability-one-name-per-declaration-modern.cpp
  test/clang-tidy/readability-one-name-per-declaration-simple.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27621.81591.patch
Type: text/x-patch
Size: 36985 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161215/f1080770/attachment-0001.bin>


More information about the cfe-commits mailing list