<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Nov 14, 2014 at 1:53 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Nov 13, 2014 at 9:13 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> On Sat, Nov 8, 2014 at 4:05 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> C++1z now allows attributes on namespace and enumerators. We already<br>
>> supported deprecated enumerators, but we do not support namespaces.<br>
>> This patch addresses that.<br>
>><br>
>> A few notes:<br>
>><br>
>> It was decided that the deprecated attribute only applies to things<br>
>> with names (this makes reporting diagnostics a bit more sane), so this<br>
>> patch disallows deprecating an anonymous namespace.<br>
>><br>
>> Since the declaration context can now cause deprecations, this<br>
>> requires updating some other test cases.<br>
><br>
><br>
> This wasn't the intention when this was discussed with EWG; the idea was<br>
> that using a deprecated name would result in a warning, but merely naming<br>
> something that's in a deprecated context would not.<br>
<br>
</span>Ahh, thank you for the further clarification. I've attached a revised<br>
patch which I think captures the intent a bit better. <br></blockquote><div><br></div><div>Thanks, LGTM</div><div><br></div><div>Can you also add a test for a namespace-alias (namespace M = N;) aliasing a deprecated namespace? I'm not sure if we've already got a DiagnoseUseOfDecl call in the right place for that case. <br></div></div><br></div></div>