[Webkit-unassigned] [Bug 34782] Normalize custom ctors for Image, Option, Audio

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Feb 20 09:37:48 PST 2010


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


Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |abarth at webkit.org




--- Comment #21 from Adam Barth <abarth at webkit.org>  2010-02-20 09:37:47 PST ---
> > p.s. Whitespace changes: AFAIK, the latest webkit style rule about whitespace
> > changes is that if I made real edits elsewhere in these files, its ok. I'm
> > using an automated tool to fix whitespace changes, and the new changes are
> > correct.
> 
> I think this rule is strange, and I don't agree with it; I'm not sure how it
> became the "latest rule". I don't think it's good to have automated tools
> making insignificant changes that harm our ability to find real changes in file
> history and provide no tangible benefit.

This might be a bit of a Chromium / WebKit culture clash.  Chromium-land is
more aggressive about cleaning up style violations than WebKit-land is.  For
folks that work in both code bases, it's easy to transplant cultural norms from
one to the other.

-- 
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