[Piglit] [PATCH 0/5] Consolidate CLAMP, MIN2, MAX2, and ARRAY_SIZE macros.

Kenneth Graunke kenneth at whitecape.org
Thu Jan 26 11:23:09 PST 2012


On 01/25/2012 05:56 PM, Paul Berry wrote:
> Many .c files in Piglit define some subset of the macros CLAMP, MIN2,
> MAX2, and ELEMENTS/ARRAY_SIZE, occasionally with variations in name
> and/or calling convention.  This patch standardizes all of the macros
> and consolidates the definitions into one place, piglit-util.h.
>
> Exception: I didn't touch the macros in Glean, since Glean is
> independent of piglit-util.
>
> No change in Piglit results on Sandy Bridge.
>
> [PATCH 1/5] Standardize argument order for CLAMP.
> [PATCH 2/5] Rename MAX to MAX2 in texunits.c.
> [PATCH 3/5] Make readtex.c include piglit-util.h.
> [PATCH 4/5] Move CLAMP, MIN2, and MAX2 macros to piglit-util.h.
> [PATCH 5/5] Change ELEMENTS macros to ARRAY_SIZE.

For the series:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>



More information about the Piglit mailing list