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