[Cmake-commits] CMake branch, next, updated. v2.8.12-3629-g6b4315d

Brad King brad.king at kitware.com
Tue Oct 8 09:20:49 EDT 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, next has been updated
       via  6b4315d8a858cf0dca870b236dbe5dbb50ec4b8d (commit)
       via  af0051f48ac54ff749faff885f05fb9f34037d5a (commit)
       via  cb4ec606a9e13ed4064fd8f72f00e51ec09ea537 (commit)
       via  d915819ad1a6c0dafb38dc12c1e1329d8130403f (commit)
       via  97124f91a5eaa5b1bf962813e72b1b2406960a76 (commit)
       via  8e96353fd85cff5f06bda071f3a11397e3faa757 (commit)
       via  d99303220922142c2d3fdce5346d0997e4f3b457 (commit)
       via  6cea3eeb4b785fa4b72d4a26c9b6df09180ae34c (commit)
       via  7e7a1c0a50e9ec589866ea4ebd2d27c1c8424d8f (commit)
      from  f99823600983f9111ceb8a62ba3280da012f5324 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=6b4315d8a858cf0dca870b236dbe5dbb50ec4b8d
commit 6b4315d8a858cf0dca870b236dbe5dbb50ec4b8d
Merge: f998236 af0051f
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Tue Oct 8 09:20:45 2013 -0400
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Tue Oct 8 09:20:45 2013 -0400

    Merge topic 'clang-warnings' into next
    
    af0051f curl, bzip2: Suppress warnings by setting initial value
    cb4ec60 cmDocumentation: Fix a few doxygen comment errors
    d915819 cm*Lexer: Populate empty doxygen @param comment
    97124f9 cmTarget: Properly escape @ char in doxygen comments
    8e96353 Fix warnings about \brief usage
    d993032 cmGraphVizWriter: Add extra space in comment to suppress warning
    6cea3ee cmNewLineStyle: Remove useless semi-colon
    7e7a1c0 curl: Fix typo in header include guard


http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=af0051f48ac54ff749faff885f05fb9f34037d5a
commit af0051f48ac54ff749faff885f05fb9f34037d5a
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 15:07:51 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:33 2013 -0400

    curl, bzip2: Suppress warnings by setting initial value
    
    Silence clang -Wsometimes-uninitialized warnings.

