Not sure that this really needs to be split from the previous patch,
but OK
Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
On Wed, 2017-09-06 at 18:15 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> server/cursor-channel.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/server/cursor-channel.c b/server/cursor-channel.c
> index d61cf4808..fad90ad00 100644
> --- a/server/cursor-channel.c
> +++ b/server/cursor-channel.c
> @@ -30,7 +30,6 @@
>
> typedef struct RedCursorPipeItem {
> RedPipeItem base;
> - QXLInstance *qxl;
> RedCursorCmd *red_cursor;
> } RedCursorPipeItem;
>
> @@ -55,7 +54,7 @@ G_DEFINE_TYPE(CursorChannel, cursor_channel,
> TYPE_COMMON_GRAPHICS_CHANNEL)
>
> static void cursor_pipe_item_free(RedPipeItem *pipe_item);
>
> -static RedCursorPipeItem *cursor_pipe_item_new(QXLInstance *qxl,
> RedCursorCmd *cmd)
> +static RedCursorPipeItem *cursor_pipe_item_new(RedCursorCmd *cmd)
> {
> RedCursorPipeItem *item = g_new0(RedCursorPipeItem, 1);
>
> @@ -63,7 +62,6 @@ static RedCursorPipeItem
> *cursor_pipe_item_new(QXLInstance *qxl, RedCursorCmd *c
>
> red_pipe_item_init_full(&item->base, RED_PIPE_ITEM_TYPE_CURSOR,
> cursor_pipe_item_free);
> - item->qxl = qxl;
> item->red_cursor = cmd;
>
> return item;
> @@ -253,13 +251,11 @@ void cursor_channel_process_cmd(CursorChannel
> *cursor, RedCursorCmd *cursor_cmd)
> {
> RedCursorPipeItem *cursor_pipe_item;
> bool cursor_show = false;
> - QXLInstance *qxl;
>
> spice_return_if_fail(cursor);
> spice_return_if_fail(cursor_cmd);
>
> - qxl =
> common_graphics_channel_get_qxl(COMMON_GRAPHICS_CHANNEL(cursor));
> - cursor_pipe_item = cursor_pipe_item_new(qxl, cursor_cmd);
> + cursor_pipe_item = cursor_pipe_item_new(cursor_cmd);
>
> switch (cursor_cmd->type) {
> case QXL_CURSOR_SET:
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel