<p dir="ltr">Thanks, LGTM</p>
<div class="gmail_quote">On 28 Nov 2013 06:58, "Lubos Lunak" <<a href="mailto:l.lunak@centrum.cz">l.lunak@centrum.cz</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thursday 28 of November 2013, Richard Smith wrote:<br>
> Some tiny things:<br>
><br>
> +  // Ignore also all pragma in all namespaces created<br>
> +  // in Preprocessor::RegisterBuiltinPragmas().<br>
><br>
> The English in this (copy-pasted) comment is a bit awkward; can you fix it<br>
> to something like "Also ignore all pragmas in [...]"<br>
><br>
> Please also add to your testcase an instance of a #pragma we don't normally<br>
> understand (#pragma this_pragma_does_not_exist or similar).<br>
<br>
 Updated.<br>
<br>
> Otherwise, LGTM, thanks!<br>
><br>
> On Wed, Nov 27, 2013 at 3:57 PM, Lubos Lunak <<a href="mailto:l.lunak@centrum.cz">l.lunak@centrum.cz</a>> wrote:<br>
> >  Hello,<br>
> ><br>
> >  the attached patch should fix PR9537. It is apparently necessary to<br>
> > install<br>
> > empty pragma handlers for all pragma namespaces that exist.<br>
> ><br>
> > --<br>
> >  Lubos Lunak<br>
<br>
--<br>
 Lubos Lunak<br>
</blockquote></div>