X server 1.6_beta1 pending pixman

John Tapsell johnflux at gmail.com
Wed Nov 26 04:19:41 PST 2008

2008/11/25 Soeren Sandmann <sandmann at daimi.au.dk>:
> Keith Packard <keithp at keithp.com> writes:
>> So, we'll see if we can't get a bit of pixman review and perhaps a
>> pixman release done tomorrow so that the X server beta can head out.
> Here are some comments on the matrix code. I didn't review all the
> numericals, but nothing jumped out at me either.
> - I'd like to have the interface const correct, for example in
>        pixman_transform_multily (struct pixman_transform_t *dst,
>                                  struct pixman_transform_t *l,
>                                  struct pixman_transform_t *r);
>  l and r could be const
> - For the rotation interfaces, maybe expand the names s and c to sin
>  and cos? I first thought c meant center and was then mystified what
>  s could mean.
> - The interfaces that take forward/reverse matrices should probably
>  accept NULL's.
> - Pixman's version numbering scheme is similar to cairo's: The git
>  master version has an odd micro number, released versions have even
>  micro numbers.
> - The name pixman_f_transform bothers me, but I don't have a better
>  suggestion since pixman_transformf would be worse.

How about   pixman_fourier_transform   ?  Why abbreviate it?


More information about the xorg mailing list