Peter Hutterer <[email protected]> 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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to