Skip to content

Commit

Permalink
Fix deprecated dynamic exception in fboss/agent/hw/bcm/BcmError.h +5
Browse files Browse the repository at this point in the history
Summary:
X-link: facebookresearch/fbpcf#539

LLVM has detected a violation of `-Wdeprecated-dynamic-exception-spec`. Dynamic exceptions were removed in C++17. This diff fixes the deprecated instance(s).

See [Dynamic exception specification](https://en.cppreference.com/w/cpp/language/except_spec) and [noexcept specifier](https://en.cppreference.com/w/cpp/language/noexcept_spec).

Reviewed By: palmje

Differential Revision: D57636124

fbshipit-source-id: 90ff1837d7d7aa919ea19d380b638a04b7d19adf
  • Loading branch information
r-barnes authored and facebook-github-bot committed May 23, 2024
1 parent 76ed72a commit df0d8d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion folly/test/ExceptionWrapperTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -473,7 +473,7 @@ namespace {
struct ThrownException {};
struct InSituException : std::exception {
InSituException() = default;
InSituException(const InSituException&) throw() {}
InSituException(const InSituException&) noexcept {}
};
struct OnHeapException : std::exception {
OnHeapException() = default;
Expand Down

0 comments on commit df0d8d9

Please sign in to comment.