Instead of setting based on set/unset, allow users to use boolean values. In the docs, use `DISABLE=true` instead of `DISABLE=1` as it's clearer IMO.
Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com> --- docs/envvars.html | 2 +- src/egl/drivers/dri2/platform_x11.c | 2 +- src/glx/glxext.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/envvars.html b/docs/envvars.html index aeb5ff2847..89563e3636 100644 --- a/docs/envvars.html +++ b/docs/envvars.html @@ -34,7 +34,7 @@ <li>LIBGL_NO_DRAWARRAYS - if set do not use DrawArrays GLX protocol (for debugging) <li>LIBGL_SHOW_FPS - print framerate to stdout based on the number of glXSwapBuffers calls per second. -<li>LIBGL_DRI3_DISABLE - disable DRI3 if set (the value does not matter) +<li>LIBGL_DRI3_DISABLE - disable DRI3 if set to `true`. </ul> diff --git a/src/egl/drivers/dri2/platform_x11.c b/src/egl/drivers/dri2/platform_x11.c index 5d5eea3773..d80eee2969 100644 --- a/src/egl/drivers/dri2/platform_x11.c +++ b/src/egl/drivers/dri2/platform_x11.c @@ -1461,7 +1461,7 @@ dri2_initialize_x11(_EGLDriver *drv, _EGLDisplay *disp) if (!env_var_as_boolean("LIBGL_ALWAYS_SOFTWARE", false)) { #ifdef HAVE_DRI3 - if (!getenv("LIBGL_DRI3_DISABLE")) + if (!env_var_as_boolean("LIBGL_DRI3_DISABLE", false)) initialized = dri2_initialize_x11_dri3(drv, disp); #endif diff --git a/src/glx/glxext.c b/src/glx/glxext.c index 35e582cb22..20b5be5987 100644 --- a/src/glx/glxext.c +++ b/src/glx/glxext.c @@ -920,7 +920,7 @@ __glXInitialize(Display * dpy) #if defined(GLX_USE_DRM) if (glx_direct && glx_accel) { #if defined(HAVE_DRI3) - if (!getenv("LIBGL_DRI3_DISABLE")) + if (!env_var_as_boolean("LIBGL_DRI3_DISABLE", false)) dpyPriv->dri3Display = dri3_create_display(dpy); #endif /* HAVE_DRI3 */ dpyPriv->dri2Display = dri2CreateDisplay(dpy); -- Cheers, Eric _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev