<div class="gmail_quote">On Tue, Feb 21, 2012 at 8:47 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">On Fri, Feb 17, 2012 at 4:51 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Thu, Feb 16, 2012 at 5:31 PM, Matt Beaumont-Gay <<a href="mailto:matthewbg@google.com" target="_blank">matthewbg@google.com</a>> wrote:<br>
> On Thu, Feb 16, 2012 at 02:46, Sebastian Redl<br>
> <<a href="mailto:sebastian.redl@getdesigned.at" target="_blank">sebastian.redl@getdesigned.at</a>> wrote:<br>
>><br>
>> On 16.02.2012, at 08:11, Matt Beaumont-Gay wrote:<br>
>><br>
>>> On Wed, Feb 15, 2012 at 20:47, Eli Friedman <<a href="mailto:eli.friedman@gmail.com" target="_blank">eli.friedman@gmail.com</a>> wrote:<br>
>>>> Mechanically, the patch looks fine...<br>
>>>><br>
>>>> What sort of testing have you done with this patch?  Has this caught<br>
>>>> any interesting issues in existing code?  If not, why is this warning<br>
>>>> useful?  Any false positives we should be aware of?<br>
>>><br>
>>> I imagine that this will not have a lot of hits on checked-in code.<br>
>>> However, I can see it being very useful during development, e.g. when<br>
>>> you forget to close a namespace at the bottom of a header file.<br>
>><br>
>> But you'd most likely get an error anyway, complaining about unmatched braces.<br>
><br>
> Maybe, maybe not -- failing to close a namespace can cause a lot of<br>
> cascading errors, which could easily hit -ferror-limit.<br>
<br>
</div>How does this patch address that? It's still only warning on the<br>
closing brace, right?<br>
<br>
Cheers,<br>
/Manuel<br>
<div><div><br></div></div></blockquote></div><div> This patch moves the checking from the parser to the lexer.  This allows the error to be emitted at the end of the file instead of at the closing brace.  Since it doesn't know where the closing brace is, the warning will only point to the opening brace.</div>

</div>
</blockquote></div><br><div>Ping.  And this time, to the list and not just to Manuel.</div>