[Spice-devel] [PATCH v2 6/9] worker: remove useless check

Jonathon Jongsma jjongsma at redhat.com
Wed Jan 27 13:35:52 PST 2016


OK

Acked-by: Jonathon Jongsma <jjongsma at redhat.com>


On Tue, 2016-01-26 at 09:44 +0000, Frediano Ziglio wrote:
> display_channel is never NULL.
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/red-worker.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/server/red-worker.c b/server/red-worker.c
> index 9f967b4..dd20bd5 100644
> --- a/server/red-worker.c
> +++ b/server/red-worker.c
> @@ -320,8 +320,7 @@ static int red_process_display(RedWorker *worker, int
> *ring_is_empty)
>              spice_error("bad command type");
>          }
>          n++;
> -        if ((worker->display_channel &&
> -             red_channel_all_blocked(&worker->display_channel->common.base))
> +        if (red_channel_all_blocked(&worker->display_channel->common.base)
>              || spice_get_monotonic_time_ns() - start > NSEC_PER_SEC / 100) {
>              worker->event_timeout = 0;
>              return n;


More information about the Spice-devel mailing list