[PATCH] cpp11-migrate: Minor command-line fixes and improvements

Edwin Vane edwin.vane at intel.com
Wed Jul 3 09:37:09 PDT 2013



================
Comment at: cpp11-migrate/tool/Cpp11Migrate.cpp:49
@@ +48,3 @@
+    "\n"
+    "  git ls-files '*.cpp' | xargs -I{} cpp11-migrate -p build/path \\\n"
+    "    -use-nullptr -add-override -override-macros {}\n");
----------------
How about an example that doesn't use -p and allows you to specify compiler args after a '--'.

================
Comment at: cpp11-migrate/tool/Cpp11Migrate.cpp:44
@@ +43,3 @@
+    "subtree:\n\n"
+    "  find path/in/subtree -name '*.cpp' | xargs -I{} \\\n"
+    "    cpp11-migrate -p build/path -loop-convert {}\n"
----------------
You can simplify this a bit by removing `-I{}` since the filename goes at the end of the cpp11-migrate command anyway.


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



More information about the cfe-commits mailing list