[Pixman] [PATCH 9/9] test: add "pixbuf" and "rpixbuf" to lowlevel-blt-bench
sandmann at cs.au.dk
Wed Mar 27 00:43:07 PDT 2013
Nemanja Lukic <nemanja.lukic at rt-rk.com> writes:
> Hi Soren,
>> - The commit adding support to lowlevel-blt-bench should come before the
>> commit that adds the fast path. That makes it easy to get before and
>> after numbers.
> I'll reorder commits. Since commit that adds support to lowlevel-blt benchmark
> only touches lowlevel-blt-bench.c file, that is only up to reordering of the pushes.
>> - Why are the pixbuf test cases only available as a compile time option?
> I decided to add them as a compile time option, since they don't cover big range
> of color formats (actually only couple) and operations. But, it can be implemented
> in different way. bench_composite function can check for pixbuf string in testname,
> and if that is detected, use same bits for src and mask images. Than, pixbuf testcases
> will not be only compile time option. Do you think that approach is
I definitely think it would be better to have these tests available
always rather than just as a compile time option.
More information about the Pixman