[PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check
Piotr Dziwinski via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 8 14:52:50 PDT 2015
piotrdz updated this revision to Diff 34267.
piotrdz marked 10 inline comments as done.
piotrdz added a comment.
I hope this is the final re-write of my code. In this version, I addressed most recent review comments, while also refactoring code to better handle template specializations and correctly display diagnostics in case when function definition is visible. In the end I had to change over half of the code, but perhaps it was worth it.
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.34267.patch
Type: text/x-patch
Size: 30247 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150908/8d615bad/attachment-0001.bin>
More information about the cfe-commits
mailing list