On 9 January 2012 14:28, Ian Romanick <[email protected]> wrote:

> On 01/09/2012 01:27 PM, Paul Berry wrote:
>
>> No functional change.  In the function
>> __indirect_**glAreTexturesResident(), the variable cmdlen is only used
>> if USE_XCB is not defined.  This patch avoids a compile warning in the
>> event that USE_XCB is defined.
>> ---
>>  src/glx/single2.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/glx/single2.c b/src/glx/single2.c
>> index 66281fa..a4355fc 100644
>> --- a/src/glx/single2.c
>> +++ b/src/glx/single2.c
>> @@ -890,6 +890,7 @@ __indirect_**glAreTexturesResident(GLsizei n, const
>> GLuint * textures,
>>     const GLuint cmdlen = 4 + __GLX_PAD((n * 4));
>>     if (__builtin_expect((n>= 0)&&  (dpy != NULL), 1)) {
>>
>>  #ifdef USE_XCB
>> +      (void) cmdlen;
>>
>
> Maybe just move cmdlen declaration inside the #else part?


Ok, that seems reasonable.  I'll follow up with a revised patch.
_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to