Commit 6fcad841 authored by Yedidya Feldblum's avatar Yedidya Feldblum Committed by Facebook GitHub Bot

fix the deprecation suppression for gcc in the exception_wrapper test

Summary: By suppressing `-Wdeprecated-declarations` v.s. `-Wdeprecated`, which latter works only under clang.

Reviewed By: Orvid

Differential Revision: D27987616

fbshipit-source-id: 4bf2a225a65d8ac7c7555123343e9536759fcfcc
parent e6e7080e
...@@ -120,7 +120,7 @@ TEST(ExceptionWrapper, try_and_catch_test) { ...@@ -120,7 +120,7 @@ TEST(ExceptionWrapper, try_and_catch_test) {
// Catch rightmost matching exception type // Catch rightmost matching exception type
FOLLY_PUSH_WARNING FOLLY_PUSH_WARNING
FOLLY_GNU_DISABLE_WARNING("-Wdeprecated") FOLLY_GNU_DISABLE_WARNING("-Wdeprecated-declarations")
exception_wrapper ew = try_and_catch<std::exception, std::runtime_error>( exception_wrapper ew = try_and_catch<std::exception, std::runtime_error>(
[=]() { throw std::runtime_error(expected); }); [=]() { throw std::runtime_error(expected); });
FOLLY_POP_WARNING FOLLY_POP_WARNING
...@@ -133,7 +133,7 @@ TEST(ExceptionWrapper, try_and_catch_test) { ...@@ -133,7 +133,7 @@ TEST(ExceptionWrapper, try_and_catch_test) {
// Changing order is like catching in wrong order. Beware of this in your // Changing order is like catching in wrong order. Beware of this in your
// code. // code.
FOLLY_PUSH_WARNING FOLLY_PUSH_WARNING
FOLLY_GNU_DISABLE_WARNING("-Wdeprecated") FOLLY_GNU_DISABLE_WARNING("-Wdeprecated-declarations")
auto ew2 = try_and_catch<std::runtime_error, std::exception>( auto ew2 = try_and_catch<std::runtime_error, std::exception>(
[=]() { throw std::runtime_error(expected); }); [=]() { throw std::runtime_error(expected); });
FOLLY_POP_WARNING FOLLY_POP_WARNING
...@@ -145,7 +145,7 @@ TEST(ExceptionWrapper, try_and_catch_test) { ...@@ -145,7 +145,7 @@ TEST(ExceptionWrapper, try_and_catch_test) {
// Catches even if not rightmost. // Catches even if not rightmost.
FOLLY_PUSH_WARNING FOLLY_PUSH_WARNING
FOLLY_GNU_DISABLE_WARNING("-Wdeprecated") FOLLY_GNU_DISABLE_WARNING("-Wdeprecated-declarations")
auto ew3 = try_and_catch<std::exception, std::runtime_error>( auto ew3 = try_and_catch<std::exception, std::runtime_error>(
[]() { throw std::exception(); }); []() { throw std::exception(); });
FOLLY_POP_WARNING FOLLY_POP_WARNING
...@@ -157,7 +157,7 @@ TEST(ExceptionWrapper, try_and_catch_test) { ...@@ -157,7 +157,7 @@ TEST(ExceptionWrapper, try_and_catch_test) {
// If does not catch, throws. // If does not catch, throws.
FOLLY_PUSH_WARNING FOLLY_PUSH_WARNING
FOLLY_GNU_DISABLE_WARNING("-Wdeprecated") FOLLY_GNU_DISABLE_WARNING("-Wdeprecated-declarations")
EXPECT_THROW( EXPECT_THROW(
try_and_catch<std::runtime_error>([]() { throw std::exception(); }), try_and_catch<std::runtime_error>([]() { throw std::exception(); }),
std::exception); std::exception);
...@@ -496,7 +496,7 @@ TEST(ExceptionWrapper, non_std_exception_test) { ...@@ -496,7 +496,7 @@ TEST(ExceptionWrapper, non_std_exception_test) {
int expected = 17; int expected = 17;
FOLLY_PUSH_WARNING FOLLY_PUSH_WARNING
FOLLY_GNU_DISABLE_WARNING("-Wdeprecated") FOLLY_GNU_DISABLE_WARNING("-Wdeprecated-declarations")
exception_wrapper ew = exception_wrapper ew =
try_and_catch<std::exception, int>([=]() { throw expected; }); try_and_catch<std::exception, int>([=]() { throw expected; });
FOLLY_POP_WARNING FOLLY_POP_WARNING
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment