[cfe-dev] Interest in a -Wusing-namespace-in-headers?

Elliot Glaysher (Chromium) erg at chromium.org
Wed Mar 16 18:12:01 PDT 2011


Attached is a new version, using Douglas's suggestion for a
-Wheader-hygiene flag, as I intend to add the anonymous namespace
warning next.

On Wed, Mar 16, 2011 at 4:51 PM, Daniel James <dnljms at gmail.com> wrote:
> But it doesn't really matter where it is or isn't used. What matters
> is that a properly scoped import is usually safe, and warning about
> one would be counter productive.

This version only warns in the global context and not warning when a
using declarative is done inside a namespace is covered by the tests.

Regards,

-- Elliot Glaysher
-------------- next part --------------
A non-text attachment was scrubbed...
Name: using-namespace-2.diff
Type: text/x-patch
Size: 3712 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20110316/9a59be50/attachment.bin>


More information about the cfe-dev mailing list