DaddyFrosty/yoga
1
0
Fork 0
Code Issues 96 Pull Requests 31 Actions Packages Projects Releases 33 Wiki Activity

Dont redefine abort() as it may leak into other files

Browse Source 
Summary: Anyone importing this header would have abort() redefined to nothing. That's not good. Fix mistake by defining a CSS_ABORT() macro instead.

Differential Revision: D3661871

fbshipit-source-id: 14a9d076299b4f21d17a2bed68aa30b796c438a5
This commit is contained in:
Emil Sjolander
2016-08-03 01:57:46 -07:00
committed by Facebook Github Bot 7
parent 9689062f6c
commit 7af5e3d68d
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
CSSLayout/CSSMacros.h
View File

@@ -21,12 +21,14 @@
#define FB_ASSERTIONS_ENABLED 1 #define FB_ASSERTIONS_ENABLED 1
#endif #endif
#if !(FB_ASSERTIONS_ENABLED) #if FB_ASSERTIONS_ENABLED
#define abort() #define CSS_ABORT() abort()
#else
#define CSS_ABORT()
#endif #endif
#define CSS_ASSERT(X, message) \ #define CSS_ASSERT(X, message) \
if (!(X)) { \ if (!(X)) { \
fprintf(stderr, "%s\n", message); \ fprintf(stderr, "%s\n", message); \
abort(); \ CSS_ABORT(); \
} }