Re: [eigen] Qt's container support

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


alright, so it seems this change brought more troubles than it solve.
eventually I'm ok to allow operator= to resize too.


On Tue, Jan 20, 2009 at 7:31 PM, Benoit Jacob <jacob.benoit.1@xxxxxxxxx> wrote:
> 2009/1/20 Gael Guennebaud <gael.guennebaud@xxxxxxxxx>:
>> this was the initial behavior, see this thread to see all the
>> arguments which lead to the addition of set():
>>
>> http://listengine.tuxfamily.org/lists.tuxfamily.org/eigen/2008/10/msg00040.html
>
> Rereading quickly through this thread I don't really see a good reason
> anymore. If you have a specific one in mind can you refresh my memory?
>
> Otherwise I'm tempted to think that the main reason for set() goes
> away once we accept to have this little if() in operator=.
> Another reason was that one of our users had trouble tracing a bug
> because operator= had resized this matrix; but now we see that the
> non-resizing behavior also leads to trouble.
>
> Cheers,
> Benoit
>
>
>



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