[eigen] Re: special matrix classes: template parameters

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]

further remarks:
 - this reallows nice simplifications in the code (in ei_traits and in typedefs)
 - while i'm at it, i'll let the toDense() methods use a ReturnByValue
 - and I'll rename ReturnByValue to ReturnValue since that's what it is

i put that on the TODO and i take care of it now.


2009/11/14 Benoit Jacob <jacob.benoit.1@xxxxxxxxx>:
> (and the same applies to DiagonalMatrix, of course, and to
> PermutationMatrix that i am coding at the moment)
> 2009/11/14 Benoit Jacob <jacob.benoit.1@xxxxxxxxx>:
>> Hi,
>> Currently the BandMatrix class takes Scalar, Rows,Cols template
>> parametes that could be replaced by a single parameter DenseMatrixType
>> representing the return value of toDense(). Not only this would allow
>> to simplify the template parameters considerably, this would also be
>> more future proof (it could read more data from the DenseMatrixType),
>> and would allow to benefit from the MaxRowsAtCompileTime notion (for
>> dynamic-size-inside-fixed-size-storage --- that is required to ensure
>> "no malloc with fixed-size objects" in complicated use cases).
>> So OK to let it take a DenseMatrixType template parameter instead? And
>> in the Options, the RowMajor stuff would also move there, so that's a
>> further simplification.
>> Benoit

Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/