[mlpack-git] master: Comment out parameter name instead of (void)ing it. Just a little style fix. (78f1cfe)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Thu Mar 5 22:11:48 EST 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/904762495c039e345beba14c1142fd719b3bd50e...f94823c800ad6f7266995c700b1b630d5ffdcf40

>---------------------------------------------------------------

commit 78f1cfeddd0a59decccbe4fe3ef71581fd16fad5
Author: Ryan Curtin <ryan at ratml.org>
Date:   Mon Jan 12 11:04:05 2015 -0500

    Comment out parameter name instead of (void)ing it.
    Just a little style fix.


>---------------------------------------------------------------

78f1cfeddd0a59decccbe4fe3ef71581fd16fad5
 src/mlpack/methods/cf/cf_impl.hpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/mlpack/methods/cf/cf_impl.hpp b/src/mlpack/methods/cf/cf_impl.hpp
index e645b4c..c21822f 100644
--- a/src/mlpack/methods/cf/cf_impl.hpp
+++ b/src/mlpack/methods/cf/cf_impl.hpp
@@ -39,7 +39,7 @@ void ApplyFactorizer(arma::mat& /* data */,
  */
 template<typename FactorizerType>
 void ApplyFactorizer(arma::mat& data,
-    arma::sp_mat& cleanedData,
+    arma::sp_mat& /* cleanedData */,
     FactorizerType& factorizer,
     const size_t rank,
     arma::mat& w,
@@ -48,7 +48,6 @@ void ApplyFactorizer(arma::mat& data,
         FactorizerTraits<FactorizerType>::UsesCoordinateList == true,
         int*>::type = 0)
 {
-  (void)cleanedData;
   factorizer.Apply(data, rank, w, h);
 }
 



More information about the mlpack-git mailing list