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

Jordan Rose jordan_rose at apple.com
Fri Apr 5 09:11:28 PDT 2013



================
Comment at: tools/extra/cpp11-migrate/AddOverride/AddOverrideMatchers.h:21
@@ +20,3 @@
+/// Name to bind with matched expressions.
+extern const char *MethodId;
+
----------------
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


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



More information about the cfe-commits mailing list