[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