[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