From 896c650b1b4bbab67a39ae2bb6e30d08f2b62d8a Mon Sep 17 00:00:00 2001 From: Adam Jackson Date: Tue, 13 Apr 2021 16:54:31 -0400 Subject: [PATCH] glx: Mark GLX_{ATI_pixel_format_float,NV_float_buffer} as supported MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ... at least as far as libGL is concerned, because they'd work just fine if you tried. The only thing the ATI extension seems to add (I can't find an official spec, this is inferred from the registry XML) is selecting the GLX_RENDER_TYPE, which we don't validate before putting on the wire. The only thing the NV extension adds is an additional fbconfig attribute, and that only known by glXGetFBConfigAttrib; our implementation of that just reads the value the server sends, if any, and doesn't try to filter out unknown attributes. Reviewed-by: Marek Olšák Reviewed-by: Ian Romanick Part-of: --- src/glx/glxextensions.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/glx/glxextensions.c b/src/glx/glxextensions.c index bfa46a27f6b..6c1763f4dba 100644 --- a/src/glx/glxextensions.c +++ b/src/glx/glxextensions.c @@ -144,13 +144,13 @@ static const struct extension_info known_glx_extensions[] = { { GLX(EXT_texture_from_pixmap), Y, N, N, N }, { GLX(EXT_visual_info), Y, Y, N, N }, { GLX(EXT_visual_rating), Y, Y, N, N }, - { GLX(ATI_pixel_format_float), N, N, N, N }, + { GLX(ATI_pixel_format_float), Y, N, N, N }, { GLX(INTEL_swap_event), Y, N, N, N }, { GLX(MESA_copy_sub_buffer), Y, N, N, N }, { GLX(MESA_multithread_makecurrent), Y, N, N, Y }, { GLX(MESA_query_renderer), Y, N, N, Y }, { GLX(MESA_swap_control), Y, N, N, Y }, - { GLX(NV_float_buffer), N, N, N, N }, + { GLX(NV_float_buffer), Y, N, N, N }, { GLX(OML_swap_method), Y, Y, N, N }, { GLX(OML_sync_control), Y, N, N, Y }, { GLX(SGIS_multisample), Y, Y, N, N },