[webkit-dev] [webkit-changes] [57262] trunk/JavaScriptCore

[Jeremy Orlow]

On Thu, Apr 8, 2010 at 5:59 PM, Alexey Proskuryakov <ap at webkit.org> wrote:

>
> On 08.04.2010, at 1:16, ossy at webkit.org wrote:
>
> + // [Qt]r57240 broke Qt build (might be a gcc bug)
>
> + // FIXME! See: https://bugs.webkit.org/show_bug.cgi?id=37253
>
>
> "FIXME! " is different from "FIXME: " in that Xcode doesn't recognize it.
>

I wasn't even aware that Xcode did recognize it or that we used that
convention because it does.  We should probably document this somewhere.


> I'm surprised that style guide doesn't say anything about FIXME vs. TODO.
>

What do you mean?  Are you suggesting that we should be using both and for
different purposes?


> But I'm not sure if a comment was even needed here - the ugliness of nested
> #ifs shouts the same.
>
>  - WBR, Alexey Proskuryakov
>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100408/2f88e83b/attachment.html>