[clang-tools-extra] r248080 - clang-tools-extra: Appease PR24881. [-Wdocumentation]

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 19:21:29 PDT 2015


Author: chapuni
Date: Fri Sep 18 21:21:28 2015
New Revision: 248080

URL: http://llvm.org/viewvc/llvm-project?rev=248080&view=rev
Log:
clang-tools-extra: Appease PR24881. [-Wdocumentation]

\returns doesn't accept \li, but \parblock \li.

Modified:
    clang-tools-extra/trunk/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h
    clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
    clang-tools-extra/trunk/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp
    clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.h
    clang-tools-extra/trunk/clang-modernize/Core/ReplacementHandling.h
    clang-tools-extra/trunk/clang-modernize/Core/Transform.h

Modified: clang-tools-extra/trunk/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h (original)
+++ clang-tools-extra/trunk/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h Fri Sep 18 21:21:28 2015
@@ -84,7 +84,8 @@ collectReplacementsFromDirectory(const l
 /// file they target.
 /// \param[in] SM SourceManager required for conflict reporting.
 ///
-/// \returns \li true If all changes were applied successfully.
+/// \returns \parblock
+///          \li true If all changes were applied successfully.
 ///          \li false If there were conflicts.
 bool mergeAndDeduplicate(const TUReplacements &TUs,
                          FileToReplacementsMap &GroupedReplacements,
@@ -97,7 +98,8 @@ bool mergeAndDeduplicate(const TUReplace
 /// \param[out] Rewrites The results of applying replacements will be applied
 /// to this Rewriter.
 ///
-/// \returns \li true If all changes were applied successfully.
+/// \returns \parblock
+///          \li true If all changes were applied successfully.
 ///          \li false If a replacement failed to apply.
 bool applyReplacements(const FileToReplacementsMap &GroupedReplacements,
                        clang::Rewriter &Rewrites);
@@ -125,7 +127,8 @@ bool writeFiles(const clang::Rewriter &R
 /// \param[in] Files Replacement files to delete.
 /// \param[in] Diagnostics DiagnosticsEngine used for error output.
 ///
-/// \returns \li true If all files have been deleted successfully.
+/// \returns \parblock
+///          \li true If all files have been deleted successfully.
 ///          \li false If at least one or more failures occur when deleting
 /// files.
 bool deleteReplacementFiles(const TUReplacementFiles &Files,

Modified: clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp (original)
+++ clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp Fri Sep 18 21:21:28 2015
@@ -131,7 +131,8 @@ static void reportConflict(
 /// to be deduplicated and checked for conflicts.
 /// \param[in] SM SourceManager required for conflict reporting.
 ///
-/// \returns \li true if conflicts were detected
+/// \returns \parblock
+///          \li true if conflicts were detected
 ///          \li false if no conflicts were detected
 static bool deduplicateAndDetectConflicts(FileToReplacementsMap &Replacements,
                                           SourceManager &SM) {

Modified: clang-tools-extra/trunk/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp (original)
+++ clang-tools-extra/trunk/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp Fri Sep 18 21:21:28 2015
@@ -100,7 +100,8 @@ static void printVersion() {
 /// \param[out] Result Contents of the file after applying replacements if
 /// replacements were provided.
 ///
-/// \returns \li true if all replacements were applied successfully.
+/// \returns \parblock
+///          \li true if all replacements were applied successfully.
 ///          \li false if at least one replacement failed to apply.
 static bool
 getRewrittenData(const std::vector<tooling::Replacement> &Replacements,
@@ -134,7 +135,8 @@ getRewrittenData(const std::vector<tooli
 /// replacements were provided.
 /// \param[in] Diagnostics For diagnostic output.
 ///
-/// \returns \li true if all replacements applied successfully.
+/// \returns \parblock
+///          \li true if all replacements applied successfully.
 ///          \li false if at least one replacement failed to apply.
 static bool
 applyReplacements(const std::vector<tooling::Replacement> &Replacements,
@@ -160,7 +162,8 @@ applyReplacements(const std::vector<tool
 /// \param[in] FormatStyle Style to apply.
 /// \param[in] Diagnostics For diagnostic output.
 ///
-/// \returns \li true if reformatting replacements were all successfully
+/// \returns \parblock
+///          \li true if reformatting replacements were all successfully
 ///          applied.
 ///          \li false if at least one reformatting replacement failed to apply.
 static bool

Modified: clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.h?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.h (original)
+++ clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.h Fri Sep 18 21:21:28 2015
@@ -41,7 +41,7 @@ public:
   /// \param File A file accessible by a SourceManager
   /// \param Include The include file as it should be written in the code.
   ///
-  /// \returns
+  /// \returns \parblock
   /// \li A null Replacement (check using \c Replacement::isApplicable()), if
   ///     the \c Include is already visible from \c File.
   /// \li Otherwise, a non-null Replacement that, when applied, inserts an

Modified: clang-tools-extra/trunk/clang-modernize/Core/ReplacementHandling.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/Core/ReplacementHandling.h?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-modernize/Core/ReplacementHandling.h (original)
+++ clang-tools-extra/trunk/clang-modernize/Core/ReplacementHandling.h Fri Sep 18 21:21:28 2015
@@ -31,7 +31,8 @@ public:
   ///
   /// \param[in] Argv0 argv[0] as passed to main().
   ///
-  /// \returns \li true if clang-apply-replacements was found.
+  /// \returns \parblock
+  ///          \li true if clang-apply-replacements was found.
   ///          \li false otherwise.
   bool findClangApplyReplacements(const char *Argv0);
 
@@ -65,7 +66,8 @@ public:
   ///
   /// \param[in] Replacements Container of replacements to serialize.
   ///
-  /// \returns \li true if all replacements were serialized successfully to
+  /// \returns \parblock
+  ///          \li true if all replacements were serialized successfully to
   ///          disk.
   ///          \li false otherwise.
   bool serializeReplacements(const TUReplacementsMap &Replacements);
@@ -76,7 +78,8 @@ public:
   /// \pre Destination directory must have been previously set by calling
   /// setDestiantionDir() or useTempDestinationDir().
   ///
-  /// \returns \li true if clang-apply-replacements was successfully launched
+  /// \returns \parblock
+  ///          \li true if clang-apply-replacements was successfully launched
   ///          and successfully completed.
   ///          \li false otherwise.
   bool applyReplacements();
@@ -97,7 +100,8 @@ public:
   /// \param[out] Error If an error occurs a description of that error is
   ///             placed in this string.
   ///
-  /// \returns \li true on success
+  /// \returns \parblock
+  ///          \li true on success
   ///          \li false if a unique file name could not be created.
   static bool generateReplacementsFileName(llvm::StringRef DestinationDir,
                                            llvm::StringRef MainSourceFile,

Modified: clang-tools-extra/trunk/clang-modernize/Core/Transform.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/Core/Transform.h?rev=248080&r1=248079&r2=248080&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-modernize/Core/Transform.h (original)
+++ clang-tools-extra/trunk/clang-modernize/Core/Transform.h Fri Sep 18 21:21:28 2015
@@ -95,7 +95,8 @@ public:
   /// \p SourcePaths.
   /// \param[in] SourcePaths list of sources to transform.
   ///
-  /// \returns \li 0 if successful
+  /// \returns \parblock
+  ///          \li 0 if successful
   ///          \li 1 otherwise
   virtual int apply(const clang::tooling::CompilationDatabase &Database,
                     const std::vector<std::string> &SourcePaths) = 0;
@@ -171,7 +172,8 @@ public:
 
   /// \brief Add a Replacement to the list for the current translation unit.
   ///
-  /// \returns \li true on success
+  /// \returns \parblock
+  ///          \li true on success
   ///          \li false if there is no current translation unit
   bool addReplacementForCurrentTU(const clang::tooling::Replacement &R);
 




More information about the cfe-commits mailing list