summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp
diff options
context:
space:
mode:
authorDmitri Gribenko <gribozavr@gmail.com>2019-10-11 12:05:42 +0000
committerDmitri Gribenko <gribozavr@gmail.com>2019-10-11 12:05:42 +0000
commit885c559369fe3d6323898c17787bd0454065fc34 (patch)
treeba43b987e078f4c2a033acc71ad3d7f1ee385a11 /clang-tools-extra/test/clang-tidy/readability-delete-null-pointer.cpp
parent9f6a873268e1ad9855873d9d8007086c0d01cf4f (diff)
downloadbcm5719-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.cpp95
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;
- }
-}
OpenPOWER on IntegriCloud