[Patch] Fix for PR9812
    Erik Verbruggen 
    erik.verbruggen at me.com
       
    Tue Jan  7 07:28:24 PST 2014
    
    
  
> On Sat, Dec 14, 2013 at 5:31 AM, Erik Verbruggen <erik.verbrug... at me.com> wrote:
> > This is a fix for PR9812, and I have to say that I don't know what more to 
> > say in the commit message than "warn about bool instead of _Bool", so 
> > feedback is welcome! I sometimes run into this, hence the patch.
> >
> > Two notes: I have to say that the patch feels a bit intrusive, but I didn't 
> > find a better way to do this. Also, in MS mode, it will warn about __wchar_t 
> > v.s. wchar_t. I haven't extensively tested this on Windows, but to me it 
> > looks like the correct thing. Again, feedback welcome!
> 
> Thank you for working on this!  This is a strict improvement for C++,
> the issue still remains for C.  In that case, we should do something
> like Sema::getPrintingPolicy().  We should not be printing 'bool' in C
> unless the said macro exists -- we need tests for both cases.  Will
> you fix the C case as well?
I can do that. Should I do that in a separate commit, or should I do it in this patch?
-- Erik
    
    
More information about the cfe-commits
mailing list