diff options
author | Dmitri Gribenko <gribozavr@gmail.com> | 2019-10-11 12:05:42 +0000 |
---|---|---|
committer | Dmitri Gribenko <gribozavr@gmail.com> | 2019-10-11 12:05:42 +0000 |
commit | 885c559369fe3d6323898c17787bd0454065fc34 (patch) | |
tree | ba43b987e078f4c2a033acc71ad3d7f1ee385a11 /clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp | |
parent | 9f6a873268e1ad9855873d9d8007086c0d01cf4f (diff) | |
download | bcm5719-llvm-885c559369fe3d6323898c17787bd0454065fc34.tar.gz bcm5719-llvm-885c559369fe3d6323898c17787bd0454065fc34.zip |
[ClangTidy] Separate tests for infrastructure and checkers
Summary:
This change moves tests for checkers and infrastructure into separate
directories, making it easier to find infrastructure tests. Tests for
checkers are already easy to find because they are named after the
checker. Tests for infrastructure were difficult to find because they
were outnumbered by tests for checkers. Now they are in a separate
directory.
Reviewers: jfb, jdoerfert, lebedev.ri
Subscribers: srhines, nemanjai, aheejin, kbarton, christof, mgrang, arphaman, jfb, lebedev.ri, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D68807
llvm-svn: 374540
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp | 95 |
1 files changed, 0 insertions, 95 deletions
diff --git a/clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp b/clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp deleted file mode 100644 index 5a7ccae4d52..00000000000 --- a/clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp +++ /dev/null @@ -1,95 +0,0 @@ -// RUN: %check_clang_tidy %s readability-delete-null-pointer %t - -#define NULL 0 - -void f() { - int *ps = 0; - if (ps /**/) // #0 - delete ps; - // CHECK-MESSAGES: :[[@LINE-2]]:3: warning: 'if' statement is unnecessary; deleting null pointer has no effect [readability-delete-null-pointer] - - // CHECK-FIXES: int *ps = 0; - // CHECK-FIXES-NEXT: {{^ }}// #0 - // CHECK-FIXES-NEXT: delete ps; - - int *p = 0; - - // #1 - if (p) { // #2 - delete p; - } // #3 - // CHECK-MESSAGES: :[[@LINE-3]]:3: warning: 'if' statement is unnecessary; deleting null pointer has no effect [readability-delete-null-pointer] - - // CHECK-FIXES: {{^ }}// #1 - // CHECK-FIXES-NEXT: {{^ }}// #2 - // CHECK-FIXES-NEXT: delete p; - // CHECK-FIXES-NEXT: {{^ }}// #3 - - int *p2 = new int[3]; - // #4 - if (p2) // #5 - delete[] p2; - // CHECK-MESSAGES: :[[@LINE-2]]:3: warning: 'if' statement is unnecessary; - - // CHECK-FIXES: // #4 - // CHECK-FIXES-NEXT: {{^ }}// #5 - // CHECK-FIXES-NEXT: delete[] p2; - - int *p3 = 0; - if (NULL != p3) { - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: 'if' statement is unnecessary; - delete p3; - } - // CHECK-FIXES-NOT: if (NULL != p3) { - // CHECK-FIXES: delete p3; - - int *p4 = nullptr; - if (p4 != nullptr) { - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: 'if' statement is unnecessary; - delete p4; - } - // CHECK-FIXES-NOT: if (p4 != nullptr) { - // CHECK-FIXES: delete p4; - - char *c; - if (c != 0) { - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: 'if' statement is unnecessary; - delete c; - } - // CHECK-FIXES-NOT: if (c != 0) { - // CHECK-FIXES: delete c; - - char *c2; - if (c2) { - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: 'if' statement is unnecessary; - // CHECK-FIXES: } else { - // CHECK-FIXES: c2 = c; - delete c2; - } else { - c2 = c; - } - struct A { - void foo() { - if (mp) // #6 - delete mp; - // CHECK-MESSAGES: :[[@LINE-2]]:7: warning: 'if' statement is unnecessary; deleting null pointer has no effect [readability-delete-null-pointer] - // CHECK-FIXES: {{^ }}// #6 - // CHECK-FIXES-NEXT: delete mp; - } - int *mp; - }; -} - -void g() { - int *p5, *p6; - if (p5) - delete p6; - - if (p5 && p6) - delete p5; - - if (p6) { - int x = 5; - delete p6; - } -} |