[PATCH] Adds the add-override transform to the cpp11-migrate tool.

Edwin Vane edwin.vane at intel.com
Fri Apr 5 10:24:51 PDT 2013



================
Comment at: tools/extra/cpp11-migrate/AddOverride/AddOverrideMatchers.h:21
@@ +20,3 @@
+/// Name to bind with matched expressions.
+extern const char *MethodId;
+
----------------
Jordan Rose wrote:
> Edwin Vane wrote:
> > My bad, I know I suggested `MethodId` but for LLVM style it should be `methodId`.
> Huh? LLVM style says all nouns (variables and types) are capitalized. http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly
Heh, I'm not sure why I said that. My mistake. Forget what I said.


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



More information about the cfe-commits mailing list