Re: [Mesa-dev] [PATCH 01/14] mesa/main: replace Elements() with ARRAY_SIZE()

2015-02-28 Thread Brian Paul
On 02/28/2015 10:32 AM, Ilia Mirkin wrote: I pulled down the patches from patchwork and verified using tools that the only changes (except for the last patch) are Elements -> ARRAY_SIZE. This is the command I used: diff -u <( git diff e8cf8d013fcd7a15e94103b9367d89394de7a067^.. | grep -P '^\+[^+

Re: [Mesa-dev] [PATCH 01/14] mesa/main: replace Elements() with ARRAY_SIZE()

2015-02-28 Thread Ilia Mirkin
I pulled down the patches from patchwork and verified using tools that the only changes (except for the last patch) are Elements -> ARRAY_SIZE. This is the command I used: diff -u <( git diff e8cf8d013fcd7a15e94103b9367d89394de7a067^.. | grep -P '^\+[^+]' | sed 's/^+//' ) <( git diff e8cf8d013fcd7

[Mesa-dev] [PATCH 01/14] mesa/main: replace Elements() with ARRAY_SIZE()

2015-02-28 Thread Brian Paul
We've been using a mix of these two macros for a while now. Let's just use the later everywhere. It seems to be the convention used by other open-source projects. --- src/mesa/main/arrayobj.c | 6 +++--- src/mesa/main/attrib.c| 2 +- src/mesa/main/blend.c | 2