[PATCH] Reduce number of symbols related to the dynamic wrappers.

Manuel Klimek klimek at google.com
Tue Jun 11 01:47:59 PDT 2013



================
Comment at: lib/ASTMatchers/Dynamic/Marshallers.h:82
@@ +81,3 @@
+
+  typedef DynTypedMatcher *(*MarshallerType)(FuncType, StringRef,
+                                             const SourceRange &,
----------------
A comment would help me understand what we're now doing here...

================
Comment at: lib/ASTMatchers/Dynamic/Marshallers.h:190
@@ +189,3 @@
+  bool HasError = false;
+  for (size_t I = 0, E = Args.size(); I != E; ++I) {
+    if (!Args[I].Value.isTypedMatcher<DerivedType>()) {
----------------
I think common practice is to use "i" and "e" for counters, and I and E for iterators...


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



More information about the cfe-commits mailing list