[PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check

Piotr Dziwinski via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 2 16:37:36 PDT 2015


piotrdz updated this revision to Diff 33879.
piotrdz added a comment.

In this third version I did the following:

- fixed problems which I noticed with template specializations,
- changed output diagnostics to be more detailed, covering multiple declarations
- added FixIt hints to refactor inconsistent declarations to parameter names we see in definition
- added new template-related testcases and extended existing ones with checking of note diagnostics and FixIt hints


http://reviews.llvm.org/D12462

Files:
  clang-tidy/readability/CMakeLists.txt
  clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
  clang-tidy/readability/InconsistentDeclarationParameterNameCheck.h
  clang-tidy/readability/ReadabilityTidyModule.cpp
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/readability-inconsistent-declaration-parameter-name.rst
  test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12462.33879.patch
Type: text/x-patch
Size: 22967 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150902/fdf6013a/attachment-0001.bin>


More information about the cfe-commits mailing list