Bugzilla – Bug 207
possible indexing error in ei_transform_right_product_impl
Last modified: 2011-09-07 15:14:03 UTC
Created attachment 111 [details]
patch to Transform.h
It uses rows() as an index. There's a comment I don't understand just above the line in question so I'm not quite sure what's going on, but with no -1 I get runtime errors.
Created attachment 112 [details]
a fix for transform * MatrixBase product
it seems to me that this product suffered from more serious issues, like neglecting the homogeneous component of the right hand side. Here is what I think is a more correct patch but it needs confirmation because I might have overseen something.
I don't really have time to do this review now, as I'm not myself very familiar with this part of the code. Could you just add tests?
fixed by rev 31cc13f677d3