>
> On Thu, Apr 12, 2018 at 11:09:39AM +0200, Christophe Fergeau wrote:
> > On Wed, Apr 11, 2018 at 01:16:57PM -0400, Frediano Ziglio wrote:
> > > >
> > > > The name is more consistent with red_marshall_cursor_init.
> > > > ---
> > > > server/cursor-channel.c | 8 ++++----
> > > > 1 file changed, 4 insertions(+), 4 deletions(-)
> > > >
> > >
> > > Why command (so red_marshall_cursor) ?
> > > The item is RED_PIPE_ITEM_TYPE_CURSOR.
> >
> > I find red_marshall_cursor misleading, especially with
> > red_marshall_cursor_init. It might make sense it this was always setting
> > the cursor shape, but this is not even the case, this might cause cursor
> > to move, or to be hidden, .... This pipe item really wraps a
> > RedCursorCommand. I can rename it too.
>
> I can squash this in, the alternative is probably a lot more changes
> than it's worth ;)
>
Yes, was thinking the same, would require lot of changes.
Acked.
> diff --git a/server/cursor-channel.c b/server/cursor-channel.c
> index a549c5fe0..ada1d62a7 100644
> --- a/server/cursor-channel.c
> +++ b/server/cursor-channel.c
> @@ -142,9 +142,9 @@ static void red_marshall_cursor_init(CursorChannelClient
> *ccc, SpiceMarshaller *
> spice_marshall_msg_cursor_init(base_marshaller, &msg);
> }
>
> -static void red_marshall_cursor_command(CursorChannelClient *ccc,
> - SpiceMarshaller *m,
> - RedCursorPipeItem *cursor_pipe_item)
> +static void red_marshall_cursor(CursorChannelClient *ccc,
> + SpiceMarshaller *m,
> + RedCursorPipeItem *cursor_pipe_item)
> {
> RedChannelClient *rcc = RED_CHANNEL_CLIENT(ccc);
> CursorChannel *cursor_channel =
> CURSOR_CHANNEL(red_channel_client_get_channel(rcc));
> @@ -212,7 +212,7 @@ static void cursor_channel_send_item(RedChannelClient
> *rcc, RedPipeItem *pipe_it
>
> switch (pipe_item->type) {
> case RED_PIPE_ITEM_TYPE_CURSOR:
> - red_marshall_cursor_command(ccc, m, SPICE_UPCAST(RedCursorPipeItem,
> pipe_item));
> + red_marshall_cursor(ccc, m, SPICE_UPCAST(RedCursorPipeItem,
> pipe_item));
> break;
> case RED_PIPE_ITEM_TYPE_INVAL_ONE:
> red_marshall_inval(rcc, m, SPICE_CONTAINEROF(pipe_item,
> RedCacheItem, u.pipe_data));
>
> Christophe
>
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel