[Mesa-dev] [PATCH] glx: Fix the default values for GLXFBConfig attributes

[Dave Airlie]

On Tue, Feb 18, 2014 at 8:10 AM, Fredrik Höglund <fredrik at kde.org> wrote:
> The default values for GLX_DRAWABLE_TYPE and GLX_RENDER_TYPE are
> GLX_WINDOW_BIT and GLX_RGBA_BIT respectively, as specified in
> the GLX 1.4 specification.
>
> This fixes the glx-choosefbconfig-defaults piglit test.

Looks good to me, I can also confirm this fixes glxinfo from mesa
libGL to nvidia libGL.

Reviewed-by: Dave Airlie <airlied at redhat.com>

Dave.

>
> Cc: "9.2 10.0 10.1" <mesa-stable at lists.freedesktop.org>
> ---
>  src/glx/glxcmds.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
> index 38a5262..43c473b 100644
> --- a/src/glx/glxcmds.c
> +++ b/src/glx/glxcmds.c
> @@ -925,13 +925,10 @@ init_fbconfig_for_chooser(struct glx_config * config,
>     if (fbconfig_style_tags) {
>        config->rgbMode = GL_TRUE;
>        config->doubleBufferMode = GLX_DONT_CARE;
> -      /* allow any kind of drawable, including those for off-screen buffers */
> -      config->drawableType = 0;
> -   } else {
> -       /* allow configs which support on-screen drawing */
> -       config->drawableType = GLX_WINDOW_BIT;
> +      config->renderType = GLX_RGBA_BIT;
>     }
>
> +   config->drawableType = GLX_WINDOW_BIT;
>     config->visualRating = GLX_DONT_CARE;
>     config->transparentPixel = GLX_NONE;
>     config->transparentRed = GLX_DONT_CARE;
> @@ -940,8 +937,6 @@ init_fbconfig_for_chooser(struct glx_config * config,
>     config->transparentAlpha = GLX_DONT_CARE;
>     config->transparentIndex = GLX_DONT_CARE;
>
> -   /* 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);
>
> --
> 1.8.5.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev