<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 21, 2014 at 5:38 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On 21/05/2014 23:19, David Majnemer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: majnemer<br>
Date: Wed May 21 15:19:59 2014<br>
New Revision: 209319<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=209319&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=209319&view=rev</a><br>
Log:<br>
Sema: Implement <span class="il">DR244</span><br>
<br>
Summary:<br>
Naming the destructor using a typedef-name for the class-name is<br>
well-formed.<br>
<br>
This fixes PR19620.<br>
<br>
Reviewers: rsmith, doug.gregor<br>
</blockquote>
<br></div>
Did Doug participate in review for this patch?<br></blockquote><div><br></div><div>No, Richard Smith did. I see a pretty complete review thread for the DR244 patch in my inbox. I've even checked and none of the emails were lost in the recent email list snafu.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Looking through SVN history, I'm seeing an alarming number of confusing review trails.<br></blockquote><div><br></div><div>Can you point them out specifically? I too keep a pretty close eye on these kinds of things and I'm not seeing any examples.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If review happened off-list that's fine, but it needs to be stated clearly because the system works on trust.<br></blockquote><div><br></div><div>I don't think off-list review is fine... It happens some times, for good or bad reasons, and its not the end of the world. But it is definitely not SOP, and I haven't seen any evidence of it becoming more pervasive. If you see it, call it out. When patches merit pre-commit review, they should get it from the whole community.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
(In fact, I'm seeing relatively inactive developer names showing up suspiciously in these "Reviewers" lines while some of the most active reviewers barely appear at all. Could this be a problem with Phabricator or some internal system you guys are using?)</blockquote>
</div><br></div><div class="gmail_extra">I'm really not sure what you're worried about here. Again, specific examples?</div></div>