diff --git a/Utilities/cmbzip2/compress.c b/Utilities/cmbzip2/compress.c
index 7d9b3da..feea233 100644
--- a/Utilities/cmbzip2/compress.c
+++ b/Utilities/cmbzip2/compress.c
@@ -239,7 +239,7 @@ static
 void sendMTFValues ( EState* s )
 {
    Int32 v, t, i, j, gs, ge, totc, bt, bc, iter;
-   Int32 nSelectors, alphaSize, minLen, maxLen, selCtr;
+   Int32 nSelectors = 0, alphaSize, minLen, maxLen, selCtr;
    Int32 nGroups, nBytes;
 
    /*--
diff --git a/Utilities/cmcurl/multi.c b/Utilities/cmcurl/multi.c
index b501f29..869d568 100644
--- a/Utilities/cmcurl/multi.c
+++ b/Utilities/cmcurl/multi.c
@@ -763,7 +763,7 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
   struct Curl_message *msg = NULL;
   bool connected;
   bool async;
-  bool protocol_connect;
+  bool protocol_connect = 0;
   bool dophase_done;
   bool done;
   CURLMcode result = CURLM_OK;

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=cb4ec606a9e13ed4064fd8f72f00e51ec09ea537
commit cb4ec606a9e13ed4064fd8f72f00e51ec09ea537
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:08:38 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:32 2013 -0400

    cmDocumentation: Fix a few doxygen comment errors
    
    Warned by clang -Wdocumentation

diff --git a/Source/cmDocumentation.h b/Source/cmDocumentation.h
index e180f60..218f44d 100644
--- a/Source/cmDocumentation.h
+++ b/Source/cmDocumentation.h
@@ -154,11 +154,11 @@ public:
   /**
    * Retrieve the list of documented module located in
    * path which match the globing expression globExpr.
-   * @param[in] path, directory where to start the search
+   * @param[in] path directory where to start the search
    *                  we will recurse into it.
-   * @param[in] globExpr, the globing expression used to
+   * @param[in] globExpr the globing expression used to
    *                      match the file in path.
-   * @param[out] the list of obtained pairs (may be empty)
+   * @param[out] docModuleList the list of obtained pairs (may be empty)
    * @return 0 on success 1 on error or empty list
    */
   int getDocumentedModulesListInDir(
@@ -180,10 +180,9 @@ public:
    * @param[in] fname the script file name to be parsed for documentation
    * @param[in,out] commands the vector of command/macros documentation
    *                entry found in the script file.
-   * @param[in,out] the cmake object instance to which variable documentation
-   *                will be attached (using @see cmake::DefineProperty)
-   * @param[in] the documentation section in which the property will be
-   *            inserted.
+   * @param[in,out] cm the cmake object instance to which variable
+   *                documentation will be attached
+   *                (using @see cmake::DefineProperty)
    * @return the number of documented items (command and variable)
    *         found in the file.
    */

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d915819ad1a6c0dafb38dc12c1e1329d8130403f
commit d915819ad1a6c0dafb38dc12c1e1329d8130403f
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:05:17 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:32 2013 -0400

    cm*Lexer: Populate empty doxygen @param comment
    
    And in one case fixed wrong param name.
    Warned by clang -Wdocumentation.

diff --git a/Source/cmCommandArgumentLexer.cxx b/Source/cmCommandArgumentLexer.cxx
index e68f6b5..e62e53e 100644
--- a/Source/cmCommandArgumentLexer.cxx
+++ b/Source/cmCommandArgumentLexer.cxx
@@ -1820,7 +1820,7 @@ void cmCommandArgument_yyset_extra (YY_EXTRA_TYPE  user_defined , yyscan_t yysca
 }
 
 /** Set the current line number.
- * @param line_number
+ * @param line_number The line number to set.
  * @param yyscanner The scanner object.
  */
 void cmCommandArgument_yyset_lineno (int  line_number , yyscan_t yyscanner)
@@ -1835,7 +1835,7 @@ void cmCommandArgument_yyset_lineno (int  line_number , yyscan_t yyscanner)
 }
 
 /** Set the current column.
- * @param column_no
+ * @param column_no The column number to set.
  * @param yyscanner The scanner object.
  */
 void cmCommandArgument_yyset_column (int  column_no , yyscan_t yyscanner)
diff --git a/Source/cmDependsFortranLexer.cxx b/Source/cmDependsFortranLexer.cxx
index 924d9d2..1eff1e4 100644
--- a/Source/cmDependsFortranLexer.cxx
+++ b/Source/cmDependsFortranLexer.cxx
@@ -2165,7 +2165,7 @@ void cmDependsFortran_yyset_extra (YY_EXTRA_TYPE  user_defined , yyscan_t yyscan
 }
 
 /** Set the current line number.
- * @param line_number
+ * @param line_number The line number to set.
  * @param yyscanner The scanner object.
  */
 void cmDependsFortran_yyset_lineno (int  line_number , yyscan_t yyscanner)
@@ -2180,7 +2180,7 @@ void cmDependsFortran_yyset_lineno (int  line_number , yyscan_t yyscanner)
 }
 
 /** Set the current column.
- * @param line_number
+ * @param column_no The column number to set.
  * @param yyscanner The scanner object.
  */
 void cmDependsFortran_yyset_column (int  column_no , yyscan_t yyscanner)
diff --git a/Source/cmDependsJavaLexer.cxx b/Source/cmDependsJavaLexer.cxx
index 0af44b0..1e505a5 100644
--- a/Source/cmDependsJavaLexer.cxx
+++ b/Source/cmDependsJavaLexer.cxx
@@ -2330,7 +2330,7 @@ void cmDependsJava_yyset_extra (YY_EXTRA_TYPE  user_defined , yyscan_t yyscanner
 }
 
 /** Set the current line number.
- * @param line_number
+ * @param line_number The line number to set.
  * @param yyscanner The scanner object.
  */
 void cmDependsJava_yyset_lineno (int  line_number , yyscan_t yyscanner)
