[Mesa-dev] GL_DEPTH_BUFFER and GL_STENCIL_BUFFER

[Jose Fonseca]

----- Original Message -----
> On 10/08/2011 05:55 AM, Jose Fonseca wrote:
> > Just noticed that the latest glext.h doesn't define
> > DEPTH_BUFFER/STENCIL_BUFFER anymore. This is what enum.spec says:
> >
> > # Due to a syncing problem between the ARB_framebuffer_object
> > extension
> > # specification and the core API specification during development,
> > the
> > # following tokens were present in the .spec file for some time.
> > They are
> > # not actually used anywhere in the OpenGL API or extensions and
> > have been
> > # withdrawn (use DEPTH or STENCIL respectively, instead,
> > as<attachment>
> > # parameters to GetFramebufferAttachmentParameteriv).
> > #    DEPTH_BUFFER                    = 0x8223
> > #    STENCIL_BUFFER                  = 0x8224
> >
> > But Mesa code actually seems to do the opposite, i.e.,
> > GetFramebufferAttachmentParameteriv handles DEPTH_BUFFER but not
> > DEPTH.
> >
> > The GL specs is consistent with the comment.
> >
> > So I think it's better to remove this. Any objects?
> 
> As Brian mentioned, that's my patch series:
> 
> http://marc.info/?l=mesa3d-dev&m=131805740908194&w=2
> 
> I'm planning to push both those patches today, so there's still time
> for
> review comments. :)
> 

Great. Looks good to me AFAICS.

Jose