[Rcpp-commits] r3199 - pkg/RcppEigen/inst/include/unsupported/Eigen/src/SparseExtra

noreply at r-forge.r-project.org noreply at r-forge.r-project.org
Fri Sep 16 20:55:33 CEST 2011


Author: dmbates
Date: 2011-09-16 20:55:32 +0200 (Fri, 16 Sep 2011)
New Revision: 3199

Modified:
   pkg/RcppEigen/inst/include/unsupported/Eigen/src/SparseExtra/CholmodSupport.h
Log:
Avoid compiler warnings about comparing signed and unsigned values.


Modified: pkg/RcppEigen/inst/include/unsupported/Eigen/src/SparseExtra/CholmodSupport.h
===================================================================
--- pkg/RcppEigen/inst/include/unsupported/Eigen/src/SparseExtra/CholmodSupport.h	2011-09-13 18:17:20 UTC (rev 3198)
+++ pkg/RcppEigen/inst/include/unsupported/Eigen/src/SparseExtra/CholmodSupport.h	2011-09-16 18:55:32 UTC (rev 3199)
@@ -235,8 +235,7 @@
 	OtherDerived& other = _other.const_cast_derived();
 	eigen_assert(m_factorizationIsOk && 
 		     "The decomposition is not in a valid state for solving, you must first call either compute() or symbolic()/numeric()");
-	const Index size = m_cholmodFactor->n;
-	eigen_assert(size==other.rows());
+	eigen_assert((Index)(m_cholmodFactor->n) == other.rows());
 	    
 	// note: cd stands for Cholmod Dense
 	cholmod_dense b_cd = viewAsCholmod(other.const_cast_derived());
@@ -394,8 +393,7 @@
 	void _solve(const MatrixBase<Rhs> &b, MatrixBase<Dest> &dest) const
 	{
 	    eigen_assert(m_factorizationIsOk && "The decomposition is not in a valid state for solving, you must first call either compute() or symbolic()/numeric()");
-	    const Index size = m_cholmodFactor->n;
-	    eigen_assert(size==b.rows());
+	    eigen_assert((Index)(m_cholmodFactor->n) == b.rows());
 	    
 	    // note: cd stands for Cholmod Dense
 	    cholmod_dense b_cd = viewAsCholmod(b.const_cast_derived());
@@ -417,8 +415,7 @@
 	void _solve(const SparseMatrix<RhsScalar,RhsOptions,RhsIndex> &b, SparseMatrix<DestScalar,DestOptions,DestIndex> &dest) const
 	{
 	    eigen_assert(m_factorizationIsOk && "The decomposition is not in a valid state for solving, you must first call either compute() or symbolic()/numeric()");
-	    const Index size = m_cholmodFactor->n;
-	    eigen_assert(size==b.rows());
+	    eigen_assert((Index)(m_cholmodFactor->n) == b.rows());
 	    
 	    // note: cs stands for Cholmod Sparse
 	    cholmod_sparse b_cs = viewAsCholmod(b);



More information about the Rcpp-commits mailing list