On Mon, 2017-01-09 at 18:14 -0800, Keith Packard wrote:

> +/* Client has requests queued or data on the network, but awaits a server 
> grab release */

80 columns max, please.

> @@ -1067,6 +1067,10 @@ AttendClient(ClientPtr client)

Additional context here:

       oc->flags &= ~OS_COMM_IGNORED;
>      set_poll_client(client);
>      if (listen_to_client(client))
>          mark_client_ready(client);
> +    else if (!(oc->flags & OS_COMM_IGNORED)) {
> +        /* grab active, mark ready when grab goes away */
> +        mark_client_saved_ready(client);
> +    }
>  }

The conditional is always true, which makes me wonder what conditional
you meant to write instead.

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

Reply via email to