Module: Mesa Branch: master Commit: 821d7d24f079dc2a154d2af0718a3618c4be9060 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=821d7d24f079dc2a154d2af0718a3618c4be9060
Author: Tomasz Lis <[email protected]> Date: Wed Jul 17 13:49:17 2013 +0200 glx: Handling RENDER_TYPE in glXCreateContext and init_fbconfig_for_chooser Set the correct values of renderType in glXCreateContext and init_fbconfig_for_chooser. Signed-off-by: Tomasz Lis <[email protected]> Reviewed-by: Ian Romanick <[email protected]> --- src/glx/glxcmds.c | 32 ++++++++++++++++++++++++++++---- 1 files changed, 28 insertions(+), 4 deletions(-) diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c index 51b2237..415e784 100644 --- a/src/glx/glxcmds.c +++ b/src/glx/glxcmds.c @@ -373,7 +373,32 @@ glXCreateContext(Display * dpy, XVisualInfo * vis, return None; } - renderType = config->rgbMode ? GLX_RGBA_TYPE : GLX_COLOR_INDEX_TYPE; + /* Choose the context render type based on DRI config values. It is + * unusual to set this type from config, but we have no other choice, as + * this old API does not provide renderType parameter. + */ + if (config->renderType & GLX_RGBA_FLOAT_BIT_ARB) { + renderType = GLX_RGBA_FLOAT_TYPE_ARB; + } else if (config->renderType & GLX_RGBA_UNSIGNED_FLOAT_BIT_EXT) { + renderType = GLX_RGBA_UNSIGNED_FLOAT_TYPE_EXT; + } else if (config->renderType & GLX_RGBA_BIT) { + renderType = GLX_RGBA_TYPE; + } else if (config->renderType & GLX_COLOR_INDEX_BIT) { + renderType = GLX_COLOR_INDEX_TYPE; + } else if (config->rgbMode) { + /* If we're here, then renderType is not set correctly. Let's use a + * safeguard - any TrueColor or DirectColor mode is RGB mode. Such + * default value is needed by old DRI drivers, which didn't set + * renderType correctly as the value was just ignored. + */ + renderType = GLX_RGBA_TYPE; + } else { + /* Safeguard - only one option left, all non-RGB modes are indexed + * modes. Again, this allows drivers with invalid renderType to work + * properly. + */ + renderType = GLX_COLOR_INDEX_TYPE; + } #endif return CreateContext(dpy, vis->visualid, config, shareList, allowDirect, @@ -876,9 +901,8 @@ init_fbconfig_for_chooser(struct glx_config * config, config->transparentAlpha = GLX_DONT_CARE; config->transparentIndex = GLX_DONT_CARE; - config->drawableType = GLX_WINDOW_BIT; - config->renderType = - (config->rgbMode) ? GLX_RGBA_BIT : GLX_COLOR_INDEX_BIT; + /* Set GLX_RENDER_TYPE property to not expect any flags by default. */ + config->renderType = 0; config->xRenderable = GLX_DONT_CARE; config->fbconfigID = (GLXFBConfigID) (GLX_DONT_CARE); _______________________________________________ mesa-commit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-commit
