<div dir="ltr">On Mon, Jun 3, 2013 at 4:58 PM, Vane, Edwin <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Alright, point taken. It was just my feeling from past review requests and comments from others saying this community is generally in favour of post-commit reviews that I thought this was the right course of action. I'll take this to mean explicit directions are required to change to post-commit.<br>
</blockquote><div><br></div><div style>Yes, the community is generally all right with post-commit review, and you should use good judgement to figure out when it's not needed. The interesting part is that once you decide that a change needs pre-commit review (usually because of being unsure about a certain part of it), you're usually right :) Always feel free to ping me about changes, or let me know if changes are the blocking part for something else you work on, so I can prioritize them.</div>
<div style><br></div><div style>Cheers,</div><div style>/Manuel</div><div style><br></div><div style><br></div><div> </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>
> -----Original Message-----<br>
> From: Manuel Klimek [mailto:<a href="mailto:klimek@google.com">klimek@google.com</a>]<br>
> Sent: Monday, June 03, 2013 9:02 AM<br>
> To: <a href="mailto:reviews%2BD882%2Bpublic%2B891a0a7537cd3886@llvm-reviews.chandlerc.com">reviews+D882+public+891a0a7537cd3886@llvm-reviews.chandlerc.com</a><br>
> Cc: Daniel Jasper; Douglas Gregor; Vane, Edwin; <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>;<br>
> Chandler Carruth<br>
> Subject: Re: [PATCH] Tooling: Call-back for begin/end of sources for<br>
> newFrontendActionFactory<br>
><br>
> On Wed, May 29, 2013 at 6:03 PM, Edwin Vane <<a href="mailto:edwin.vane@intel.com">edwin.vane@intel.com</a>> wrote:<br>
><br>
><br>
><br>
>         Moving to post-commit review.<br>
><br>
><br>
><br>
> Just FYI: this is usually a discouraged pattern :) If you have the feeling that<br>
> something needs pre-commit review once, there are very few things that would<br>
> make it actually useful to switch to post-commit (apart from doug telling you on<br>
> IRC, which you would then note in the review)<br>
><br>
> Cheers,<br>
> /Manuel<br>
><br>
><br>
><br>
>       <a href="http://llvm-reviews.chandlerc.com/D882" target="_blank">http://llvm-reviews.chandlerc.com/D882</a><br>
><br>
>       BRANCH<br>
>         callbacks<br>
><br>
>       ARCANIST PROJECT<br>
>         clang<br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>