diff options
| author | Victor Costan <pwnall@chromium.org> | 2018-02-23 22:55:25 (GMT) |
|---|---|---|
| committer | Victor Costan <pwnall@chromium.org> | 2018-02-27 15:50:03 (GMT) |
| commit | 84ec2e0365d791e4ebc7ec249f09078fb5ab6caa (patch) | |
| tree | c408ccc78b5a07196a10ccf24dda2d65ba1b57a7 /googletest/include | |
| parent | ac34e6c950925df7165e626becd3f9d64dcd584b (diff) | |
| download | googletest-84ec2e0365d791e4ebc7ec249f09078fb5ab6caa.zip googletest-84ec2e0365d791e4ebc7ec249f09078fb5ab6caa.tar.gz googletest-84ec2e0365d791e4ebc7ec249f09078fb5ab6caa.tar.bz2 | |
Switch default death test style back to "fast".refs/pull/1464/head
Google Test has recently (02/09/2018) switched the default death test
style from "fast" to "threadsafe" in
https://github.com/google/googletest/commit/ec7faa943d7817c81ce7bdf71a21ebc9244dc8de
Threadsafe death tests have been used internally for a while, and are
proven to be a better default.
However, adopting this better default can be challenging for large
projects with a significant investment in custom infrastructure built on
top of Google Test. The same custom infrastructure can make it difficult
for large projects to switch back to the old default by passing in
--gtest_death_test_style=fast.
For the reasons above, the default switch is considered too disruptive,
and this CL reverts it. This CL also introduces the
GTEST_DEFAULT_DEATH_TEST_STYLE preprocesor macro, which replaces the
hard-coded default. The macro can be defined in
gtest/internal/custom/gtest-port.h by projects that are ready to migrate
to thread-safe death tests.
Diffstat (limited to 'googletest/include')
| -rw-r--r-- | googletest/include/gtest/internal/gtest-port.h | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/googletest/include/gtest/internal/gtest-port.h b/googletest/include/gtest/internal/gtest-port.h index 81f047b..2c819c9 100644 --- a/googletest/include/gtest/internal/gtest-port.h +++ b/googletest/include/gtest/internal/gtest-port.h @@ -107,6 +107,12 @@ // GTEST_CREATE_SHARED_LIBRARY // - Define to 1 when compiling Google Test itself // as a shared library. +// GTEST_DEFAULT_DEATH_TEST_STYLE +// - The default value of --gtest_death_test_style. +// The legacy default has been "fast" in the open +// source version since 2008. The recommended value +// is "threadsafe", and can be set in +// custom/gtest-port.h. // Platform-indicating macros // -------------------------- @@ -974,6 +980,10 @@ using ::std::tuple_size; # define GTEST_API_ #endif // GTEST_API_ +#ifndef GTEST_DEFAULT_DEATH_TEST_STYLE +# define GTEST_DEFAULT_DEATH_TEST_STYLE "fast" +#endif // GTEST_DEFAULT_DEATH_TEST_STYLE + #ifdef __GNUC__ // Ask the compiler to never inline a given function. # define GTEST_NO_INLINE_ __attribute__((noinline)) |