[PATCH] Tooling: Call-back for begin/end of sources for newFrontendActionFactory

Vane, Edwin edwin.vane at intel.com
Mon Jun 3 07:58:27 PDT 2013


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. 

> -----Original Message-----
> From: Manuel Klimek [mailto:klimek at google.com]
> Sent: Monday, June 03, 2013 9:02 AM
> To: reviews+D882+public+891a0a7537cd3886 at llvm-reviews.chandlerc.com
> Cc: Daniel Jasper; Douglas Gregor; Vane, Edwin; cfe-commits at cs.uiuc.edu;
> Chandler Carruth
> Subject: Re: [PATCH] Tooling: Call-back for begin/end of sources for
> newFrontendActionFactory
> 
> On Wed, May 29, 2013 at 6:03 PM, Edwin Vane <edwin.vane at intel.com> wrote:
> 
> 
> 
> 	  Moving to post-commit review.
> 
> 
> 
> Just FYI: this is usually a discouraged pattern :) If you have the feeling that
> something needs pre-commit review once, there are very few things that would
> make it actually useful to switch to post-commit (apart from doug telling you on
> IRC, which you would then note in the review)
> 
> Cheers,
> /Manuel
> 
> 
> 
> 	http://llvm-reviews.chandlerc.com/D882
> 
> 	BRANCH
> 	  callbacks
> 
> 	ARCANIST PROJECT
> 	  clang
> 
> 





More information about the cfe-commits mailing list