X server 1.6_beta1 pending pixman

Soeren Sandmann sandmann at daimi.au.dk
Tue Nov 25 08:58:50 PST 2008


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.

- Is there any particular reason for the fixed point epsilon of 2 (as
  opposed to 1)?

- There is a comment about floating point interfaces, but the file
  contains both fixed and floating point interfaces.



Thanks,
Soren



More information about the xorg mailing list