Re: FW: [Fwd: Bug in NSAffineTransform]

[Adam Fedor]

On 2006-02-23 13:11:00 -0700 "Vaisburd, Haim" <HVaisbur@Advent.COM> 
wrote:

> The matrix multiplication is not commutative and the last line should 
> be
> instead
>   matrix = matrix_multiply(matrix, tranm);

Do you have a test that shows this is wrong? All my tests work with 
the current version (as well as give the same result as Mac OSX).


> I prefer the old version though:
>
> - (void) translateXBy: (float)tranX  yBy: (float)tranY
> {
>   TX += tranX;
>   TY += tranY;
> }
>
> it's clearer and faster.

But incorrect. Remember we are translating a coordinate system, not a 
point.