[Piglit] [PATCH] outerproduct_tests: Add positional argument specifiers.

Jamey Sharp jamey at minilop.net
Wed May 14 21:41:47 PDT 2014


I can't merge Piglit patches, but for whatever it's worth this is
obviously correct, so:

Reviewed-by: Jamey Sharp <jamey at minilop.net>

On Wed, May 14, 2014 at 05:13:27PM -0700, Vinson Lee wrote:
> Fix build errors with Python < 2.7.
> 
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> Reviewed-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>  generated_tests/gen_outerproduct_tests.py |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/generated_tests/gen_outerproduct_tests.py b/generated_tests/gen_outerproduct_tests.py
> index 38bf834..fdc3810 100644
> --- a/generated_tests/gen_outerproduct_tests.py
> +++ b/generated_tests/gen_outerproduct_tests.py
> @@ -46,13 +46,13 @@ def main():
>  
>      for c, r in itertools.product(xrange(2, 5), repeat=2):
>          vecs = [
> -            Parameters(c, r, 'vec', 'mat{}x{}'.format(r, c)),
> -            Parameters(c, r, 'ivec', 'mat{}x{}'.format(r, c))
> +            Parameters(c, r, 'vec', 'mat{0}x{1}'.format(r, c)),
> +            Parameters(c, r, 'ivec', 'mat{0}x{1}'.format(r, c))
>          ]
>          if r == c:
>              vecs.extend([
> -                Parameters(c, r, 'vec', 'mat{}'.format(r)),
> -                Parameters(c, r, 'ivec', 'mat{}'.format(r))
> +                Parameters(c, r, 'vec', 'mat{0}'.format(r)),
> +                Parameters(c, r, 'ivec', 'mat{0}'.format(r))
>              ])
>  
>          for shader in ['vs', 'fs']:
> -- 
> 1.7.1
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140514/77d04936/attachment.sig>


More information about the Piglit mailing list