================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.h:26
@@ +25,3 @@
+  IncludeExcludeInfo();
+  IncludeExcludeInfo(llvm::StringRef Include, llvm::StringRef Exclude);
+
----------------
Better document this now to indicate what format the Include and Exclude data 
should be in (i.e. comma-separated lists of paths)

================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.h:28
@@ +27,3 @@
+
+  /// \brief Determine if the given file is safe to transform.
+  bool safeToTransform(llvm::StringRef FilePath);
----------------
"Safe to Transform" is not a concept for this class really. Maybe call it 
"isFileIncluded()".


http://llvm-reviews.chandlerc.com/D648
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to