diff options
author | Eric Engestrom <[email protected]> | 2019-02-02 11:38:45 +0000 |
---|---|---|
committer | Eric Engestrom <[email protected]> | 2019-02-06 11:53:24 +0000 |
commit | 54fa5eceae237598d6fb329ec7cbe1d3a85e7d39 (patch) | |
tree | 870badc295851feaba8cf7e43318ed9542a7c0a7 /src/egl/main/eglsurface.c | |
parent | a81d5587d6ad20506c13199a1f4573fdf6350709 (diff) |
egl: use coherent variable names
`EGLDisplay` variables (the opaque Khronos type) have mostly been
consistently called `dpy`, as this is the name used in the Khronos
specs.
However, `_EGLDisplay` variables (our internal struct) have been
randomly called `dpy` when there was no local variable clash with
`EGLDisplay`s, and `disp` otherwise.
Let's be consistent and use `dpy` for the Khronos type, and `disp`
for our struct.
Signed-off-by: Eric Engestrom <[email protected]>
Acked-by: Emil Velikov <[email protected]>
Acked-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/egl/main/eglsurface.c')
-rw-r--r-- | src/egl/main/eglsurface.c | 58 |
1 files changed, 29 insertions, 29 deletions
diff --git a/src/egl/main/eglsurface.c b/src/egl/main/eglsurface.c index cd1c7ea4c56..ffef1b8f259 100644 --- a/src/egl/main/eglsurface.c +++ b/src/egl/main/eglsurface.c @@ -53,7 +53,7 @@ static EGLint _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) { - _EGLDisplay *dpy = surf->Resource.Display; + _EGLDisplay *disp = surf->Resource.Display; EGLint type = surf->Type; EGLint texture_type = EGL_PBUFFER_BIT; EGLint i, err = EGL_SUCCESS; @@ -63,7 +63,7 @@ _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) if (!attrib_list) return EGL_SUCCESS; - if (dpy->Extensions.NOK_texture_from_pixmap) + if (disp->Extensions.NOK_texture_from_pixmap) texture_type |= EGL_PIXMAP_BIT; for (i = 0; attrib_list[i] != EGL_NONE; i++) { @@ -73,7 +73,7 @@ _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) switch (attr) { /* common attributes */ case EGL_GL_COLORSPACE_KHR: - if (!dpy->Extensions.KHR_gl_colorspace) { + if (!disp->Extensions.KHR_gl_colorspace) { err = EGL_BAD_ATTRIBUTE; break; } @@ -89,84 +89,84 @@ _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) surf->GLColorspace = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_RX_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_r.x = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_RY_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_r.y = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_GX_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_g.x = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_GY_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_g.y = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_BX_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_b.x = val; break; case EGL_SMPTE2086_DISPLAY_PRIMARY_BY_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.display_primary_b.y = val; break; case EGL_SMPTE2086_WHITE_POINT_X_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.white_point.x = val; break; case EGL_SMPTE2086_WHITE_POINT_Y_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.white_point.y = val; break; case EGL_SMPTE2086_MAX_LUMINANCE_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.max_luminance = val; break; case EGL_SMPTE2086_MIN_LUMINANCE_EXT: - if (!dpy->Extensions.EXT_surface_SMPTE2086_metadata) { + if (!disp->Extensions.EXT_surface_SMPTE2086_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.min_luminance = val; break; case EGL_CTA861_3_MAX_CONTENT_LIGHT_LEVEL_EXT: - if (!dpy->Extensions.EXT_surface_CTA861_3_metadata) { + if (!disp->Extensions.EXT_surface_CTA861_3_metadata) { err = EGL_BAD_ATTRIBUTE; break; } surf->HdrMetadata.max_cll = val; break; case EGL_CTA861_3_MAX_FRAME_AVERAGE_LEVEL_EXT: - if (!dpy->Extensions.EXT_surface_CTA861_3_metadata) { + if (!disp->Extensions.EXT_surface_CTA861_3_metadata) { err = EGL_BAD_ATTRIBUTE; break; } @@ -217,7 +217,7 @@ _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) } break; case EGL_POST_SUB_BUFFER_SUPPORTED_NV: - if (!dpy->Extensions.NV_post_sub_buffer || + if (!disp->Extensions.NV_post_sub_buffer || type != EGL_WINDOW_BIT) { err = EGL_BAD_ATTRIBUTE; break; @@ -333,7 +333,7 @@ _eglParseSurfaceAttribList(_EGLSurface *surf, const EGLint *attrib_list) * \return EGL_TRUE if no errors, EGL_FALSE otherwise. */ EGLBoolean -_eglInitSurface(_EGLSurface *surf, _EGLDisplay *dpy, EGLint type, +_eglInitSurface(_EGLSurface *surf, _EGLDisplay *disp, EGLint type, _EGLConfig *conf, const EGLint *attrib_list) { const char *func; @@ -366,7 +366,7 @@ _eglInitSurface(_EGLSurface *surf, _EGLDisplay *dpy, EGLint type, /* The config can't be used to create a surface of this type */ return _eglError(EGL_BAD_MATCH, func); - _eglInitResource(&surf->Resource, sizeof(*surf), dpy); + _eglInitResource(&surf->Resource, sizeof(*surf), disp); surf->Type = type; surf->Config = conf; surf->Lost = EGL_FALSE; @@ -426,7 +426,7 @@ _eglInitSurface(_EGLSurface *surf, _EGLDisplay *dpy, EGLint type, EGLBoolean -_eglQuerySurface(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, +_eglQuerySurface(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *surface, EGLint attribute, EGLint *value) { switch (attribute) { @@ -513,7 +513,7 @@ _eglQuerySurface(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, *value = surface->VGColorspace; break; case EGL_GL_COLORSPACE_KHR: - if (!dpy->Extensions.KHR_gl_colorspace) + if (!disp->Extensions.KHR_gl_colorspace) return _eglError(EGL_BAD_ATTRIBUTE, "eglQuerySurface"); *value = surface->GLColorspace; @@ -522,11 +522,11 @@ _eglQuerySurface(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, *value = surface->PostSubBufferSupportedNV; break; case EGL_BUFFER_AGE_EXT: - if (!dpy->Extensions.EXT_buffer_age) + if (!disp->Extensions.EXT_buffer_age) return _eglError(EGL_BAD_ATTRIBUTE, "eglQuerySurface"); _EGLContext *ctx = _eglGetCurrentContext(); - EGLint result = drv->API.QueryBufferAge(drv, dpy, surface); + EGLint result = drv->API.QueryBufferAge(drv, disp, surface); /* error happened */ if (result < 0) return EGL_FALSE; @@ -585,7 +585,7 @@ _eglQuerySurface(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, * Default fallback routine - drivers might override this. */ EGLBoolean -_eglSurfaceAttrib(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, +_eglSurfaceAttrib(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *surface, EGLint attribute, EGLint value) { EGLint confval; @@ -621,7 +621,7 @@ _eglSurfaceAttrib(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, surface->MultisampleResolve = value; break; case EGL_RENDER_BUFFER: - if (!dpy->Extensions.KHR_mutable_render_buffer) { + if (!disp->Extensions.KHR_mutable_render_buffer) { err = EGL_BAD_ATTRIBUTE; break; } @@ -710,7 +710,7 @@ _eglSurfaceAttrib(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, EGLBoolean -_eglBindTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, +_eglBindTexImage(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *surface, EGLint buffer) { EGLint texture_type = EGL_PBUFFER_BIT; @@ -719,7 +719,7 @@ _eglBindTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, * Drivers must implement the real stuff. */ - if (dpy->Extensions.NOK_texture_from_pixmap) + if (disp->Extensions.NOK_texture_from_pixmap) texture_type |= EGL_PIXMAP_BIT; if (!(surface->Type & texture_type)) @@ -740,7 +740,7 @@ _eglBindTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surface, } EGLBoolean -_eglReleaseTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surf, +_eglReleaseTexImage(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *surf, EGLint buffer) { /* Just do basic error checking and return success/fail. @@ -764,7 +764,7 @@ _eglReleaseTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surf, if (buffer != EGL_BACK_BUFFER) return _eglError(EGL_BAD_PARAMETER, "eglReleaseTexImage"); - if (dpy->Extensions.NOK_texture_from_pixmap) + if (disp->Extensions.NOK_texture_from_pixmap) texture_type |= EGL_PIXMAP_BIT; if (!(surf->Type & texture_type)) @@ -777,7 +777,7 @@ _eglReleaseTexImage(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surf, EGLBoolean -_eglSwapInterval(_EGLDriver *drv, _EGLDisplay *dpy, _EGLSurface *surf, +_eglSwapInterval(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *surf, EGLint interval) { return EGL_TRUE; |