[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