On Thu, Oct 18, 2012 at 7:12 PM, Thomas Jaeger <thjae...@gmail.com> wrote:
> On 09/29/2012 07:58 PM, Jason Gerecke wrote:
>> @@ -419,6 +419,14 @@ static int wcmDevInit(DeviceIntPtr pWcm)
>>               return FALSE;
>>       }
>>
>> +#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) >= 16
>> +     if (!InitTouchClassDeviceStruct(pInfo->dev, 0, XIDependentTouch, 2)) {
>> +             xf86Msg(X_ERROR, "Unable to init touch class device 
>> struct!\n");
>> +             return FALSE;
>> +     }
>> +     priv->common->touch_mask = valuator_mask_new(2);
>> +#endif
>> +
>>       if (!IsPad(priv))
>>       {
>>               wcmInitialToolSize(pInfo);
>
> This will register any wacom device as a direct touch device, even pens
> and erasers.
>
> Tom

Good catch! I'm not sure if it makes a difference to clients or not,
but its definitely not what I had intended. Enclosing it in an
IsTouch(priv) check should do the trick; I'll whip up a patch when I
get back to the office in the morning.

Jason

---
When you're rife with devastation / There's a simple explanation:
You're a toymaker's creation / Trapped inside a crystal ball.
And whichever way he tilts it / Know that we must be resilient
We won't let them break our spirits / As we sing our silly song.

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to