[Webkit-unassigned] [Bug 31806] Make GIF decoder support down-sampling

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 2 05:25:13 PST 2009


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





--- Comment #10 from George Staikos <staikos at kde.org>  2009-12-02 05:25:13 PST ---
(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > We shouldn't violate the style guide simply because we don't like some of its
> > > rules, or there is no point to having a style guide.
> > 
> > That's why they're "guidelines" and not hard rules.
> 
> The “Contributing Code” page at <http://webkit.org/coding/contributing.html>
> could hardly be clearer about this:
> 
> > Patches must comply with the code style guidelines.
> 
> It’s also worth noting that the reviewer policy explicitly states:
> 
> > Reviewers are expected to ensure that patches they review follow project policies.
> 
> Coding style is one such policy.  Project policy should be followed, not
> personal preference.

The amount of time spent on this policy with no valid argument that I can see
is awesome.  Just imagine how many bugs we didn't implement because we spent
time debating where we put the braces in two perfectly reasonable and clean
cases instead of writing more code.  Now we even write policing tools for it! 
Extra awesome.  I propose that the next step is we optimize gcc to only accept
code according to the webkit coding style.  Surely the parser will be faster,
resulting in faster compile times of WebKit which in the end make this all
worthwhile. A+

Now back to getting something real done.

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


More information about the webkit-unassigned mailing list