Bug 29488

Summary: [patch] MesaGLUT needs to support GLX_ARB_multisample
Product: Mesa Reporter: Nigel Stewart <nstewart>
Component: GLUTAssignee: mesa-dev
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: nstewart
Version: 7.6   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments: Diff w.r.t MesaGlut 7.8.2

Description Nigel Stewart 2010-08-10 09:47:50 UTC
Created attachment 37772 [details]
Diff w.r.t MesaGlut 7.8.2

In glut_dstr.c there is logic based on GLX_SGIS_multisample that also ought to support the ARB version: GLX_ARB_multisample.

Recent Nvidia drivers don't seem to support the SGIS extension, but _do_ support the ARB extension.

Tested with the following GLUT call on Linux:

glutInitDisplayString("rgb double depth samples~4");

Without the patch glutCreateWindow fails.
Comment 1 Nigel Stewart 2010-08-10 10:09:28 UTC
Some more observation in support of this patch:

Note that glut_win.c already does support GLX_ARB_multisample.

However, glut_overlay doesn't seem to.
Comment 2 Brian Paul 2010-08-10 16:30:16 UTC
Thanks.  I've applied your patch and updated glut_overlay.c too.
Commit 6768afbe9fb4cb2324317f460230ebbbd1e0e295

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.