[PATCH] D86691: [analyzer] Fix wrong parameter name in printFormattedEntry

Yang Fan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 31 00:42:58 PDT 2020


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



More information about the cfe-commits mailing list