Re: [PATCH libXrender] Fix alpha premultiplication in XRenderParseColor.

2012-01-03 Thread Emanuele Giaquinta
On Tue, Jan 03, 2012 at 08:21:32AM -0500, Jeremy Huddleston wrote: I believe just the division is unsigned with those changes. The multiplication will still be signed. No, the type of XRenderColor fields is unsigned short. On Jan 2, 2012, at 2:58 PM, Emanuele Giaquinta wrote: Due to C

[PATCH libXrender] Fix alpha premultiplication in XRenderParseColor.

2012-01-02 Thread Emanuele Giaquinta
Due to C arithmetic conversion rules we must use an unsigned constant (or a cast) to perform the multiplication using unsigned arithmetic. --- src/Color.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Color.c b/src/Color.c index 9c76e58..23ef800 100644 ---