[cairo] PIXMAN_x8b8r8g8 not defined
Soeren Sandmann
sandmann at daimi.au.dk
Tue Mar 25 06:31:52 PDT 2008
"André Tupinambá" <andrelrt at gmail.com> writes:
> But, I found this comment in MMX (and C) fast-path table, and went I
> try to uncomment it a receive an error that PIXMAN_x8b8r8g8 not
> defined.
>
> #if 0
> /* FIXME: This code is commented out since it's apparently not
> actually faster than the generic code. */
> { PIXMAN_OP_OVER, PIXMAN_x8r8g8b8, PIXMAN_a8, PIXMAN_x8r8g8b8,
> fbCompositeOver_x888x8x8888mmx, 0 },
> { PIXMAN_OP_OVER, PIXMAN_x8r8g8b8, PIXMAN_a8, PIXMAN_a8r8g8b8,
> fbCompositeOver_x888x8x8888mmx, 0 },
> { PIXMAN_OP_OVER, PIXMAN_x8b8r8g8, PIXMAN_a8, PIXMAN_x8b8g8r8,
> fbCompositeOver_x888x8x8888mmx, 0 },
> { PIXMAN_OP_OVER, PIXMAN_x8b8r8g8, PIXMAN_a8, PIXMAN_a8r8g8b8,
> fbCompositeOver_x888x8x8888mmx, 0 },
> #endif
>
> What is wrong?
> Is It a typo and the correct is PIXMAN_x8b8g8r8 (b r g => b g r)? Or;
> Is It a deprecated constant? May I recreate it?
It is a typo. The constant is PIXMAN_x8b8r8g8.
> BTW, I'm running the cairo's make check to test the pixman. But I had
> some problems to test the fast-paths, not all paths are called. There
> are another way to test the pixman library?
Which paths are not being hit? Some of the pixman fast paths were
created back when I experimented with an X render based compositor. If
we are only talking about a few paths, maybe we can come up with a few
simple tests for them.
Eventually I'd like to have a pixman regression test suite
independently of the cairo one.
Soren
More information about the cairo
mailing list