================
Comment at: 
clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h:101-103
@@ -93,1 +100,5 @@
 
+/// \brief Given a collection of Replacements and existing changed source 
ranges
+/// for a single file, updates the list of source ranges to enclose \c
+/// Replacements.
+///
----------------
Is this comment still relevant? The list is not "updated" but just filled-in 
now.

================
Comment at: 
clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h:106
@@ +105,3 @@
+/// \param[in] Replacements Replacements from a single file.
+/// \param[in,out] ChangedRanges Upon input, this vector may contain a list of
+/// source ranges describe where changes were previously made to the file. Upon
----------------
`param [out]`

================
Comment at: unittests/clang-apply-replacements/ReformattingTest.cpp:1
@@ +1,2 @@
+//===- clang-modernize/ReformattingTest.cpp - Reformatting unit tests 
-----===//
+//
----------------
clang-modernize -> clang-apply-replacements


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

Reply via email to