[webkit-dev] renaming ASSERT macro

Mark Rowe mrowe at apple.com
Wed Jul 2 00:44:56 PDT 2008


On 2008-07-02, at 00:40, Jörg Bornemann wrote:

> Hi Darin,
>
> Thanks for your detailed comments!
>
>> Adding <windows.h> to Assertions.h will not cause it to be included  
>> in
>> public headers. Assertions.h is not designed to be used in public
>> headers; it's for internal use inside the WebKit project.
>
> I've just executed the following:
> find . -name '*.h' -exec grep -Hn 'Assertions.h' '{}' \;
>
> You're sure, that none of the 40+ files, the above call yields, is a
> public header or used inside a public header?
>   But well, if Assertions.h is meant to be part of the private API,
> that particular argument of mine is void. A public header including
> Assertions.h, even an indirect include, should then be considered as  
> bug?

On Mac OS X, Assertions.h is not installed on developer systems. This  
means that if any API header were to include it, applications using  
the API would be unable to build.  I would expect that the other ports  
behave similarly.

- Mark

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2413 bytes
Desc: not available
Url : http://lists.webkit.org/pipermail/webkit-dev/attachments/20080702/52fb9f10/attachment.bin 


More information about the webkit-dev mailing list