<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi,<div class=""><br class=""></div><div class="">Do we still need separate WebCorePrefix.h and config.h? The former has this comment, which I don't think is true any more:</div><div class=""><br class=""></div><div class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class="">/* This prefix file should contain only:</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class=""> *    1) files to precompile for faster builds</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class=""> *    2) in one case at least: OS-X-specific performance bug workarounds</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class=""> *    3) the special trick to catch us using new or delete without including "config.h"</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class=""> * The project should be able to build without this header, although we rarely test that.</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class=""> */</div></div><div class=""><br class=""></div><div class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class="">/* Things that need to be defined globally should go into "config.h". */</div></div><div class=""><br class=""></div><div class="">There are many things that contradict this comment in this file. And even when precompiled header is not in use, we include WebCorePrefix.h from config.h anyway:</div><div class=""><br class=""></div><div class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(0, 116, 0); background-color: rgb(255, 255, 255);" class="">// Using CMake with Unix makefiles does not use prefix headers.</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(100, 56, 32); background-color: rgb(255, 255, 255);" class="">#if PLATFORM(MAC) && defined(BUILDING_WITH_CMAKE)</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(196, 26, 22); background-color: rgb(255, 255, 255);" class=""><span style="color: #643820" class="">#include </span>"WebCorePrefix.h"</div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(100, 56, 32); background-color: rgb(255, 255, 255);" class="">#endif</div></div><div class=""><br class=""></div><div class="">I'm mostly looking at some HAVE and ENABLE macros that are in these and should be elsewhere, but the confusion between these files bothers me a lot. Should we move everything from config.h to WebCorePrefix.h, and only keep config.h just to include WebCorePrefix for the lone build scenario where that's needed, and to undef new/delete?</div><div class=""><br class=""><div class="">- Alexey<br class=""><br class=""></div></div></body></html>