[PATCH] cpp11-migrate: Refactor for driver model of operation

Tareq A. Siraj tareq.a.siraj at intel.com
Fri Aug 30 08:21:30 PDT 2013


  Some minor comments and please address @Sarcasm's comment about using .data().


================
Comment at: clang-replace/lib/Tooling/Makefile:1
@@ -1,1 +1,2 @@
+
 ##===- clang-replace/Makefile ------------------------------*- Makefile -*-===##
----------------
Empty line.

================
Comment at: clang-replace/lib/Tooling/Replace.cpp:37
@@ -34,2 +36,3 @@
 
+
 llvm::error_code
----------------
Empty line.

================
Comment at: cpp11-migrate/Core/FileOverrides.h:62
@@ -60,3 +61,3 @@
 
-/// \brief Container for storing override information for a single headers.
-class HeaderOverride {
+// \brief Maintains current state of transformed files and tracks source ranges
+// where changes have been made.
----------------
Use Doxygen style comments here and below.


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

BRANCH
  driver-model

ARCANIST PROJECT
  clang-tools-extra



More information about the cfe-commits mailing list