diff options
author | Rob Clark <robclark@freedesktop.org> | 2016-12-01 10:57:24 -0800 |
---|---|---|
committer | Chad Versace <chadversary@chromium.org> | 2016-12-01 10:57:24 -0800 |
commit | 2ba4c7e1548c6251085cd75563c182cc6708eaab (patch) | |
tree | 143a457f3e558473e6a83ddd6e8b440e9f656065 /src/egl | |
parent | cce04a463089ef3ce39df3e2ae25066c2e1ddfdf (diff) |
egl: un-fallthrough sync attr parsing
Doesn't work so well when you start having more than one possible
attrib. Prep-work for next patch.
Signed-off-by: Rob Clark <robdclark@gmail.com>
Tested-by: Rafael Antognolli <rafael.antognolli@intel.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
Tested-by: Chad Versace <chadversary@chromium.org>
Diffstat (limited to 'src/egl')
-rw-r--r-- | src/egl/main/eglsync.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/egl/main/eglsync.c b/src/egl/main/eglsync.c index 005cb31fc31..7b2c882d813 100644 --- a/src/egl/main/eglsync.c +++ b/src/egl/main/eglsync.c @@ -55,11 +55,11 @@ _eglParseSyncAttribList(_EGLSync *sync, const EGLAttrib *attrib_list) case EGL_CL_EVENT_HANDLE_KHR: if (sync->Type == EGL_SYNC_CL_EVENT_KHR) { sync->CLEvent = val; - break; + } else { + err = EGL_BAD_ATTRIBUTE; } - /* fall through */ + break; default: - (void) val; err = EGL_BAD_ATTRIBUTE; break; } |