Re: [Piglit] [PATCH] ARB_explicit_attrib_location: Remove commas from require_extensions line.

2014-06-24 Thread Dylan Baker
On Tuesday, June 24, 2014 02:23:23 PM Matt Turner wrote:
> ---
>  tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag | 2 +-
> tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git
> a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> index 541e075..2b187f2 100644
> --- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> +++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> @@ -1,7 +1,7 @@
>  // [config]
>  // expect_result: pass
>  // glsl_version: 1.10
> -// require_extensions: GL_ARB_explicit_attrib_location,
> GL_ARB_blend_func_extended +// require_extensions:
> GL_ARB_explicit_attrib_location GL_ARB_blend_func_extended // [end config]
>  //
>  // Even though the specified locations overlap, the spec says that a *link*
> diff --git
> a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> index 246d45f..2be0403 100644
> --- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> +++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> @@ -1,7 +1,7 @@
>  // [config]
>  // expect_result: pass
>  // glsl_version: 1.10
> -// require_extensions: GL_ARB_explicit_attrib_location,
> GL_ARB_blend_func_extended +// require_extensions:
> GL_ARB_explicit_attrib_location GL_ARB_blend_func_extended // [end config]
>  //
>  // Even though the specified locations overlap, the spec says that a *link*

Reviewed-by: Dylan Baker 

signature.asc
Description: This is a digitally signed message part.
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH] ARB_explicit_attrib_location: Remove commas from require_extensions line.

2014-06-24 Thread Chris Forbes
Reviewed-by: Chris Forbes 

On Wed, Jun 25, 2014 at 9:23 AM, Matt Turner  wrote:
> ---
>  tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag | 2 +-
>  tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git 
> a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag 
> b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> index 541e075..2b187f2 100644
> --- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> +++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
> @@ -1,7 +1,7 @@
>  // [config]
>  // expect_result: pass
>  // glsl_version: 1.10
> -// require_extensions: GL_ARB_explicit_attrib_location, 
> GL_ARB_blend_func_extended
> +// require_extensions: GL_ARB_explicit_attrib_location 
> GL_ARB_blend_func_extended
>  // [end config]
>  //
>  // Even though the specified locations overlap, the spec says that a *link*
> diff --git 
> a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag 
> b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> index 246d45f..2be0403 100644
> --- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> +++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
> @@ -1,7 +1,7 @@
>  // [config]
>  // expect_result: pass
>  // glsl_version: 1.10
> -// require_extensions: GL_ARB_explicit_attrib_location, 
> GL_ARB_blend_func_extended
> +// require_extensions: GL_ARB_explicit_attrib_location 
> GL_ARB_blend_func_extended
>  // [end config]
>  //
>  // Even though the specified locations overlap, the spec says that a *link*
> --
> 1.8.3.2
>
> ___
> Piglit mailing list
> Piglit@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit


[Piglit] [PATCH] ARB_explicit_attrib_location: Remove commas from require_extensions line.

2014-06-24 Thread Matt Turner
---
 tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag | 2 +-
 tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag 
b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
index 541e075..2b187f2 100644
--- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
+++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-12.frag
@@ -1,7 +1,7 @@
 // [config]
 // expect_result: pass
 // glsl_version: 1.10
-// require_extensions: GL_ARB_explicit_attrib_location, 
GL_ARB_blend_func_extended
+// require_extensions: GL_ARB_explicit_attrib_location 
GL_ARB_blend_func_extended
 // [end config]
 //
 // Even though the specified locations overlap, the spec says that a *link*
diff --git 
a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag 
b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
index 246d45f..2be0403 100644
--- a/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
+++ b/tests/spec/arb_explicit_attrib_location/1.10/compiler/layout-13.frag
@@ -1,7 +1,7 @@
 // [config]
 // expect_result: pass
 // glsl_version: 1.10
-// require_extensions: GL_ARB_explicit_attrib_location, 
GL_ARB_blend_func_extended
+// require_extensions: GL_ARB_explicit_attrib_location 
GL_ARB_blend_func_extended
 // [end config]
 //
 // Even though the specified locations overlap, the spec says that a *link*
-- 
1.8.3.2

___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit