On Mon, Dec 21, 2015 at 10:33:12AM +0000, Frediano Ziglio wrote:
> ---
>  server/red-worker.c | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/server/red-worker.c b/server/red-worker.c
> index b0b7df1..0347404 100644
> --- a/server/red-worker.c
> +++ b/server/red-worker.c
> @@ -71,6 +71,7 @@ struct RedWorker {
>      QXLInstance *qxl;
>      RedDispatcher *red_dispatcher;
>      int running;
> +    SpiceCoreInterfaceInternal core;
>      struct pollfd poll_fds[MAX_EVENT_SOURCES];
>      struct SpiceWatch watches[MAX_EVENT_SOURCES];
>      unsigned int event_timeout;
> @@ -534,16 +535,12 @@ static void worker_watch_update_mask(SpiceWatch *watch, 
> int event_mask)
>  static SpiceWatch *worker_watch_add(const SpiceCoreInterfaceInternal *iface,
>                                      int fd, int event_mask, SpiceWatchFunc 
> func, void *opaque)
>  {
> +    RedWorker *worker = SPICE_CONTAINEROF(iface, RedWorker, core);
>      /* Since we are a channel core implementation, we always get called from
>         red_channel_client_create(), so opaque always is our rcc */
>      RedChannelClient *rcc = opaque;

I would remove this, or at least move this to the block where it's used
(some debug statement in an error case).

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to