[webkit-dev] #endif // Foo_h
levin at google.com
Tue Jan 5 10:38:22 PST 2010
It sounds like no one wants this (passionately) as part of the WebKit style,
so I put up a patch to remove this warning (and I won't add that text to the
WebKit style guide when I modify it).
On Mon, Jan 4, 2010 at 4:02 PM, Dirk Schulze <vbs85 at gmx.de> wrote:
> We have somtimes constructs like
> #endif // ENABLE(SVG)
> #endif // Foo_h
> It just helps to understand why there are two endif's and what they are
> good for. I think it's not a style issue not to write this comment, but
> it can be helpful.
> Am Montag, den 04.01.2010, 15:41 -0800 schrieb Darin Adler:
> > On Jan 4, 2010, at 3:33 PM, Darin Fisher wrote:
> > > I noticed recently that the style bot started complaining about header
> files ending with "#endif" instead of "#endif // Foo_h". I was surprised by
> this since the style guide does not require it.
> > I personally see little value in this sort of comment at the end of a
> header file. I’m not sure what kind of problem it’s supposed to prevent.
> > -- Darin
> > _______________________________________________
> > webkit-dev mailing list
> > webkit-dev at lists.webkit.org
> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the webkit-dev