LGTM, go ahead and commit!
Jordan

On Mar 28, 2013, at 11:13 , Alexander Zinenko <[email protected]> wrote:

> Indeed, thanks!
> Sorry that such a simple patch takes so much time.
> Here is the updated one.
> 
> On 28 March 2013 19:53, Jordan Rose <[email protected]> wrote:
> Actually, I just realized that the SourceRange here is a token range, so 
> CreateReplacement will do the right thing even if you just pass a single 
> location. (I realized this because you shouldn't have to do the -1 offset; 
> CharSourceRange is a half-open range like other C++ ranges.)
> 
> Jordan
> 
> 
> <13824fixit.patch>

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to