Re: [Spice-devel] [PATCH spice-gtk] spice-gtk: Use time comparisons that still work after wraparound

2017-07-17 Thread Christophe de Dinechin
> On 17 Jul 2017, at 10:33, Christophe Fergeau wrote: > > On Thu, Jul 13, 2017 at 04:21:36PM +0200, Christophe de Dinechin wrote: >> From: Christophe de Dinechin >> >> The mm timer is a millisecond timer that wraps around after ~49 days. >> All

Re: [Spice-devel] [PATCH spice-gtk] spice-gtk: Use time comparisons that still work after wraparound

2017-07-17 Thread Christophe Fergeau
On Thu, Jul 13, 2017 at 04:21:36PM +0200, Christophe de Dinechin wrote: > From: Christophe de Dinechin > > The mm timer is a millisecond timer that wraps around after ~49 days. > All comparisons that look like a Instead, use signed ((int)(a-b)<0), which may fail if there is

[Spice-devel] [PATCH spice-gtk] spice-gtk: Use time comparisons that still work after wraparound

2017-07-13 Thread Christophe de Dinechin
From: Christophe de Dinechin The mm timer is a millisecond timer that wraps around after ~49 days. All comparisons that look like a --- src/channel-display-gst.c | 4 ++-- src/channel-display-mjpeg.c | 4 ++-- src/channel-display.c | 2 +- src/channel-playback.c