Re: [Piglit] [PATCH 44/50] gl-2.0: set KHR_no_error status

2017-08-22 Thread Samuel Pitoiset



On 08/22/2017 02:15 AM, Timothy Arceri wrote:



On 11/08/17 01:22, Samuel Pitoiset wrote:

Signed-off-by: Samuel Pitoiset 
---
  tests/spec/gl-2.0/api/active-sampler-conflict.c |  1 +
  tests/spec/gl-2.0/api/clip-flag-behavior.c  | 33 
++---


You missed:

getattriblocation-conventional.c
attrib-assignments.c

There are also some more that need to be updated in the parent dir:

tests/spec/gl-2.0/


Right, fixed!






  2 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/tests/spec/gl-2.0/api/active-sampler-conflict.c 
b/tests/spec/gl-2.0/api/active-sampler-conflict.c

index 74aed3689..848122ace 100644
--- a/tests/spec/gl-2.0/api/active-sampler-conflict.c
+++ b/tests/spec/gl-2.0/api/active-sampler-conflict.c
@@ -61,6 +61,7 @@
  PIGLIT_GL_TEST_CONFIG_BEGIN
  config.supports_gl_compat_version = 20;
+    config.khr_no_error_support = PIGLIT_HAS_ERRORS;
  PIGLIT_GL_TEST_CONFIG_END
diff --git a/tests/spec/gl-2.0/api/clip-flag-behavior.c 
b/tests/spec/gl-2.0/api/clip-flag-behavior.c

index 25d7b4efd..c0e031e97 100644
--- a/tests/spec/gl-2.0/api/clip-flag-behavior.c
+++ b/tests/spec/gl-2.0/api/clip-flag-behavior.c
@@ -42,6 +42,7 @@ PIGLIT_GL_TEST_CONFIG_BEGIN
  config.supports_gl_compat_version = 10;
  config.window_visual = PIGLIT_GL_VISUAL_RGB | 
PIGLIT_GL_VISUAL_DOUBLE;

+    config.khr_no_error_support = PIGLIT_NO_ERRORS;
  PIGLIT_GL_TEST_CONFIG_END
@@ -146,25 +147,27 @@ piglit_init(int argc, char **argv)
  pass = check_enable_state(enum_name, enum_value, false) && 
pass;

  }
-    /* Check behavior of GL_CLIP_PLANE0 + n where n == 
max_clip_planes */

-    enum_value = GL_CLIP_PLANE0 + max_clip_planes;
-    sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
+    if (!piglit_khr_no_error) {
+    /* Check behavior of GL_CLIP_PLANE0 + n where n == 
max_clip_planes */

+    enum_value = GL_CLIP_PLANE0 + max_clip_planes;
+    sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
-    printf("Trying glIsEnabled(%s): ", enum_name);
-    b = glIsEnabled(enum_value);
-    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+    printf("Trying glIsEnabled(%s): ", enum_name);
+    b = glIsEnabled(enum_value);
+    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() 
&& pass;

-    printf("Trying glGetBooleanv(%s): ", enum_name);
-    glGetBooleanv(enum_value, &b);
-    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+    printf("Trying glGetBooleanv(%s): ", enum_name);
+    glGetBooleanv(enum_value, &b);
+    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() 
&& pass;

-    printf("Trying glEnable(%s): ", enum_name);
-    glEnable(enum_value);
-    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+    printf("Trying glEnable(%s): ", enum_name);
+    glEnable(enum_value);
+    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() 
&& pass;

-    printf("Trying glDisable(%s): ", enum_name);
-    glDisable(enum_value);
-    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+    printf("Trying glDisable(%s): ", enum_name);
+    glDisable(enum_value);
+    pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() 
&& pass;

+    }
  piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
  }


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


Re: [Piglit] [PATCH 44/50] gl-2.0: set KHR_no_error status

2017-08-21 Thread Timothy Arceri



On 11/08/17 01:22, Samuel Pitoiset wrote:

Signed-off-by: Samuel Pitoiset 
---
  tests/spec/gl-2.0/api/active-sampler-conflict.c |  1 +
  tests/spec/gl-2.0/api/clip-flag-behavior.c  | 33 ++---


You missed:

getattriblocation-conventional.c
attrib-assignments.c

There are also some more that need to be updated in the parent dir:

tests/spec/gl-2.0/




  2 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/tests/spec/gl-2.0/api/active-sampler-conflict.c 
b/tests/spec/gl-2.0/api/active-sampler-conflict.c
index 74aed3689..848122ace 100644
--- a/tests/spec/gl-2.0/api/active-sampler-conflict.c
+++ b/tests/spec/gl-2.0/api/active-sampler-conflict.c
@@ -61,6 +61,7 @@
  PIGLIT_GL_TEST_CONFIG_BEGIN
  
  	config.supports_gl_compat_version = 20;

+   config.khr_no_error_support = PIGLIT_HAS_ERRORS;
  
  PIGLIT_GL_TEST_CONFIG_END
  
diff --git a/tests/spec/gl-2.0/api/clip-flag-behavior.c b/tests/spec/gl-2.0/api/clip-flag-behavior.c

