diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2018-04-05 14:51:01 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2018-04-05 14:51:01 +0000 |
| commit | be92ce14e13708c96eac3fe6da8c7019e24b3e2a (patch) | |
| tree | 8b7a1375c939a4f1bddef3780a19c68eab421ed2 /clang-tools-extra/test/clang-tidy | |
| parent | 92357a233679fb46957e8daeebfa44aaf9095686 (diff) | |
| download | bcm5719-llvm-be92ce14e13708c96eac3fe6da8c7019e24b3e2a.tar.gz bcm5719-llvm-be92ce14e13708c96eac3fe6da8c7019e24b3e2a.zip | |
[clang-tidy] Remove google-runtime-member-string-references
This is triggering on a pattern that's both too broad (const
std::string& members can be used safely) and too narrow (std::string is
not the only class with this problem). It has a very low true positive
rate, just remove it until we find a better solution for dangling string
references.
llvm-svn: 329292
Diffstat (limited to 'clang-tools-extra/test/clang-tidy')
| -rw-r--r-- | clang-tools-extra/test/clang-tidy/google-runtime-member-string-references.cpp | 49 |
1 files changed, 0 insertions, 49 deletions
diff --git a/clang-tools-extra/test/clang-tidy/google-runtime-member-string-references.cpp b/clang-tools-extra/test/clang-tidy/google-runtime-member-string-references.cpp deleted file mode 100644 index 64553c27f79..00000000000 --- a/clang-tools-extra/test/clang-tidy/google-runtime-member-string-references.cpp +++ /dev/null @@ -1,49 +0,0 @@ -// RUN: %check_clang_tidy %s google-runtime-member-string-references %t - -namespace std { -template<typename T> - class basic_string {}; - -typedef basic_string<char> string; -} - -class string {}; - - -struct A { - const std::string &s; -// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: const string& members are dangerous; it is much better to use alternatives, such as pointers or simple constants [google-runtime-member-string-references] -}; - -struct B { - std::string &s; -}; - -struct C { - const std::string s; -}; - -template <typename T> -struct D { - D(); - const T &s; - const std::string &s2; -// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: const string& members are dangerous -}; - -D<std::string> d; - -struct AA { - const string &s; -// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: const string& members are dangerous -}; - -struct BB { - string &s; -}; - -struct CC { - const string s; -}; - -D<string> dd; |

