Add a fixme: if we're dropping sync event, libedev loses state info too

If a client doesn't care about syncing that's one thing, but
libevdev functions still need to return the right _current_ (i.e. synced)
value.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
Peter Hutterer 2013-05-31 15:01:32 +10:00
parent 20c4c73adf
commit 897dfba2ef

View file

@ -541,8 +541,12 @@ int libevdev_next_event(struct libevdev *dev, unsigned int flags, struct input_e
if (rc != 0)
return rc;
}
} else if (dev->need_sync)
} else if (dev->need_sync) {
/* FIXME: still need to call update_state for all events
* here, otherwise the library has the wrong view of the
* device too */
queue_shift_multiple(dev, dev->queue_nsync, NULL);
}
/* Always read in some more events. Best case this smoothes over a potential SYN_DROPPED,
worst case we don't read fast enough and end up with SYN_DROPPED anyway */