[Piglit] [PATCH] GL 3.2: Fix layered clear color test to work with MSVC.
Vinson Lee
vlee at freedesktop.org
Wed Sep 11 18:59:43 PDT 2013
On Tue, Sep 10, 2013 at 2:05 PM, Paul Berry <stereotype441 at gmail.com> wrote:
> MSVC doesn't support dynamically sized local arrays.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=69054
> ---
> tests/spec/gl-3.2/layered-rendering/clear-color.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/tests/spec/gl-3.2/layered-rendering/clear-color.c b/tests/spec/gl-3.2/layered-rendering/clear-color.c
> index c93ae4e..6d15f11 100644
> --- a/tests/spec/gl-3.2/layered-rendering/clear-color.c
> +++ b/tests/spec/gl-3.2/layered-rendering/clear-color.c
> @@ -109,7 +109,9 @@ piglit_init(int argc, char **argv)
> {
> int i, j;
> GLenum fbstatus;
> - float colorLayers[layers][piglit_width*piglit_height*3];
> + float *colorLayers =
> + malloc(sizeof(float) * layers *
> + piglit_width * piglit_height * 3);
> float colors[3][3] = {
> {0.0, 0.0, 1.0},
> {0.0, 1.0, 0.0},
> @@ -118,10 +120,12 @@ piglit_init(int argc, char **argv)
>
> /* Create color data for texture */
> for(j = 0; j < layers; j++) {
> + float *thisLayer =
> + &colorLayers[j * piglit_width * piglit_height * 3];
> for(i = 0; i < piglit_width*piglit_height; i++) {
> - colorLayers[j][i*3+0] = colors[j][0];
> - colorLayers[j][i*3+1] = colors[j][1];
> - colorLayers[j][i*3+2] = colors[j][2];
> + thisLayer[i*3+0] = colors[j][0];
> + thisLayer[i*3+1] = colors[j][1];
> + thisLayer[i*3+2] = colors[j][2];
> }
> }
>
> --
> 1.8.4
>
This patch fixes the MSVC build for me.
Tested-by: Vinson Lee <vlee at freedesktop.org>
More information about the Piglit
mailing list