[PATCH] dix/events: Set currentTime to the given time stamp in NoticeTime
Keith Packard
keithp at keithp.com
Sun Apr 26 21:27:14 PDT 2015
Peter Hutterer <peter.hutterer at who-t.net> writes:
> tbh, I'm in favour of Rui's patch. That NoticeTime wasn't assigning
> currentTime is a bug and should be fixed. But one of the points of having
> NoticeTime is that we don't have to manually set the various global time
> fields. Even if (currently) only one uses it, it's a lot more maintainable
> if we have one function that handles this.
It seems odd to me that this function is almost universally passed
'currentTime', almost as if there should be a separate function which
updates device time stamps to currentTime?
But, Rui's simple patch will definitely dtrt with minimal changes.
--
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20150426/dbb9dd80/attachment.sig>
More information about the xorg-devel
mailing list