diff options
author | Sam McCall <sam.mccall@gmail.com> | 2018-07-13 11:41:56 +0000 |
---|---|---|
committer | Sam McCall <sam.mccall@gmail.com> | 2018-07-13 11:41:56 +0000 |
commit | 57b66c8ba324787bec1edc6fce0e6c1dc7aad239 (patch) | |
tree | 9f9a6d4b26a12fab795b237b36109724102c5c10 /clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp | |
parent | 1a728f66efbf02f703a1e782efd4172842fd1bb8 (diff) | |
download | bcm5719-llvm-57b66c8ba324787bec1edc6fce0e6c1dc7aad239.tar.gz bcm5719-llvm-57b66c8ba324787bec1edc6fce0e6c1dc7aad239.zip |
[clang-tidy] readability-inconsistent-declaration-parameter-name: accept approximate name matches.
Summary:
The goal is to reduce false positives when the difference is intentional, like:
foo(StringRef name);
foo(StringRef name_ref) {
string name = cleanup(name_ref);
...
}
Or semantically unimportant, like:
foo(StringRef full_name);
foo(StringRef name) { ... }
There are other matching names we won't recognise (e.g. syns vs synonyms) but
this catches many that we see in practice, and gives people a systematic
workaround.
The old behavior is available as a 'Strict' option.
Subscribers: xazax.hun, cfe-commits
Differential Revision: https://reviews.llvm.org/D49285
llvm-svn: 336992
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp b/clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp index 43412f528fe..c98258bd931 100644 --- a/clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-inconsistent-declaration-parameter-name.cpp @@ -2,6 +2,8 @@ void consistentFunction(int a, int b, int c); void consistentFunction(int a, int b, int c); +void consistentFunction(int prefixA, int b, int cSuffix); +void consistentFunction(int a, int b, int c); void consistentFunction(int a, int b, int /*c*/); void consistentFunction(int /*c*/, int /*c*/, int /*c*/); |