[webkit-dev] Unofficial reviews (WAS: Why I'm reviewing patches outside my area (and why you should too))

2010-03-10 Thread Jeremy Orlow
On Wed, Mar 10, 2010 at 7:45 AM, Zoltan Herczeg zherc...@inf.u-szeged.huwrote: Hi, It's also a big help when peers (which aren't necessarily WebKit reviewers) look over it and give review-style feedback as well. Especially when said peers know more about that code than any of the

Re: [webkit-dev] Unofficial reviews (WAS: Why I'm reviewing patches outside my area (and why you should too))

2010-03-10 Thread Adam Treat
On Wednesday 10 March 2010 07:06:16 am Jeremy Orlow wrote: I can give you a success story though: michaeln is probably the most qualified reviewer of WebSQLDatabase code these days. He looks at most patches that go by, and I think on average he offers more and better comments than the

Re: [webkit-dev] Unofficial reviews (WAS: Why I'm reviewing patches outside my area (and why you should too))

2010-03-10 Thread Jeremy Orlow
On Wed, Mar 10, 2010 at 1:16 PM, Adam Treat tr...@kde.org wrote: On Wednesday 10 March 2010 07:06:16 am Jeremy Orlow wrote: I can give you a success story though: michaeln is probably the most qualified reviewer of WebSQLDatabase code these days. He looks at most patches that go by, and I