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 2506a9e..99e86a0 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) {