On Thu, Sep 30, 2010 at 10:31, Daniel Stutzbach <daniel@stutzbachenterprises.com> wrote:
Of course, this is only true if the core developers *do* submit to the same rules. Is anyone proposing that current core committers have all their work reviewed before it is accepted?
I think most would welcome (or at least tolerate ;) ) additional review of their code.
The hard part is encouraging contributors to find the time and motivation to thoroughly review code that they aren't personally interested in (and perhaps not even familiar with).
I definitely welcome additional, or in some cases, *any* review. Looking for reviews of Windows features/bugs sometimes equates to looking in a ghost town, but I have the self-inflicted problem of using Windows in the first place ;)
Anyways, a big +1 to expanding review, especially incorporating something like Rietveld. Although I'm replying out of order, Barry's big response lays out a lot of good ideas that I think we can use.