+bool DeclContext::isStdNamespace(bool IncludeInlineNamespace) const {
[...]
+ if (!getParent()->getRedeclContext()->isTranslationUnit())
+ return false;
This returns the wrong result if 'std' is itself in an inline namespace,
but I guess we don't need to care about that.
I think the default of IncludeInlineNamespace should be true, not false.
The call in SemaExceptionSpec should set it to 'true', as should all the
calls in the static analyzer, and that only leaves the
SemaTemplateInstantiateDecl call, where we don't care. Can the parameter be
removed altogether?
On Wed, Apr 30, 2014 at 3:32 PM, Richard Trieu <[email protected]> wrote:
> Take two, with proper diff context this time.
>
> http://reviews.llvm.org/D3333
>
> Files:
> include/clang/AST/DeclBase.h
> lib/AST/Decl.cpp
> lib/AST/DeclBase.cpp
> lib/Analysis/Consumed.cpp
> lib/Sema/SemaExceptionSpec.cpp
> lib/Sema/SemaTemplateInstantiateDecl.cpp
> lib/StaticAnalyzer/Checkers/LLVMConventionsChecker.cpp
> lib/StaticAnalyzer/Core/BugReporterVisitors.cpp
> lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
