commit 0b2412df5081e54fe2ff67a51ab24a097e36ce91
author Gael Guennebaud <g.gael@free.fr> Tue Sep 01 22:31:30 2015 +0200
committer Gael Guennebaud <g.gael@free.fr> Tue Sep 01 22:31:30 2015 +0200
tree 0610fd61cedb9a60879ef914167cbef269a83b1e
parent 9001f4a46b62344d9d0e589de1ad0e7b3413b479

Remove duplicated temporary in Sparse to Sparse assignment

Eigen/src/SparseCore/SparseAssign.h

diff --git a/Eigen/src/SparseCore/SparseAssign.h b/Eigen/src/SparseCore/SparseAssign.h
index 1ce60f3..9e8a31a 100644
--- a/Eigen/src/SparseCore/SparseAssign.h
+++ b/Eigen/src/SparseCore/SparseAssign.h
@@ -33,8 +33,9 @@
 template<typename OtherDerived>
 inline Derived& SparseMatrixBase<Derived>::operator=(const SparseMatrixBase<OtherDerived>& other)
 {
-  // FIXME, by default sparse evaluation do not alias, so we should be able to bypass the generic call_assignment
-  internal::call_assignment/*_no_alias*/(derived(), other.derived());
+  // by default sparse evaluation do not alias, so we can safely bypass the generic call_assignment routine
+  internal::Assignment<Derived,OtherDerived,internal::assign_op<Scalar> >
+          ::run(derived(), other.derived(), internal::assign_op<Scalar>());
   return derived();
 }
 
@@ -68,8 +69,6 @@
 template<typename DstXprType, typename SrcXprType>
 void assign_sparse_to_sparse(DstXprType &dst, const SrcXprType &src)
 {
-  eigen_assert(dst.rows() == src.rows() && dst.cols() == src.cols());
-  
   typedef typename DstXprType::Scalar Scalar;
   typedef typename internal::evaluator<DstXprType>::type DstEvaluatorType;
   typedef typename internal::evaluator<SrcXprType>::type SrcEvaluatorType;
@@ -129,8 +128,6 @@
 {
   static void run(DstXprType &dst, const SrcXprType &src, const internal::assign_op<typename DstXprType::Scalar> &/*func*/)
   {
-    eigen_assert(dst.rows() == src.rows() && dst.cols() == src.cols());
-    
     assign_sparse_to_sparse(dst.derived(), src.derived());
   }
 };