<div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 27, 2012 at 2:29 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
the attached patch changes<br>
<br>
  test.cc:4:11: error: unexpected ':' in nested name specifier<br>
    return A:a;<br>
            ^<br>
            ::<br>
  1 error generated.<br>
<br>
to<br>
<br>
  test.cc:4:11: error: unexpected ':' in nested name specifier, did<br>
you mean '::'?<br>
    return A:a;<br>
            ^<br>
            ::<br>
  1 error generated.<br>
<br>
<br>
True, there's a fixit for the '::' already, but it always takes me a<br>
second to see that. One coworker asked for the same change. Is this<br>
change ok? (The diagnostic is only used in one place, and always shows<br>
the '::' fixit.)</blockquote><div> </div><div>Hi Nico,</div><div><br></div><div>This seems fine to me. It's easy to not notice a small fixit like this, and we already have plenty of 'did you mean' diagnostics with matching fixits. Please use ';' rather than ',' before the 'did you mean' to match the existing diagnostics; LGTM other than that.</div>
<div><br></div><div>Thanks!</div><div>Richard</div></div></div>