PiotrZSL updated this revision to Diff 518353.
PiotrZSL added a comment.
Rebase
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148458/new/
https://reviews.llvm.org/D148458
Files:
clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape-throw.cpp
clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
Index: clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
===================================================================
--- clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
@@ -32,11 +32,6 @@
throw 1;
}
-void throwing_throw_nothing() throw() {
- // CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'throwing_throw_nothing' which should not throw exceptions
- throw 1;
-}
-
void throw_and_catch() noexcept {
// CHECK-MESSAGES-NOT: :[[@LINE-1]]:6: warning: an exception may be thrown
in function 'throw_and_catch' which should not throw exceptions
try {
@@ -557,7 +552,9 @@
throw 1;
}
-void explicit_int_thrower() throw(int);
+void explicit_int_thrower() noexcept(false) {
+ throw 1;
+}
void indirect_implicit() noexcept {
// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'indirect_implicit' which should not throw exceptions
@@ -676,15 +673,6 @@
// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in
function 'sub_throws' which should not throw exceptions
};
-struct super_throws_again {
- super_throws_again() throw(int);
-};
-
-struct sub_throws_again : super_throws_again {
- sub_throws_again() noexcept : super_throws_again() {}
- // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in
function 'sub_throws_again' which should not throw exceptions
-};
-
struct init_member_throws {
super_throws s;
Index:
clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape-throw.cpp
===================================================================
--- /dev/null
+++
clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape-throw.cpp
@@ -0,0 +1,31 @@
+// RUN: %check_clang_tidy -std=c++11,c++14 %s bugprone-exception-escape %t --
-- -fexceptions
+
+void throwing_throw_nothing() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'throwing_throw_nothing' which should not throw exceptions
+ throw 1;
+}
+
+void explicit_int_thrower() throw(int);
+
+void implicit_int_thrower() {
+ throw 5;
+}
+
+void indirect_implicit() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'indirect_implicit' which should not throw exceptions
+ implicit_int_thrower();
+}
+
+void indirect_explicit() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'indirect_explicit' which should not throw exceptions
+ explicit_int_thrower();
+}
+
+struct super_throws {
+ super_throws() throw(int) { throw 42; }
+};
+
+struct sub_throws : super_throws {
+ sub_throws() throw() : super_throws() {}
+ // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in
function 'sub_throws' which should not throw exceptions
+};
Index: clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
===================================================================
--- clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape.cpp
@@ -32,11 +32,6 @@
throw 1;
}
-void throwing_throw_nothing() throw() {
- // CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'throwing_throw_nothing' which should not throw exceptions
- throw 1;
-}
-
void throw_and_catch() noexcept {
// CHECK-MESSAGES-NOT: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'throw_and_catch' which should not throw exceptions
try {
@@ -557,7 +552,9 @@
throw 1;
}
-void explicit_int_thrower() throw(int);
+void explicit_int_thrower() noexcept(false) {
+ throw 1;
+}
void indirect_implicit() noexcept {
// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'indirect_implicit' which should not throw exceptions
@@ -676,15 +673,6 @@
// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in function 'sub_throws' which should not throw exceptions
};
-struct super_throws_again {
- super_throws_again() throw(int);
-};
-
-struct sub_throws_again : super_throws_again {
- sub_throws_again() noexcept : super_throws_again() {}
- // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in function 'sub_throws_again' which should not throw exceptions
-};
-
struct init_member_throws {
super_throws s;
Index: clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape-throw.cpp
===================================================================
--- /dev/null
+++ clang-tools-extra/test/clang-tidy/checkers/bugprone/exception-escape-throw.cpp
@@ -0,0 +1,31 @@
+// RUN: %check_clang_tidy -std=c++11,c++14 %s bugprone-exception-escape %t -- -- -fexceptions
+
+void throwing_throw_nothing() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'throwing_throw_nothing' which should not throw exceptions
+ throw 1;
+}
+
+void explicit_int_thrower() throw(int);
+
+void implicit_int_thrower() {
+ throw 5;
+}
+
+void indirect_implicit() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'indirect_implicit' which should not throw exceptions
+ implicit_int_thrower();
+}
+
+void indirect_explicit() throw() {
+// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in function 'indirect_explicit' which should not throw exceptions
+ explicit_int_thrower();
+}
+
+struct super_throws {
+ super_throws() throw(int) { throw 42; }
+};
+
+struct sub_throws : super_throws {
+ sub_throws() throw() : super_throws() {}
+ // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: an exception may be thrown in function 'sub_throws' which should not throw exceptions
+};
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
