On Mar 17, 2011, at 11:05 AM, Elliot Glaysher (Chromium) wrote:

> Adds a -Wheader-hygiene warning for warnings that should only trigger
> in #included files.
> 
> The first -Wheader-hygiene check is to make sure a using directive
> isn't placed in the global context in a header.
> 
> On Wed, Mar 16, 2011 at 6:15 PM, Douglas Gregor <[email protected]> wrote:
>> You don't need to perform the getDiagnosticLevel() check yourself, because 
>> the diagnostic system will handle warning suppression itself.
>> 
>> However, I do suggest performing the CurContext->getDeclKind() == 
>> Decl::TranslationUnit check before the isFromMainFile() check, since the 
>> former is cheaper.
> 
> Done.

Committed in r127881, thanks!

        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to