[PATCH] D61480: Added an AST matcher for declarations that are in the `std` namespace

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 3 03:58:33 PDT 2019


gribozavr marked 2 inline comments as done.
gribozavr added inline comments.


================
Comment at: clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp:2079
+}
+
 TEST(EqualsBoundNodeMatcher, QualType) {
----------------
aaron.ballman wrote:
> Can you also add a negative test case for:
> ```
> namespace foo {
> namespace std {
>   void bar(); // isInStdNamespace() is false here, right?
> }
> }
> ```
Done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61480/new/

https://reviews.llvm.org/D61480





More information about the cfe-commits mailing list