On Wed, Jun 22, 2011 at 10:17:32PM -0500, Chris Bagwell wrote:
> On Wed, Jun 22, 2011 at 8:01 PM, Peter Hutterer
> <[email protected]> wrote:
> > On Wed, Jun 22, 2011 at 07:18:13PM -0500, [email protected] wrote:
[blah]
> >> @@ -255,7 +255,7 @@ int wcmDeviceTypeKeys(InputInfoPtr pInfo)
> >>               case 0xE2: /* TPC with 2FGT */
> >>               case 0xE3: /* TPC with 2FGT */
> >>               case 0xE6: /* TPC with 2FGT */
> >> -                     priv->common->tablet_type = WCM_TPC;
> >> +                     priv->common->tablet_type |= WCM_TPC;
> >>                       break;
> >>
> >>               case 0x93: /* TPC with 1FGT */
> >> --
> >> 1.7.5.2
> >
> > yes, but no :) I'd rather have a macro/function to do this for us.
> >
> 
> Sure thing.  I couldn't bring myself to create a TabletSetFeature()
> since I saw perfectly good MaskSet() already defined in
> xf86WacomDefs.h.  Let me know if you prefer otherwise.

IMO if we have a TabletHasFeature, we should be symmetrical with
TabletSetFeature. No reason _that_ can't use MaskSet though :)
 
Cheers,
  Peter

------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Data protection magic?
Nope - It's vRanger. Get your free trial download today.
http://p.sf.net/sfu/quest-sfdev2dev
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to