[PATCH] clang-apply-replacements: Add code-formatting post processing step

Guillaume Papin guillaume.papin at epitech.eu
Thu Sep 26 09:18:09 PDT 2013



================
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



More information about the cfe-commits mailing list