Re: [eigen] Qt's container support

[ Thread Index | Date Index | More Archives ]

On Tuesday 20 January 2009, Gael Guennebaud wrote:
> Issue3:
> The last issue I found is with QVector::fill(const T&,size_t) which is
> the equivalent of std::vector<T>::resize(size_t,T). Here the argument
> is passed by reference so no alignment issue. The problem is the use
> of the default ctor and operator= to initialize/copy the new elements
> while our operator= does not allow to set an uninitialized matrix.
> Solutions:
> a) allow operator= to set/resize an uninitialized matrix
> b) specialize QVector for Eigen's matrix type using a similar trick
> than for std::vector such that we only have to redefine the fill()
> method to use Matrix::set()
> c) do nothing and enforce the user to use .resize() followed by a for
> loop instead of fill().
> I vote for solution b)
> any opinion ?
b) isn't a trick, it's the C++ solution for this kind of problem :) ( c) is not an acceptable solution ).

Cyrille Berger

Mail converted by MHonArc 2.6.19+