nullptr.cpp added a comment. Herald added a subscriber: danielkiss. In D86691#2241886 <https://reviews.llvm.org/D86691#2241886>, @martong wrote:
> In D86691#2241604 <https://reviews.llvm.org/D86691#2241604>, @nullptr.cpp > wrote: > >> In D86691#2241401 <https://reviews.llvm.org/D86691#2241401>, @Szelethus >> wrote: >> >>> Nice, thank you! Did you stumble across this, or found it with a tool? >> >> I just happened to see these code:-). >> Maybe we should write a checker to check this situation? I will start doing >> this! > > Good idea! And thanks for the fix! :) There already have a clang-tidy checker can check this situation. See readability-inconsistent-declaration-parameter-name <http://clang.llvm.org/extra/clang-tidy/checks/readability-inconsistent-declaration-parameter-name.html> for more information. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86691/new/ https://reviews.llvm.org/D86691 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
