[Spice-devel] [PATCH 08/10] server/inputs_channel: use outgoing marshaller in red_channel/RedChannel

[Alon Levy]

On Thu, Jan 13, 2011 at 08:52:38AM +0100, Hans de Goede wrote:
> Hi,
> 
> Hi,
> 
> On 01/13/2011 06:01 AM, Alon Levy wrote:
> 
> <snip>
> 
> >-static void inputs_pipe_add_init(InputsChannel *channel)
> >+static void inputs_pipe_add_init(InputsChannel *inputs_channel)
> >  {
> >-    SpiceMsgInputsInit inputs_init;
> >-    InputsPipeItem *pipe_item = inputs_pipe_item_new(channel, PIPE_ITEM_INIT);
> >-    SpiceMarshaller *m = pipe_item->m;
> >+    InputsPipeItem *item = inputs_pipe_item_new(inputs_channel, PIPE_ITEM_INIT);
> >
> >-    inputs_init.keyboard_modifiers = kbd_get_leds(keyboard);
> >-    spice_marshall_msg_inputs_init(m,&inputs_init);
> >-    red_channel_pipe_add(&channel->base,&pipe_item->base);
> >+    red_channel_pipe_add(&inputs_channel->base,&item->base);
> >  }
> >
> 
> You seem to no longer be setting the keyboard_modifiers of the init message.

That can't be good. Will fix and send new (just this patch).
> 
> <snip>
> 
> Regards,
> 
> Hans
> 
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel