[Mesa-dev] [PATCH mesa 04/10] glx: turn LIBGL_ALLOW_SOFTWARE into a boolean
Eric Engestrom
eric.engestrom at imgtec.com
Fri Sep 8 12:40:50 UTC 2017
Instead of setting based on set/unset, allow users to use boolean values.
In the help string, use `ALLOW=true` instead of `ALLOW=1` as it's clearer IMO.
Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
---
src/glx/apple/apple_visual.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/glx/apple/apple_visual.c b/src/glx/apple/apple_visual.c
index a491823917..d8fd7b0cb3 100644
--- a/src/glx/apple/apple_visual.c
+++ b/src/glx/apple/apple_visual.c
@@ -96,7 +96,7 @@ apple_visual_create_pfobj(CGLPixelFormatObj * pfobj, const struct glx_config * m
attr[numattr++] = kCGLPFARendererID;
attr[numattr++] = kCGLRendererGenericFloatID;
}
- else if (getenv("LIBGL_ALLOW_SOFTWARE") != NULL) {
+ else if (env_var_as_boolean("LIBGL_ALLOW_SOFTWARE", false)) {
apple_glx_diagnostic
("Software rendering is not being excluded. Not using kCGLPFAAccelerated.\n");
}
@@ -190,7 +190,7 @@ apple_visual_create_pfobj(CGLPixelFormatObj * pfobj, const struct glx_config * m
if (!*pfobj) {
snprintf(__crashreporter_info_buff__, sizeof(__crashreporter_info_buff__),
- "No matching pixelformats found, perhaps try using LIBGL_ALLOW_SOFTWARE\n");
+ "No matching pixelformats found, perhaps try setting LIBGL_ALLOW_SOFTWARE=true\n");
fprintf(stderr, "%s", __crashreporter_info_buff__);
abort();
}
--
Cheers,
Eric
More information about the mesa-dev
mailing list