<div dir="ltr">The little "comment" dropdown above the comment box (the one at the bottom of the review, after all the diff changes) has an "abandon" option, which I think is what we're doing for code reviews that end up not going anywhere.<br><br>(there's also a "close" option too, I'm not sure of the distinction)<br><br>Maybe we should be using whatever feature this refers to: <a href="https://secure.phabricator.com/T4720">https://secure.phabricator.com/T4720</a> so that we can all just mark code reviews as abandoned, rather than requesting the original author do so.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 29, 2015 at 11:16 PM, 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#148431" target="_blank">http://reviews.llvm.org/D2369#148431</a>, @chandlerc wrote:<br>
<br>
> Pretty sure this is obsolete.<br>
<br>
<br>
</span>Well, ultimately the revised content of this patch landed in r200892.  The email record exists if you want to look at it, circa Feb 5 2014.<br>
But by that time it was clear Phabricator hated me and I had stopped trying to do anything with it.<br>
More recently I've been persuaded to try it again, and a few basic things do seem to work okay.<br>
<br>
Phab's "help" is astoundingly un-helpful.  If there's something you actually want me to _do_ here, please (a) point me to a Phab help page that describes it, or (b) explain in enough detail that I can document it on LLVM's Phab page.<br>
Thanks,<br>
--paulr<br>
<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>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>