diff options
| author | Kenneth Graunke <kenneth@whitecape.org> | 2014-02-02 02:48:03 -0800 |
|---|---|---|
| committer | Kenneth Graunke <kenneth@whitecape.org> | 2014-02-05 15:31:26 -0800 |
| commit | 8eb40d7e829500ff4e22d91745cc7c4f0f6e64d0 (patch) | |
| tree | a2a3ae971f150915f416a3bd45abfde63a086969 /tests/texturing | |
| parent | 8491b801bb4cd539c2992bc31c9b7fe5a51c2238 (diff) | |
max-texture-size: Cut SubImage data allocation by 1/4.
I honestly have no idea why, but the original test calls TexSubImage
with a width and height of *half* the texture size. Yet it allocates
enough temporary pixel data for a full texture. This makes no sense,
and clearly one or the other should change.
Without understanding why the existing test does what it does, I chose
to not change it and simply cut the memory allocation. Cutting memory
usage also may reduce the likelihood of GL_OUT_OF_MEMORY errors.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Diffstat (limited to 'tests/texturing')
| -rw-r--r-- | tests/texturing/max-texture-size.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/texturing/max-texture-size.c b/tests/texturing/max-texture-size.c index e0e16849f..60ebc159a 100644 --- a/tests/texturing/max-texture-size.c +++ b/tests/texturing/max-texture-size.c @@ -265,7 +265,7 @@ test_non_proxy_texture_size(GLenum target, GLenum internalformat) piglit_get_gl_enum_name(internalformat), maxSide); /* Allocate and initialize texture data array */ - pixels = initTexData(target, maxSide); + pixels = initTexData(target, maxSide/2); if (pixels == NULL) { printf("Error allocating texture data array for target %s, size %d\n", |