According to version 15 of the EGL_KHR_create_context spec,
EGL_CONTEXT_OPENGL_DEBUG_BIT_KHR is a valid flag for OpenGL ES contexts.
But the test "EGL_KHR_create_context/invalid flag GLES" verifies that it
is an *invalid* flag.

Signed-off-by: Chad Versace <chad.vers...@linux.intel.com>
---
 .../spec/egl_khr_create_context/invalid-flag-gles.c   | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/tests/egl/spec/egl_khr_create_context/invalid-flag-gles.c 
b/tests/egl/spec/egl_khr_create_context/invalid-flag-gles.c
index b12db20..5af773b 100644
--- a/tests/egl/spec/egl_khr_create_context/invalid-flag-gles.c
+++ b/tests/egl/spec/egl_khr_create_context/invalid-flag-gles.c
@@ -60,19 +60,20 @@ int main(int argc, char **argv)
        uint32_t flag = 0x80000000;
        bool ran_test = false;
 
-       /* The EGL_KHR_create_context spec says:
+       /* According to the EGL_KHR_create_context spec, version 15, there
+        * exists exactly one valid flag for OpenGL ES contexts: the debug
+        * flag.
         *
-        *     "The value for attribute EGL_CONTEXT_FLAGS_KHR specifies a set of
-        *     flag bits affecting the context. Flags are only defined for 
OpenGL
-        *     context creation, and specifying a flags value other than zero 
for
-        *     other types of contexts, including OpenGL ES contexts, will 
generate
-        *     an error."
+        *     If the EGL_CONTEXT_OPENGL_DEBUG_BIT_KHR flag bit is set in
+        *     EGL_CONTEXT_FLAGS_KHR, then a <debug context> will be created.
+        *     [...] This bit is supported for OpenGL and OpenGL ES contexts.
         */
-       uint32_t first_valid_flag = 0;
+       const EGLint valid_flags = EGL_CONTEXT_OPENGL_DEBUG_BIT_KHR;
+       const EGLint invalid_flags = ~valid_flags;
 
        if (EGL_KHR_create_context_setup(EGL_OPENGL_ES_BIT)) {
                ran_test = true;
-               while (flag != first_valid_flag) {
+               while (flag & invalid_flags) {
                        pass = pass && try_flag(flag);
                        flag >>= 1;
                }
@@ -83,7 +84,7 @@ int main(int argc, char **argv)
        if (EGL_KHR_create_context_setup(EGL_OPENGL_ES2_BIT)) {
                ran_test = true;
                flag = 0x80000000;
-               while (flag != first_valid_flag) {
+               while (flag & invalid_flags) {
                        pass = pass && try_flag(flag);
                        flag >>= 1;
                }
-- 
1.8.3.1

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

Reply via email to