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

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Mon Jan 12 11:17:39 EST 2015


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

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/22bbb89eae5ef8f08b61539865a0beeabe345b34...3ad083bcbf243915de035eead1b0772ac3f35dee

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

commit 56f3554788b670bebb2a8a738fb704c13a5addde
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.


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

56f3554788b670bebb2a8a738fb704c13a5addde
 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