1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

src: remove GTEST_DONT_DEFINE_ASSERT_EQ in util.h

Browse Source 
As indicated by the FIXME comment, this macro guard is no longer needed.

PR-URL: https://github.com/nodejs/node/pull/12638
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
This commit is contained in:
Daniel Bevenius 2017-04-25 10:27:15 +02:00 committed by Anna Henningsen
parent e927809eec
commit 5bb06e8596
No known key found for this signature in database
GPG Key ID: D8B9F5AEAE84E4CF
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/util.h
View File

@ -124,14 +124,6 @@ template <typename T> using remove_reference = std::remove_reference<T>;
} \ } \
} while (0) } while (0)
// FIXME(bnoordhuis) cctests don't link in node::Abort() and node::Assert().
#ifdef GTEST_DONT_DEFINE_ASSERT_EQ
#undef ABORT
#undef CHECK
#define ABORT ABORT_NO_BACKTRACE
#define CHECK assert
#endif
#ifdef NDEBUG #ifdef NDEBUG
#define ASSERT(expr) #define ASSERT(expr)
#else #else