Testing requested for pixman 0.16.1

Soeren Sandmann sandmann at daimi.au.dk
Mon Sep 21 06:26:48 PDT 2009


Hi,

> How about this one in pixman-trap.c _pixman_edge_multi_init()
> 
> Shouldn't the type of nx be 'pixman_fixed_48_16_t' not 'int'?

As I read the code, it is dividing two pixman_fixed_48_16 values and
the result of that is an integer and not a fixed point value.

If we were actually using the 48 bits of the 48.16 types, we would
need a 48 bit integer, but we only use the 48.16 types for 32.16
types.


Thanks,
Soren


More information about the xorg-devel mailing list