[cfe-commits] [PATCH] Matchers for NestedNameSpecifiers

Daniel Jasper reviews at llvm-reviews.chandlerc.com
Thu Sep 13 06:12:48 PDT 2012



================
Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:2855
@@ +2854,3 @@
+      "struct A { struct B { struct C {}; }; }; A::B::C c;",
+      nestedNameSpecifierLoc(hasPrefix(loc(specifiesType(asString("struct A")))))));
+}
----------------
Manuel Klimek wrote:
> 80 columns.
Done.

================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:1024
@@ +1023,3 @@
+    return Loc.getNestedNameSpecifier();
+  }
+
----------------
Manuel Klimek wrote:
> Perhaps add // FIXME: implement extract overloads for Types.
Done.


http://llvm-reviews.chandlerc.com/D39



More information about the cfe-commits mailing list