[Bug 134401] Re: [needs-packaging] add support for fujitsu tablet input device

[Seva Gluschenko]

the patch contains an error in this hunk:

@@ -330,8 +330,8 @@
          * Device reports motions on 2 axes in absolute coordinates.
          * Axes min and max values are reported in raw coordinates.
          */
-        if (InitValuatorClassDeviceStruct(dev, 2, xf86GetMotionEvents,
-                                          local->history_size, Absolute) == FALSE)
+        if (InitValuatorClassDeviceStruct(dev, 2,
+                                          GetMotionHistorySize(), Absolute) == FALSE)
         {
                 ErrorF ("Unable to allocate Fuji touchscreen ValuatorClassDeviceStruct\n");
                 return !Success;

actually, the xf86GetMotionEvents argument must retain. otherwise
compilation breaks up with the following error message:

fujitsu.c: In function 'DeviceInit':
fujitsu.c:334: warning: passing argument 3 of 'InitValuatorClassDeviceStruct' makes pointer from integer without a cast
fujitsu.c:334: error: too few arguments to function 'InitValuatorClassDeviceStruct'

The fixed patch is attached.

** Attachment added: "xf86-input-fujitouch-0.6.5 ubuntu patch"
   http://launchpadlibrarian.net/24969901/fujitouch.diff

-- 
[needs-packaging] add support for fujitsu tablet input device
https://bugs.launchpad.net/bugs/134401
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs at lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs