<div dir="ltr">Hello,<br><br>I've noticed the GLX_LARGEST_PBUFFER property is not implemented correctly (it is, but only in Gallium). Specification:<br><br><a href="http://www.opengl.org/sdk/docs/man2/xhtml/glXCreatePbuffer.xml" target="_blank">http://www.opengl.org/sdk/docs/man2/xhtml/glXCreatePbuffer.xml</a><br>


<br>The value should be used to decrease buffer size when necessary, but also stored so it can be retrieved by glXQueryDrawable().<br><br>Currently CreatePbuffer() from glx_pbuffer.c just calls XCreatePixmap() without checking the property. Also, the CreateDRIDrawable() receives it in a list of attributes, but does nothing with it.<br>
<br>Width and height parameters should be probably checked and corrected before the XCreatePixmap() call. Passing original width and height and value of the GLX_LARGEST_PBUFFER attribute to XCreatePixmap() would allow more advanced size checking, but that would require changing XCreatePixmap() definition.<br>
<br><br>The GLX_LARGEST_PBUFFER property isn't used very often, but it is a part of GLX specification - so definitely can't be left without implementation.<br>
<br>What do you think about it?<br>Any thoughts about how this should be implemented?<br><br>I'm curious to hear your opinion,<br>Tomasz<br><br>
</div>