[Webkit-unassigned] [Bug 57535] Fix compilation on Solaris 10 with Sun Studio 12, missing definition for time_t

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=57535





--- Comment #18 from David Levin <levin at chromium.org>  2011-04-09 20:30:12 PST ---
(In reply to comment #15)
> cosmetic fix, despite having logged bugs against check-webkit-style

I see no bugs filed by you about check-webkit-style.

Also I've looked at all issues mentioned in the bug and they are all real problems in your patches.

1. There should only be one space before end of line comments, so this
# "header.h"        // This is for whatever.
doesn't follow the proper style, but this does
# "header.h" // This is for whatever.

2. Headers that aren't ifdef'ed are sorted.  Yours is not. I see
#include <wtf/Platform.h>
   occurring after things like
#include <wtf/Vector.h>

which isn't alphabetical.

3. When you run check-webkit-style on a file without your changes and get other issues, you can actually see that it is working correctly. It did not point out these problems on your patches because they were not in lines that you changed. However, in the patches that you submitted, it is pointing out correct issues on correct lines. :)

PS check-webkit-style does have some issues at times but we strive to have them be pretty rare and none are exhibited in these issues as far as I see.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.