Commit b604632b authored by Nicholas Ormrod's avatar Nicholas Ormrod Committed by Sara Golemon

Change FBString's #define throw

Summary:
Instead of defining throw to nothing, define it to something
which creates a grepable error message (re @tudorb in D1005889).

@override-unit-failures

Test Plan:
fbconfig -r folly
fbmake dbg && fbmake runtests && fbmake opt && fbmake runtests_opt

All tests passing.

Added throw to a few places in FBString; noticed that error messages
were produced, and that they contained the defined text. Those changes
not committed.

Reviewed By: tudorb@fb.com

FB internal diff: D1011963
parent 8c13c0b9
...@@ -107,7 +107,7 @@ ...@@ -107,7 +107,7 @@
// FBString cannot use throw when replacing std::string, though it may still // FBString cannot use throw when replacing std::string, though it may still
// use std::__throw_* // use std::__throw_*
#define throw #define throw FOLLY_FBSTRING_MAY_NOT_USE_THROW
#ifdef _LIBSTDCXX_FBSTRING #ifdef _LIBSTDCXX_FBSTRING
namespace std _GLIBCXX_VISIBILITY(default) { namespace std _GLIBCXX_VISIBILITY(default) {
......
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