[Spice-devel] [spice-server 1/3] Remove unneeded rcc->priv->latency_monitor.timer checks

[Jonathon Jongsma]

On Tue, 2017-04-11 at 17:05 +0200, Christophe Fergeau wrote:
> On Tue, Apr 11, 2017 at 08:54:15AM -0400, Frediano Ziglio wrote:
> > > 
> > > red_channel_client_start_ping_timer(),
> > > red_channel_client_cancel_ping_timer(),
> > > red_channel_client_restart_ping_timer() are no-ops when
> > > rcc->priv->latency_monitor.timer is NULL, so there is no need to
> > > explicitly check it's not NULL before calling these functions.
> > > 
> > > Signed-off-by: Christophe Fergeau <cfergeau at redhat.com>
> > 
> > Looking at code it seems the check is moved, not removed.
> 
> I've changed the commit log to:
> 
>     Remove unneeded rcc->priv->latency_monitor.timer checks
> 
>     Beforoe this commit, red_channel_client_start_ping_timer() and

"Before"

>     red_channel_client_cancel_ping_timer() had checks to be no-ops
> when
>     rcc->priv->latency_monitor.timer is NULL.
> 
>     This commit adds a similar check to
>     red_channel_client_restart_ping_timer(), and removes explicit
> NULL
>     checks before calls to
> red_channel_client_{cancel,restart,start}_ping_timer()
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel


otherwise looks fine to me.

Jonathon