[webkit-dev] #endif // Foo_h

Dirk Schulze vbs85 at gmx.de
Mon Jan 4 16:02:40 PST 2010


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.

-Dirk

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




More information about the webkit-dev mailing list