On Sat, Apr 21, 2012 at 9:45 AM, Antti Koivisto <koivi...@iki.fi> wrote:

> There is generally too much pointless drive-by refactoring going on in the
> project. I think we should take harder line against these "No new test /
> code cleanup only" type patches to reduce noise level.
>

 +1 to this.

Furthermore, I think part of the problem is drive-by reviews. Many of these
pseudo-cleanup patches may seem straightforward enough to basically
rubber-stamp, but if you don't know the code well enough it's better to
defer to someone who does. I doubt the appropriate reviewer(s) would have
reason to postpone trivial 2-line code reviews for too long anyway.

-Kling
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to