On 6 August 2013 21:33, Matt Turner <[email protected]> wrote:

> ---
>  generated_tests/gen_texture_query_lod_tests.py | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>

A quick note in the commit message explaining the rationale for this patch
would be nice.  (I assume you're doing this because without it, the tests
that get generated for GLSL 4.00 don't have anything in their pathname to
identify them as textureQueryLod tests).

In any case, the series is:

Reviewed-by: Paul Berry <[email protected]>


>
> diff --git a/generated_tests/gen_texture_query_lod_tests.py
> b/generated_tests/gen_texture_query_lod_tests.py
> index 1341f7d..be30a69 100755
> --- a/generated_tests/gen_texture_query_lod_tests.py
> +++ b/generated_tests/gen_texture_query_lod_tests.py
> @@ -112,6 +112,8 @@ template = Template(dedent("""\
>  """))
>
>  for api, requirement in requirements.iteritems():
> +    Lod = 'Lod' if api == 'glsl-4.00' else 'LOD'
> +
>      for sampler_type, coord_type in
> sampler_type_to_coord_type.iteritems():
>          for execution_stage in ("vs", "fs"):
>              file_extension = 'frag' if execution_stage == 'fs' else 'vert'
> @@ -119,8 +121,9 @@ for api, requirement in requirements.iteritems():
>                                      api,
>                                      "compiler",
>                                      "built-in-functions",
> -                                    "{0}.{1}".format(sampler_type,
> -                                                     file_extension))
> +                                    "textureQuery{0}-{1}.{2}".format(Lod,
> +
> sampler_type,
> +
> file_extension))
>              print filename
>
>              dirname = os.path.dirname(filename)
> @@ -137,8 +140,6 @@ for api, requirement in requirements.iteritems():
>                      'usamplerCubeArray', 'samplerCubeArrayShadow']:
>                  extensions += ['GL_ARB_texture_cube_map_array']
>
> -            Lod = 'Lod' if api == 'glsl-4.00' else 'LOD'
> -
>              f = open(filename, "w")
>              f.write(template.render(version=version,
>                                      extensions=extensions,
> --
> 1.8.1.5
>
> _______________________________________________
> Piglit mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/piglit
>
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to