<div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 26, 2012 at 6:08 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.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="im">On Thu, Apr 26, 2012 at 5:37 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> On Thu, Apr 26, 2012 at 4:54 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> On Thu, Apr 26, 2012 at 4:36 PM, Kaelyn Uhrain <<a href="mailto:rikka@google.com">rikka@google.com</a>> wrote:<br>
>>  > +void Bar(int); // expected-note{{non-type 'Bar' shadowing class 'Bar'<br>
>> declared here}}<br>
>><br>
>> To me this sort of reads strangely - "non-type 'Bar' shadowing (class<br>
>> 'Bar' declared here)" rather than "non-type 'Bar' shadowing (class<br>
>> 'Bar') declared here" - but perhaps I'm being pedantic. "class 'Bar'<br>
>> shadows non-type 'Bar' declared here" feels more clear to me, but does<br>
>> still suffer from the same ambiguity...<br>
><br>
><br>
> I suggested earlier on IRC: "class 'Bar' is hidden by a non-type declaration<br>
> of 'Bar' here". Does that read better to you?<br>
<br>
</div>Yes, that seems unambiguous.<br></blockquote><div><br></div><div>I've committed the new wording as r155723 :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
> I'm torn between 'hidden' and<br>
> 'shadowed' -- I think the former is clearer (and is the standard term), but<br>
> the latter is already used in other diagnostics and -Wshadow.<br>
<br>
</div>I'd fall on the side of 'hidden' because I prefer standard wording<br>
where possible. Pity about the existing convention, though. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#888888"><br>
- David<br>
</font></blockquote></div><br></div>