[PATCH] Added summary option to cpp11-migrate tool

Ariel Bernal ariel.j.bernal at intel.com
Fri Mar 1 12:46:26 PST 2013



================
Comment at: cpp11-migrate/Transform.h:109
@@ -108,3 +108,3 @@
 public:
-  Transform() {
+  Transform(const llvm::StringRef &name) : Name(name) {
     Reset();
----------------
Dmitri Gribenko wrote:
> Variable names should start with a capital letter.  Shadowing the member is OK.
done

================
Comment at: cpp11-migrate/Transform.h:163
@@ +162,3 @@
+  }
+  void setRejectedChanges(unsigned changes) {
+    RejectedChanges = changes;
----------------
Dmitri Gribenko wrote:
> Variable names should start with a capital letter.
done


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



More information about the cfe-commits mailing list