[Spice-devel] [PATCH 10/18] worker: remove display_channel prefix from channel callbacks

Jonathon Jongsma jjongsma at redhat.com
Mon Nov 23 13:55:33 PST 2015


On Mon, 2015-11-23 at 17:01 +0000, Frediano Ziglio wrote:
> From: Marc-André Lureau <marcandre.lureau at gmail.com>
> 
> ---
>  server/red_worker.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/server/red_worker.c b/server/red_worker.c
> index ac1ef0c..656f9ab 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -3847,7 +3847,7 @@ static void
> red_marshall_stream_activate_report(RedChannelClient *rcc,
>      spice_marshall_msg_display_stream_activate_report(base_marshaller, &msg);
>  }
>  
> -static void display_channel_send_item(RedChannelClient *rcc, PipeItem
> *pipe_item)
> +static void send_item(RedChannelClient *rcc, PipeItem *pipe_item)
>  {
>      SpiceMarshaller *m = red_channel_client_get_marshaller(rcc);
>      DisplayChannelClient *dcc = RCC_TO_DCC(rcc);
> @@ -3946,7 +3946,7 @@ static inline void red_push(RedWorker *worker)
>      }
>  }
>  
> -static void display_channel_client_on_disconnect(RedChannelClient *rcc)
> +static void on_disconnect(RedChannelClient *rcc)
>  {
>      DisplayChannel *display;
>      DisplayChannelClient *dcc = RCC_TO_DCC(rcc);
> @@ -4223,7 +4223,7 @@ static inline void flush_all_qxl_commands(RedWorker
> *worker)
>      flush_cursor_commands(worker);
>  }
>  
> -static int display_channel_handle_migrate_mark(RedChannelClient *rcc)
> +static int handle_migrate_flush_mark(RedChannelClient *rcc)
>  {
>      DisplayChannel *display_channel = SPICE_CONTAINEROF(rcc->channel,
> DisplayChannel, common.base);
>      RedChannel *channel = RED_CHANNEL(display_channel);
> @@ -4428,7 +4428,7 @@ RedChannel *red_worker_new_channel(RedWorker *worker,
> int size,
>      return channel;
>  }
>  
> -static void display_channel_hold_pipe_item(RedChannelClient *rcc, PipeItem
> *item)
> +static void hold_item(RedChannelClient *rcc, PipeItem *item)
>  {
>      spice_assert(item);
>      switch (item->type) {
> @@ -4449,7 +4449,7 @@ static void
> display_channel_hold_pipe_item(RedChannelClient *rcc, PipeItem *item
>      }
>  }
>  
> -static void display_channel_release_item(RedChannelClient *rcc, PipeItem
> *item, int item_pushed)
> +static void release_item(RedChannelClient *rcc, PipeItem *item, int
> item_pushed)
>  {
>      DisplayChannelClient *dcc = RCC_TO_DCC(rcc);
>  
> @@ -4462,11 +4462,11 @@ static void display_channel_create(RedWorker *worker,
> int migrate, int stream_vi
>  {
>      DisplayChannel *display_channel;
>      ChannelCbs cbs = {
> -        .on_disconnect = display_channel_client_on_disconnect,
> -        .send_item = display_channel_send_item,
> -        .hold_item = display_channel_hold_pipe_item,
> -        .release_item = display_channel_release_item,
> -        .handle_migrate_flush_mark = display_channel_handle_migrate_mark,
> +        .on_disconnect = on_disconnect,
> +        .send_item = send_item,
> +        .hold_item = hold_item,
> +        .release_item = release_item,
> +        .handle_migrate_flush_mark = handle_migrate_flush_mark,
>          .handle_migrate_data = handle_migrate_data,
>          .handle_migrate_data_get_serial = handle_migrate_data_get_serial
>      };


I don't really see any reason to change these function names, but OK.

Acked-by: Jonathon Jongsma <jjongsma at redhat.com>


More information about the Spice-devel mailing list