[PATCH] [clang-tidy] Add a checker that flags unnamed parameters.

Benjamin Kramer benny.kra at gmail.com
Tue Jul 15 06:09:23 PDT 2014


Hi alexfh, djasper,

We still allow the escape hatch foo(int /*x*/) and also suggest this
in a fixit. This is more powerful than the corresponding cpplint.py check
it also flags functions with multiple arguments as naming all arguments is
recommended by the google style guide.

http://reviews.llvm.org/D4518

Files:
  clang-tidy/google/CMakeLists.txt
  clang-tidy/google/GoogleTidyModule.cpp
  clang-tidy/google/NamedParameterCheck.cpp
  clang-tidy/google/NamedParameterCheck.h
  test/clang-tidy/google-readability-function.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4518.11437.patch
Type: text/x-patch
Size: 7478 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140715/97603461/attachment.bin>


More information about the cfe-commits mailing list