Minor comments, otherwise LGTM.
================
Comment at: cpp11-migrate/Core/FileOverrides.h:30
@@ +29,3 @@
+struct SourceOverrides {
+ //SourceOverrides() {}
+
----------------
Remove commented out code if not needed.
================
Comment at: cpp11-migrate/Core/SyntaxCheck.cpp:39
@@ +38,3 @@
+
+class EndSyntaxArgumentsAdjuster : public ArgumentsAdjuster {
+ CommandLineArguments Adjust(const CommandLineArguments &Args) {
----------------
Maybe call it FinalSyntaxArgumentAdjuster to be consistent with the Action name?
http://llvm-reviews.chandlerc.com/D964
BRANCH
override_structs
ARCANIST PROJECT
clang-tools-extra
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
