apart from that, lg

================
Comment at: unittests/ASTMatchers/ASTMatchersTest.h:55
@@ -55,1 +54,3 @@
+                                              bool ExpectMatch,
+                                              std::string Arg) {
   bool Found = false;
----------------
Either make that const std::string& or llvm::StringRef.
Also, I'd call it CompileArg (Arg might mean anything).


http://llvm-reviews.chandlerc.com/D46
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to