index 25d7b4efd..c0e031e97 100644
--- a/tests/spec/gl-2.0/api/clip-flag-behavior.c
+++ b/tests/spec/gl-2.0/api/clip-flag-behavior.c
@@ -42,6 +42,7 @@ PIGLIT_GL_TEST_CONFIG_BEGIN
  
  	config.supports_gl_compat_version = 10;

config.window_visual = PIGLIT_GL_VISUAL_RGB | PIGLIT_GL_VISUAL_DOUBLE;
+   config.khr_no_error_support = PIGLIT_NO_ERRORS;
  
  PIGLIT_GL_TEST_CONFIG_END
  
@@ -146,25 +147,27 @@ piglit_init(int argc, char **argv)

pass = check_enable_state(enum_name, enum_value, false) && pass;
}
  
-	/* Check behavior of GL_CLIP_PLANE0 + n where n == max_clip_planes */

-   enum_value = GL_CLIP_PLANE0 + max_clip_planes;
-   sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
+   if (!piglit_khr_no_error) {
+   /* Check behavior of GL_CLIP_PLANE0 + n where n == 
max_clip_planes */
+   enum_value = GL_CLIP_PLANE0 + max_clip_planes;
+   sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
  
-	printf("Trying glIsEnabled(%s): ", enum_name);

-   b = glIsEnabled(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glIsEnabled(%s): ", enum_name);
+   b = glIsEnabled(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
  
-	printf("Trying glGetBooleanv(%s): ", enum_name);

-   glGetBooleanv(enum_value, &b);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glGetBooleanv(%s): ", enum_name);
+   glGetBooleanv(enum_value, &b);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
  
-	printf("Trying glEnable(%s): ", enum_name);

-   glEnable(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glEnable(%s): ", enum_name);
+   glEnable(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
  
-	printf("Trying glDisable(%s): ", enum_name);

-   glDisable(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glDisable(%s): ", enum_name);
+   glDisable(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
+   }
  
  	piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);

  }


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


[Piglit] [PATCH 44/50] gl-2.0: set KHR_no_error status

2017-08-10 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset 
---
 tests/spec/gl-2.0/api/active-sampler-conflict.c |  1 +
 tests/spec/gl-2.0/api/clip-flag-behavior.c  | 33 ++---
 2 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/tests/spec/gl-2.0/api/active-sampler-conflict.c 
b/tests/spec/gl-2.0/api/active-sampler-conflict.c
index 74aed3689..848122ace 100644
--- a/tests/spec/gl-2.0/api/active-sampler-conflict.c
+++ b/tests/spec/gl-2.0/api/active-sampler-conflict.c
@@ -61,6 +61,7 @@
 PIGLIT_GL_TEST_CONFIG_BEGIN
 
config.supports_gl_compat_version = 20;
+   config.khr_no_error_support = PIGLIT_HAS_ERRORS;
 
 PIGLIT_GL_TEST_CONFIG_END
 
diff --git a/tests/spec/gl-2.0/api/clip-flag-behavior.c 
b/tests/spec/gl-2.0/api/clip-flag-behavior.c
index 25d7b4efd..c0e031e97 100644
--- a/tests/spec/gl-2.0/api/clip-flag-behavior.c
+++ b/tests/spec/gl-2.0/api/clip-flag-behavior.c
@@ -42,6 +42,7 @@ PIGLIT_GL_TEST_CONFIG_BEGIN
 
config.supports_gl_compat_version = 10;
config.window_visual = PIGLIT_GL_VISUAL_RGB | PIGLIT_GL_VISUAL_DOUBLE;
+   config.khr_no_error_support = PIGLIT_NO_ERRORS;
 
 PIGLIT_GL_TEST_CONFIG_END
 
@@ -146,25 +147,27 @@ piglit_init(int argc, char **argv)
pass = check_enable_state(enum_name, enum_value, false) && pass;
}
 
-   /* Check behavior of GL_CLIP_PLANE0 + n where n == max_clip_planes */
-   enum_value = GL_CLIP_PLANE0 + max_clip_planes;
-   sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
+   if (!piglit_khr_no_error) {
+   /* Check behavior of GL_CLIP_PLANE0 + n where n == 
max_clip_planes */
+   enum_value = GL_CLIP_PLANE0 + max_clip_planes;
+   sprintf(enum_name, "GL_CLIP_PLANE0 + %d", max_clip_planes);
 
-   printf("Trying glIsEnabled(%s): ", enum_name);
-   b = glIsEnabled(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glIsEnabled(%s): ", enum_name);
+   b = glIsEnabled(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
 
-   printf("Trying glGetBooleanv(%s): ", enum_name);
-   glGetBooleanv(enum_value, &b);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glGetBooleanv(%s): ", enum_name);
+   glGetBooleanv(enum_value, &b);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
 
-   printf("Trying glEnable(%s): ", enum_name);
-   glEnable(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glEnable(%s): ", enum_name);
+   glEnable(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
 
-   printf("Trying glDisable(%s): ", enum_name);
-   glDisable(enum_value);
-   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && pass;
+   printf("Trying glDisable(%s): ", enum_name);
+   glDisable(enum_value);
+   pass = piglit_check_gl_error(GL_INVALID_ENUM) && print_ok() && 
pass;
+   }
 
piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
 }
-- 
2.14.0

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