[Piglit] [PATCH 7/7] astc_3d: Move slice 'for' loop further down
Nanley Chery
nanleychery at gmail.com
Mon Aug 1 22:38:31 UTC 2016
On Mon, Aug 01, 2016 at 03:16:20PM -0700, Anuj Phogat wrote:
> Cc: Nanley Chery <nanley.g.chery at intel.com>
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
> .../oes_compressed_astc-miptree-3d.c | 35 +++++++++++-----------
> 1 file changed, 17 insertions(+), 18 deletions(-)
>
> diff --git a/tests/spec/oes_texture_compression_astc/oes_compressed_astc-miptree-3d.c b/tests/spec/oes_texture_compression_astc/oes_compressed_astc-miptree-3d.c
> index d743f3c..0219e52 100644
> --- a/tests/spec/oes_texture_compression_astc/oes_compressed_astc-miptree-3d.c
> +++ b/tests/spec/oes_texture_compression_astc/oes_compressed_astc-miptree-3d.c
> @@ -212,10 +212,6 @@ bool draw_compare_levels(bool check_error,
> y += h;
> }
>
> - /* Delete bound textures */
> - glDeleteTextures(1, &compressed_tex);
> - glDeleteTextures(1, &decompressed_tex);
> -
Please see my comment on patch 4/7.
- Nanley
> piglit_present_results();
> return result;
> }
> @@ -265,22 +261,21 @@ test_miptrees(void* input_type)
> piglit_set_tolerance_for_bits(8, 8, 8, 8);
>
> for ( ; block_dims < ARRAY_SIZE(block_dim_str); ++block_dims) {
> - for (slice = 0; slice < LEVEL0_DEPTH; slice++) {
> -
> - /* Texture objects. */
> - GLuint tex_compressed = 0;
> - GLuint tex_decompressed = 0;
> -
> - /* Load texture for current submode and block size */
> - load_texture("compressed/3D", tests[subtest],
> + /* Texture objects. */
> + GLuint tex_compressed = 0;
> + GLuint tex_decompressed = 0;
> +
> + /* Load texture for current submode and block size */
> + load_texture("compressed/3D", tests[subtest],
> + block_dim_str[block_dims],
> + &tex_compressed);
> + if (!check_error) {
> + load_texture("decompressed/3D", tests[subtest],
> block_dim_str[block_dims],
> - &tex_compressed);
> - if (!check_error) {
> - load_texture("decompressed/3D", tests[subtest],
> - block_dim_str[block_dims],
> - &tex_decompressed);
> - }
> + &tex_decompressed);
> + }
>
> + for (slice = 0; slice < LEVEL0_DEPTH; slice++) {
> /* Draw and compare each level of the two textures */
> glClear(GL_COLOR_BUFFER_BIT);
> if (!draw_compare_levels(check_error,
> @@ -296,6 +291,10 @@ test_miptrees(void* input_type)
> return PIGLIT_FAIL;
> }
> }
> + /* Delete bound textures */
> + glDeleteTextures(1, &tex_compressed);
> + if (!check_error)
> + glDeleteTextures(1, &tex_decompressed);
> }
> return PIGLIT_PASS;
> }
> --
> 2.5.5
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
More information about the Piglit
mailing list