glx: Fix drawable type inference in visual/fbconfig setup
authorAdam Jackson <ajax@redhat.com>
Fri, 6 Jan 2023 17:24:11 +0000 (12:24 -0500)
committerMarge Bot <emma+marge@anholt.net>
Fri, 13 Jan 2023 19:59:26 +0000 (19:59 +0000)
commit2cc3763d0455de13e78a82123ddc47a00e5ea7eb
tree192c79a1f226e67bc1fb150fda4b45d0c53f1795
parent86c9a31d99d37f7b8838e069631e2f8b16d991e9
glx: Fix drawable type inference in visual/fbconfig setup

This was just all kinds of wrong. Note that the comment implying "this
is a workaround for old XQuartz" is on the "not apple" side of the
ifdef. Anyway. xserver didn't start sending GLX_DRAWABLE_TYPE in the
fbconfig until:

    commit 8cde0af3c57f0375ba8ba77af9fdf74b79d9496d
    Author: Kristian Høgsberg <krh@redhat.com>
    Date:   Wed Apr 2 19:06:40 2008 -0400

        Send the GLX_EXT_texture_from_pixmap attributes to the client.

So we can remove this default from the fbconfig path. But we preserve it
for the GLXGetVisualConfigs path, because that is specified not to send
GLX_DRAWABLE_TYPE.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20549>
src/glx/glxext.c