31 #ifdef __cplusplus 32 #define new ("if you use new/delete make sure to include config.h at the top of the file"()) 33 #define delete ("if you use new/delete make sure to include config.h at the top of the file"()) This change caused build error in moc_qwkhistory.cpp and moc_qwkpage.cpp . These files are generated from qwkhistory.h and qwkpage.h by Qt's moc and they only inlcude its own header file. To fix this build break I added including config.h to these headers: http://trac.webkit.org/changeset/77088 I know we shouldn't include config.h in headers, but I don't know if it is possible to make moc add including config.h to moc_[...].cpp files. I leave this bug open to find a correct solution for this build break.
Adding Source/JavascriptCore to INCLUDEPATH was needed, because config.h includes 3 headears from wtf: - wtf/FastMalloc.h - wtf/DisallowCType.h - wtf/Platform.h
The correct solution is to throw away the prefix header let trust in ourselves and check-webkit-style to not forget about config.h. This is our policy in WebCore and JavaScriptCore as well.
Created attachment 80597 [details] Patch
Comment on attachment 80597 [details] Patch Clearing flags on attachment: 80597 Committed r77092: <http://trac.webkit.org/changeset/77092>
All reviewed patches have been landed. Closing bug.