On Aug 4, 2010, at 7:15 AM, Jeremy Orlow wrote: > 2. ENABLE(FOO) #endif comments > > #if ENABLE(FOO) > .. > #endif // ENABLE(FOO) > > Shall we remove the comment, or require it explicitely in the style rules?
I find these comments especially helpful when there are nested #ifs involved. I also find them helpful (though less so) when there are no nested #ifs, but a lot of code is between the #if/#endif. I don't find them useful when a whole file (either .h or .cpp) is compiled out. -Adam _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev