On Tue, 14 Oct 2014, Ben Avison <bavi...@riscosopen.org> wrote:
On Sun, 05 Oct 2014, Søren Sandmann <soren.sandm...@gmail.com> wrote: Comments on the patch series below. In the future, please consider sending smaller patch sets so that I can review them in smaller chunks of time.
[...]
Perhaps the thing to do is to thrash out each small group of related patches one at a time.
[...]
I have updated these two patches locally and will repost when I hear your preference about the use of macros in operator_name() and format_name().
Let's have another go at unblocking these patches. I'll probably be doing some work on ARMv7 soon (Raspberry Pi has finally moved on from a 12-year- old CPU design!) and I'd rather not keep diverging further and further from upstream. I will shortly repost the first sub-group of patches, hopefully having taken care of your concerns. Patch 3 takes the macro trick I used in the (now removed) operator/format parser part of affine-bench.c and applies it to the version from check-format.c/utils.c. Hopefully these will be considered OK and I'll move on to the next group of patches. Ben _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/pixman
