Instead of setting based on set/unset, allow users to use boolean values.
In the docs, use `NO_DRAWARRAYS=true` instead of `NO_DRAWARRAYS=1` as it's
clearer IMO.
Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com>
---
docs/envvars.html | 2 +-
src/glx/indirect_glx.c | 6 +++++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/docs/envvars.html b/docs/envvars.html
index 89563e3636..45e45eb0c9 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -31,7 +31,7 @@
<li>LIBGL_DRIVERS_PATH - colon-separated list of paths to search for DRI
drivers
<li>LIBGL_ALWAYS_INDIRECT - if set to `true`, forces an indirect rendering
context/connection.
<li>LIBGL_ALWAYS_SOFTWARE - if set to `true`, always use software rendering
-<li>LIBGL_NO_DRAWARRAYS - if set do not use DrawArrays GLX protocol (for
debugging)
+<li>LIBGL_NO_DRAWARRAYS - if set to `true`, 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 to `true`.
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index 51dd2cacb2..4302a8ff28 100644
--- a/src/glx/indirect_glx.c
+++ b/src/glx/indirect_glx.c
@@ -30,9 +30,13 @@
* Kristian Høgsberg (k...@bitplanet.net)
*/
+#include <stdbool.h>
+
#include "glapi.h"
#include "glxclient.h"
+#include "util/debug.h"
+
#ifndef GLX_USE_APPLEGL
extern struct _glapi_table *__glXNewIndirectAPI(void);
@@ -371,7 +375,7 @@ indirect_create_context(struct glx_screen *psc,
return NULL;
}
gc->client_state_private = state;
- state->NoDrawArraysProtocol = (getenv("LIBGL_NO_DRAWARRAYS") != NULL);
+ state->NoDrawArraysProtocol = env_var_as_boolean("LIBGL_NO_DRAWARRAYS",
false);
/*
** Create a temporary buffer to hold GLX rendering commands. The size
--
Cheers,
Eric
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
