[PATCH] PR9537: Do not warn about unknown pragmas with -Eonly/-M etc.
richard at metafoo.co.uk
Thu Nov 28 13:39:09 PST 2013
On 28 Nov 2013 06:58, "Lubos Lunak" <l.lunak at centrum.cz> wrote:
> On Thursday 28 of November 2013, Richard Smith wrote:
> > Some tiny things:
> > + // Ignore also all pragma in all namespaces created
> > + // in Preprocessor::RegisterBuiltinPragmas().
> > The English in this (copy-pasted) comment is a bit awkward; can you fix
> > to something like "Also ignore all pragmas in [...]"
> > Please also add to your testcase an instance of a #pragma we don't
> > understand (#pragma this_pragma_does_not_exist or similar).
> > Otherwise, LGTM, thanks!
> > On Wed, Nov 27, 2013 at 3:57 PM, Lubos Lunak <l.lunak at centrum.cz> wrote:
> > > Hello,
> > >
> > > the attached patch should fix PR9537. It is apparently necessary to
> > > install
> > > empty pragma handlers for all pragma namespaces that exist.
> > >
> > > --
> > > Lubos Lunak
> Lubos Lunak
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits