[2/2] gdiplus: fix a warning in GdipMultiplyTextureTransform

Nikolay Sivov bunglehead at gmail.com
Wed Dec 3 11:11:24 CST 2008


Henri Verbeet wrote:
> 2008/12/3 Nikolay Sivov <bunglehead at gmail.com>:
>   
>> -    return GdipMultiplyMatrix(brush->transform, matrix, order);
>> +    return GdipMultiplyMatrix(brush->transform, (GpMatrix*)matrix, order);
>>     
>
> I don't think casting const away makes it much better. Considering
> what the function does, and the fact that the matrix *is* const in the
> prototype for Matrix::Multiply(), I'd say it would be better to just
> change the prototype for GdipMultiplyMatrix. MS doesn't seem to care a
> whole lot about their plain C interfaces these days.
>   
Our prototypes should match MS', shouldn't they?. In this case I don't 
see another way to silent this warning
which means nothing (you've already named the reason why).
> Slightly OT, the code in matrix_multiply() isn't quite the most
> efficient way to multiply two 2x3 matrices, although if you're lucky
> the compiler will unroll the loop for you and fold the constants.
>   
Of course you're right here.



More information about the wine-devel mailing list