<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 30, 2015 at 10:11 AM, Paul Robinson <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">In <a href="http://reviews.llvm.org/D2369#148975" target="_blank">http://reviews.llvm.org/D2369#148975</a>, @dblaikie wrote:<br>
<br>
> The little "comment" dropdown above the comment box (the one at the bottom<br>
>  of the review, after all the diff changes) has an "abandon" option, which I<br>
>  think is what we're doing for code reviews that end up not going anywhere.<br>
<br>
<br>
</span>Okay, I see that.  But in fact this review did "go somewhere" and a later version<br>
of the patch was actually committed.</blockquote><div><br>I think the mental model to use here is that /this/ change was abandoned - albeit in favor of some other, related change. But if /this/ change went somewhere it would've been this one that was approved/committed (if it was this change that got approved/committed, but it was just done outside Phab, you could manually mark this as accepted, and close it & just mention the revision in the message when you close the revision)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  So "abandon" is semantically wrong, even<br>
if it would do something to cause it to not be on lists of open reviews.<br>
(BTW I am completely sympathetic about getting it off lists of open reviews.<br>
I'm just at a loss for how to navigate this, shall we say, help-challenged product.)<br>
<span class=""><br>
> (there's also a "close" option too, I'm not sure of the distinction)<br>
<br>
<br>
</span>Not seeing that in the drop-down list of Actions, where is that?<br></blockquote><div><br>I have a close option - though apparently it only works if the revision has been accepted. Not sure what the equivalent action is on the owner's side (nor why it shows me an option that's not valid for the current state).<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D2369" target="_blank">http://reviews.llvm.org/D2369</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>