Hi,

On 4 October 2011 16:00, Michal Suchanek <[email protected]> wrote:
> On 4 October 2011 16:57, Matt Turner <[email protected]> wrote:
>> On Tue, Oct 4, 2011 at 2:55 PM, Michal Suchanek <[email protected]> wrote:
>>> On 4 October 2011 16:51, Matt Turner <[email protected]> wrote:
>>>> On Tue, Oct 4, 2011 at 2:33 PM, Michal Suchanek <[email protected]> 
>>>> wrote:
>>>>> BTW it adds a warning about stuff variable being unused  - probably
>>>>> defined in one of the macros.
>>>>
>>>> It's because the swap macros stopped taking a second argument.
>>>
>>> No, that would be the 'n' variable which you patched out already.
>>
>> Then I have no idea what you're talking about.
>>
> ../../Xext/xselinux_ext.c: In function ‘SProcSELinuxQueryVersion’:
> ../../Xext/xselinux_ext.c:529:5: warning: unused variable ‘stuff’
> [-Wunused-variable]

'stuff' is the request variable generated by the REQUEST_* macros
(stupid name, I know).  Since SProcSELinuxQueryVersion no longer
touches the macro, you can just delete the REQUEST_* macro and make it
simply return ProcSELinuxQueryVersion(client).

Cheers,
Daniel
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to