@@ -2345,7 +2345,7 @@ void cmDependsJava_yyset_lineno (int  line_number , yyscan_t yyscanner)
 }
 
 /** Set the current column.
- * @param column_no
+ * @param column_no The column number to set.
  * @param yyscanner The scanner object.
  */
 void cmDependsJava_yyset_column (int  column_no , yyscan_t yyscanner)
diff --git a/Source/cmExprLexer.cxx b/Source/cmExprLexer.cxx
index 9947c77..aa384cd 100644
--- a/Source/cmExprLexer.cxx
+++ b/Source/cmExprLexer.cxx
@@ -1716,7 +1716,7 @@ void cmExpr_yyset_extra (YY_EXTRA_TYPE  user_defined , yyscan_t yyscanner)
 }
 
 /** Set the current line number.
- * @param line_number
+ * @param line_number The line number to set.
  * @param yyscanner The scanner object.
  */
 void cmExpr_yyset_lineno (int  line_number , yyscan_t yyscanner)
@@ -1731,7 +1731,7 @@ void cmExpr_yyset_lineno (int  line_number , yyscan_t yyscanner)
 }
 
 /** Set the current column.
- * @param column_no
+ * @param column_no The column number to set.
  * @param yyscanner The scanner object.
  */
 void cmExpr_yyset_column (int  column_no , yyscan_t yyscanner)

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=97124f91a5eaa5b1bf962813e72b1b2406960a76
commit 97124f91a5eaa5b1bf962813e72b1b2406960a76
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:03:38 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:32 2013 -0400

    cmTarget: Properly escape @ char in doxygen comments
    
    Found by clang's -Wdocumentation.

diff --git a/Source/cmTarget.h b/Source/cmTarget.h
index 27b74ca..527cbc7 100644
--- a/Source/cmTarget.h
+++ b/Source/cmTarget.h
@@ -372,7 +372,7 @@ public:
   /** Get the soname of the target.  Allowed only for a shared library.  */
   std::string GetSOName(const char* config);
 
