<div dir="ltr"><div><div><div><div><div>Thank you for your feedback.<br><br></div>I will remove GL_TRUE/FALSE and treat these as C logic values.<br></div>I
 will change setting of the mutually exclusive flags. While I think 
using bitwise OR emphases the fact that these are bit flags and not 
integer values, both ways will work.<br>
<br></div>Please make sure someone will verify the possible clang problem - if we can't solve this, I will just remove the whole section.<br></div><br></div>I will leave the fakeglx hunks unchanged.<br><br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">2013/7/16 Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.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 07/15/2013 12:33 PM, Ian Romanick wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 07/15/2013 07:28 AM, Tomasz Lis wrote:<br>
</blockquote></div>
[...]<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/src/mesa/drivers/x11/<u></u>fakeglx.c<br>
b/src/mesa/drivers/x11/<u></u>fakeglx.c<br>
index 969ee7d..7a2cfbe 100644<br>
--- a/src/mesa/drivers/x11/<u></u>fakeglx.c<br>
+++ b/src/mesa/drivers/x11/<u></u>fakeglx.c<br>
</blockquote>
<br>
I'm not sure these last two hunks are necessary.  I don't think the<br>
fakeglx patch can ever see float configs.  Brian knows that code much<br>
better than I do.<br>
</blockquote>
<br></div>
They don't do any harm, AFAICT.  It's possible I might add float support to the fake glx code someday.<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -1090,6 +1090,9 @@ choose_visual( Display *dpy, int screen, const<br>
int *list, GLboolean fbConfig )<br>
              else if (*parselist & GLX_COLOR_INDEX_BIT) {<br>
                 rgb_flag = GL_FALSE;<br>
              }<br>
+            else if (*parselist &<br>
(GLX_RGBA_FLOAT_BIT_ARB|GLX_<u></u>RGBA_UNSIGNED_FLOAT_BIT_EXT)) {<br>
+               rgb_flag = GL_TRUE;<br>
+            }<br>
              else if (*parselist == 0) {<br>
                 rgb_flag = GL_TRUE;<br>
              }<br>
@@ -1761,7 +1764,9 @@ get_config( XMesaVisual xmvis, int attrib, int<br>
*value, GLboolean fbconfig )<br>
        case GLX_RENDER_TYPE_SGIX:<br>
           if (!fbconfig)<br>
              return GLX_BAD_ATTRIBUTE;<br>
-         if (xmvis->mesa_visual.rgbMode)<br>
+         if (xmvis->mesa_visual.floatMode)<br>
+            *value = GLX_RGBA_FLOAT_BIT_ARB;<br>
+         else if (xmvis->mesa_visual.rgbMode)<br>
              *value = GLX_RGBA_BIT;<br>
           else<br>
              *value = GLX_COLOR_INDEX_BIT;<br>
<br>
</blockquote></blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-Brian<br>
<br>
<br>
</font></span></blockquote></div><br></div>