<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Not necessarily -- I believe there are a number of generated headers that don't get generated if corresponding features are disabled, so the headers need to have #if ENABLED(...) checks around them... i think :D<div><br class="webkit-block-placeholder"></div><div>--Oliver</div><div><br><div><div>On Dec 31, 2007, at 8:36 PM, Jan Alonzo wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">(reposting to the list)<br><br>On Jan 1, 2008 8:58 AM, Darin Adler &lt;<a href="mailto:darin@apple.com" target="_blank">darin@apple.com</a>&gt; wrote:<br><br>&lt;snip&gt;<div class="Ih2E3d"><br><br> <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> There are two approaches -- #if in the headers and #if in source files<br>where they include the headers.<br><br>Despite what Jan Alonzo claimed in that <span class="nfakPe">bug</span>, we are consistently doing<br>the #if in all the headers for these features; the #if in source files <br>are mistakes.</blockquote></div><div><br>Yes, my mistake because i didn't poke in each of the header files and based my comment on what I've seen in some of the source files #if'ng header files. It's good to know that #ifng headers in source files is a <span class="nfakPe">bug</span>. <br><br>Thanks for the clarification.<br><br></div></div>Cheers _______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo/webkit-dev">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br></blockquote></div><br></div></body></html>