-  /** Whether this library has @rpath and platform supports it.  */
+  /** Whether this library has \@rpath and platform supports it.  */
   bool HasMacOSXRpath(const char* config);
 
   /** Test for special case of a third-party shared library that has
@@ -421,12 +421,12 @@ public:
   bool IsChrpathUsed(const char* config);
 
   /** Return the install name directory for the target in the
-    * build tree.  For example: "@rpath/", "@loader_path/",
+    * build tree.  For example: "\@rpath/", "\@loader_path/",
     * or "/full/path/to/library".  */
   std::string GetInstallNameDirForBuildTree(const char* config);
 
   /** Return the install name directory for the target in the
-    * install tree.  For example: "@rpath/" or "@loader_path/". */
+    * install tree.  For example: "\@rpath/" or "\@loader_path/". */
   std::string GetInstallNameDirForInstallTree();
 
   cmComputeLinkInformation* GetLinkInformation(const char* config,

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=8e96353fd85cff5f06bda071f3a11397e3faa757
commit 8e96353fd85cff5f06bda071f3a11397e3faa757
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:02:31 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:32 2013 -0400

    Fix warnings about \brief usage
    
    A doxygen \brief is ended with a newline, so remove a newline right
    after \brief to fix clang -Wdocumentation warning.

diff --git a/Source/cmAddCustomCommandCommand.h b/Source/cmAddCustomCommandCommand.h
index 1cc1e3a..2504185 100644
--- a/Source/cmAddCustomCommandCommand.h
+++ b/Source/cmAddCustomCommandCommand.h
@@ -16,9 +16,7 @@
 #include "cmDocumentGeneratorExpressions.h"
 
 /** \class cmAddCustomCommandCommand
- * \brief
- *
- *  cmAddCustomCommandCommand defines a new command (rule) that can
+ * \brief cmAddCustomCommandCommand defines a new command (rule) that can
  *  be executed within the build process
  *
  */
diff --git a/Source/cmCreateTestSourceList.h b/Source/cmCreateTestSourceList.h
index 3aa0a79..b9c6341 100644
--- a/Source/cmCreateTestSourceList.h
+++ b/Source/cmCreateTestSourceList.h
@@ -15,7 +15,7 @@
 #include "cmCommand.h"
 
 /** \class cmCreateTestSourceList
- * \brief
+ * \brief Test driver generation command
  *
  */
 
diff --git a/Source/cmIncludeCommand.h b/Source/cmIncludeCommand.h
index d97b7c3..8ff8b20 100644
--- a/Source/cmIncludeCommand.h
+++ b/Source/cmIncludeCommand.h
@@ -15,9 +15,7 @@
 #include "cmCommand.h"
 
 /** \class cmIncludeCommand
- * \brief
- *
- *  cmIncludeCommand defines a list of distant
+ * \brief cmIncludeCommand defines a list of distant
  *  files that can be "included" in the current list file.
  *  In almost every sense, this is identical to a C/C++
  *  #include command.  Arguments are first expended as usual.

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d99303220922142c2d3fdce5346d0997e4f3b457
commit d99303220922142c2d3fdce5346d0997e4f3b457
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:00:58 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:31 2013 -0400

    cmGraphVizWriter: Add extra space in comment to suppress warning
    
    Clang's -Wdocumentation thought it was malformed doxygen.

diff --git a/Source/cmGraphVizWriter.cxx b/Source/cmGraphVizWriter.cxx
index 4816da9..3a7070e 100644
--- a/Source/cmGraphVizWriter.cxx
+++ b/Source/cmGraphVizWriter.cxx
@@ -534,8 +534,8 @@ int cmGraphVizWriter::CollectAllExternalLibs(int cnt)
           ostr << this->GraphNodePrefix << cnt++;
           this->TargetNamesNodes[libName] = ostr.str();
           this->TargetPtrs[libName] = NULL;
-          //str << "    \"" << ostr.c_str() << "\" [ label=\"" << libName
-          //<<  "\" shape=\"ellipse\"];" << std::endl;
+          // str << "    \"" << ostr.c_str() << "\" [ label=\"" << libName
+          // <<  "\" shape=\"ellipse\"];" << std::endl;
           }
         }
       }

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=6cea3eeb4b785fa4b72d4a26c9b6df09180ae34c
commit 6cea3eeb4b785fa4b72d4a26c9b6df09180ae34c
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 13:00:07 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:31 2013 -0400

    cmNewLineStyle: Remove useless semi-colon
    
    Warned by clang.

diff --git a/Source/cmNewLineStyle.cxx b/Source/cmNewLineStyle.cxx
index 6f7b6a9..a7d7429 100644
--- a/Source/cmNewLineStyle.cxx
+++ b/Source/cmNewLineStyle.cxx
@@ -78,7 +78,7 @@ const std::string cmNewLineStyle::GetCharacters() const
       return "\r\n";
     default:
       ;
-    };
+    }
   return "";
 }
 

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7e7a1c0a50e9ec589866ea4ebd2d27c1c8424d8f
commit 7e7a1c0a50e9ec589866ea4ebd2d27c1c8424d8f
Author:     Sean McBride <sean at rogue-research.com>
AuthorDate: Thu Sep 19 12:59:27 2013 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Oct 8 09:16:27 2013 -0400

    curl: Fix typo in header include guard
    
    From clang's -Wheader-guard.

diff --git a/Utilities/cmcurl/parsedate.h b/Utilities/cmcurl/parsedate.h
index 0ea2d83..b29fe9b 100644
--- a/Utilities/cmcurl/parsedate.h
+++ b/Utilities/cmcurl/parsedate.h
@@ -1,5 +1,5 @@
 #ifndef __PARSEDATE_H
-#define __PARSEDATEL_H
+#define __PARSEDATE_H
 /***************************************************************************
  *                                  _   _ ____  _
  *  Project                     ___| | | |  _ \| |

-----------------------------------------------------------------------

Summary of changes:


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list