|Product:||WebKit||Reporter:||Christian Dywan <christian>|
|Version:||528+ (Nightly build)|
Description Christian Dywan 2008-06-15 06:01:07 PDT
Comment 1 Christian Dywan 2008-06-15 06:03:31 PDT
Comment 2 Christian Dywan 2008-06-15 08:00:48 PDT
Created attachment 21712 [details] Use only C style comments, including CF and BSTR headers Updated patch, including CF and BSTR headers. Also the "mode" lines should now say "c" instead of "c++" where appropriate.
Comment 3 Sam Weinig 2008-06-15 10:24:59 PDT
Comment on attachment 21712 [details] Use only C style comments, including CF and BSTR headers This needs a ChangeLog.
Comment 4 Darin Adler 2008-06-15 10:41:41 PDT
I don't think it's good for public header to have mode lines.
Comment 5 Christian Dywan 2008-06-15 12:01:35 PDT
> I don't think it's good for public header to have mode lines. I tend to agree here yet left them here because virtually all source files have them at the moment. I will not commit for a few hours to see if there are others who would agree with removing the mode files entirely. Opinions welcome.
Comment 6 Alexey Proskuryakov 2008-06-15 17:17:00 PDT
Yes, please remove the mode lines.