On 9 January 2012 14:28, Ian Romanick <span dir="ltr">&lt;<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>On 01/09/2012 01:27 PM, Paul Berry wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
No functional change.  In the function<br>
__indirect_<u></u>glAreTexturesResident(), the variable cmdlen is only used<br>
if USE_XCB is not defined.  This patch avoids a compile warning in the<br>
event that USE_XCB is defined.<br>
---<br>
  src/glx/single2.c |    1 +<br>
  1 files changed, 1 insertions(+), 0 deletions(-)<br>
<br>
diff --git a/src/glx/single2.c b/src/glx/single2.c<br>
index 66281fa..a4355fc 100644<br>
--- a/src/glx/single2.c<br>
+++ b/src/glx/single2.c<br>
@@ -890,6 +890,7 @@ __indirect_<u></u>glAreTexturesResident(GLsizei n, const GLuint * textures,<br>
     const GLuint cmdlen = 4 + __GLX_PAD((n * 4));<br></div>
     if (__builtin_expect((n&gt;= 0)&amp;&amp;  (dpy != NULL), 1)) {<div><br>
  #ifdef USE_XCB<br>
+      (void) cmdlen;<br>
</div></blockquote>
<br>
Maybe just move cmdlen declaration inside the #else part?</blockquote><div><br>Ok, that seems reasonable.  I&#39;ll follow up with a revised patch.<br></div></div>