[PATCH evdev 2/2] Reshuffle to avoid the need for XI86_CONFIGURED.

Benjamin Tissoires tissoire at cena.fr
Tue Oct 26 23:42:28 PDT 2010


Le 27/10/2010 00:34, Peter Hutterer a écrit :
> ...
>>> +    {
>>> +        pInfo->flags = XI86_CONFIGURED;
>>
>> You just undefined XI86_CONFIGURED... I'm a bit lost here.
>> The define seems to have been purged in ABI_XINPUT>= 12, so
>> shouldn't we #ifdef this or no?
>>
>> The rest of the patch seems good.
>
> right, this is a bit hard to detect from the patch context.
> there are two EvdevPreInit, one for ABI<  12 and one for ABI 12, with the
> former calling the latter as NewEvdevPreInit (see the context above).
>
> This hunk is within a ABI<  12 ifdef, so XI86_CONFIGURED exists here.
>

ok, thanks,

Reviewed-by: Benjamin Tissoires <tissoire at cena.fr>

Cheers,
Benjamin


More information about the xorg-devel mailing list