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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
