Nemanja Lukic <[email protected]> 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 > better?
I definitely think it would be better to have these tests available always rather than just as a compile time option. Søren _______________________________________________ Pixman mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pixman
