Drop unnecessary memset
authorPeter Hutterer <peter.hutterer@who-t.net>
Wed, 2 Apr 2014 01:57:41 +0000 (11:57 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Thu, 3 Apr 2014 02:08:28 +0000 (12:08 +1000)
The EVICOCGMTSLOTS ioctl returns all slot values for the requested code or an
error code, it doesn't return the number of bytes successfully transferred.
Thus all values in the input array are always defined (on success), we don't
need to memset it.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
libevdev/libevdev.c

index 2506a9e751b1f187a0490b7751121986c8a9d562..99e86a042ac3efc45d00ae0e1995bc394fbcd709 100644 (file)
@@ -571,7 +571,6 @@ sync_mt_state(struct libevdev *dev, int create_events)
                if (!libevdev_has_event_code(dev, EV_ABS, axis))
                        continue;
 
-               memset(&mt_state, 0, sizeof(mt_state));
                mt_state.code = axis;
                rc = ioctl(dev->fd, EVIOCGMTSLOTS(sizeof(struct mt_state)), &mt_state);
                if (rc < 0) {