On Thu, Jan 5, 2012 at 7:49 AM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Not to pick on anyone in particular, but when reading bugmail I<br>
occasionally see messages like "pinging for review."  I review a lot<br>
of patches, but I don't find these messages particularly helpful<br>
because I don't know whether I'm supposed to review the patch.<br>
<br>
Another approach that might work better for you is to address your<br>
comment at someone in particular.  For example, if the message says<br>
"Adam, can you please review this patch?", then there's a pretty good<br>
chance I'll click through and try to answer your question.<br>
<br>
If you're unsure who to ask for review, one approach is to look at the<br>
svn log for the files you're changing and see who has written/reviewed<br>
patches for those files recently.  You can also ask folks who've been<br>
around the project for a while to suggest someone.<br></blockquote><div><br></div><div>True that.</div><div><br></div><div>You can also find reviewers for a particular area here on the WebKit Team wiki page: <a href="http://trac.webkit.org/wiki/WebKit%20Team">http://trac.webkit.org/wiki/WebKit%20Team</a> (and I encourage people to keep their entry up-to-date.)</div>
<div><br></div><div>-Kling</div></div>