<div dir="ltr">I agree with Ian's proposition to add integer counter which replaces hard-coded index. This also allows to add:<br><br>assert(i <= sizeof(attribs)/sizeof(attribs[0]))<br><br>While both GLX 1.4 spec and older GLX_SGIX_pbuffer do not mention what value to return if GLX_PRESERVED_CONTENTS is unset on creation, they clearly indicate that the new buffer should behave as if GLX_PRESERVED_CONTENTS was = True:<br>
<br>"<pre>If this (GLX_PRESERVED_CONTENTS) attribute is not
    specified, or if it is specified as True in <attrib_list>, then when a
    resource conflict occurs the contents of the pbuffer will be preserved
    (most likely by swapping out portions of the buffer to main memory).<br>"<br></pre>I don't think this code returns apps settings, i think it is always used to query a specific buffer (but I may be wrong).<br>
<br><div class="gmail_quote">2013/3/1 Adam Jackson <span dir="ltr"><<a href="mailto:ajax@redhat.com" target="_blank">ajax@redhat.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, 2013-02-28 at 13:55 -0800, Ian Romanick wrote:<br>
> On 02/25/2013 02:04 PM, Adam Jackson wrote:<br>
> > We back pixmaps with pbuffers so they're never actually clobbered, so<br>
> > we're really just recording the value passed in at create time so that<br>
> > you get the same thing when you query.<br>
><br>
> Is that actually right?  There are some cases where they query returns<br>
> the actual state, and there are some cases where the query returns the<br>
> apps setting.  I haven't checked to see which this is.  In Mesa, we've<br>
> taken to quoting the spec in cases like this.<br>
<br>
</div>If the spec had anything to say about the matter, I would happily quote<br>
it.  Unfortunately GLX is much less rigorous about this than GL.  My<br>
reply to Eric was based on a careful reading of GLX 1.4.  While it's<br>
clearly within the law to treat all pbuffers as preserved, the spec is<br>
silent on whether the queried attribute value reflects the value given<br>
at pbuffer creation.<br>
<br>
- ajax<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</div></div></blockquote></div><br></div>