Hi, On Thu, Sep 01, 2016 at 06:26:18AM -0400, Frediano Ziglio wrote: > > > > Prepare for port to GObject > > Honestly I prefer the old name. > DISPLAY_CHANNEL_CLIENT looks like you can convert anything to a DisplayChannelClient, > I would expect a NULL check for every conversion. > > Frediano I'm guessing that this macro will use G_TYPE_CHECK_INSTANCE_CAST later on? If yes, NULL check (and warning in case cast is wrong) should be covered. Maybe this patch can be introduced with the gobjectification of dcc? > > > --- > > server/dcc-send.c | 86 > > ++++++++++++++++++++++++------------------------ > > server/dcc.c | 6 ++-- > > server/dcc.h | 2 +- > > server/display-channel.c | 4 +-- > > 4 files changed, 49 insertions(+), 49 deletions(-) > > > > diff --git a/server/dcc-send.c b/server/dcc-send.c > > index d6b9b41..ecf0938 100644 > > --- a/server/dcc-send.c > > +++ b/server/dcc-send.c > > @@ -133,7 +133,7 @@ static int is_surface_area_lossy(DisplayChannelClient > > *dcc, uint32_t surface_id, > > static int is_bitmap_lossy(RedChannelClient *rcc, SpiceImage *image, > > SpiceRect *area, > > Drawable *drawable, BitmapData *out_data) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > > > if (image == NULL) { > > // self bitmap > > @@ -197,7 +197,7 @@ static void > > red_display_add_image_to_pixmap_cache(RedChannelClient *rcc, > > SpiceImage *image, > > SpiceImage *io_image, > > int is_lossy) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > DisplayChannel *display_channel = DCC_TO_DC(dcc); > > > > if ((image->descriptor.flags & SPICE_IMAGE_FLAGS_CACHE_ME)) { > > @@ -242,7 +242,7 @@ static void > > marshal_sub_msg_inval_list_wait(SpiceMarshaller *m, > > /* use legacy SpiceDataHeader (with sub_list) */ > > static void send_free_list_legacy(RedChannelClient *rcc) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > FreeList *free_list = &dcc->priv->send_data.free_list; > > SpiceMarshaller *marshaller; > > int sub_list_len = 1; > > @@ -273,7 +273,7 @@ static void send_free_list_legacy(RedChannelClient *rcc) > > /* use mini header and SPICE_MSG_LIST */ > > static void send_free_list(RedChannelClient *rcc) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > FreeList *free_list = &dcc->priv->send_data.free_list; > > int sub_list_len = 1; > > SpiceMarshaller *urgent_marshaller; > > @@ -508,7 +508,7 @@ static FillBitsType fill_bits(DisplayChannelClient *dcc, > > SpiceMarshaller *m, > > static void fill_mask(RedChannelClient *rcc, SpiceMarshaller *m, > > SpiceImage *mask_bitmap, Drawable *drawable) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > > > if (mask_bitmap && m) { > > if (dcc->priv->image_compression != SPICE_IMAGE_COMPRESSION_OFF) { > > @@ -540,7 +540,7 @@ static void marshall_qxl_draw_fill(RedChannelClient *rcc, > > { > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMarshaller *brush_pat_out; > > SpiceMarshaller *mask_bitmap_out; > > SpiceFill fill; > > @@ -760,7 +760,7 @@ static void > > red_add_lossless_drawable_dependencies(RedChannelClient *rcc, > > SpiceRect *deps_areas[], > > int num_deps) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > DisplayChannel *display = DCC_TO_DC(dcc); > > RedDrawable *drawable = item->red_drawable; > > int sync_rendered = FALSE; > > @@ -827,7 +827,7 @@ static void > > red_lossy_marshall_qxl_draw_fill(RedChannelClient *rcc, > > SpiceMarshaller *m, > > RedDrawablePipeItem *dpi) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > > > @@ -885,7 +885,7 @@ static FillBitsType > > red_marshall_qxl_draw_opaque(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi, > > int src_allowed_lossy) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *brush_pat_out; > > @@ -918,7 +918,7 @@ static void > > red_lossy_marshall_qxl_draw_opaque(RedChannelClient *rcc, > > SpiceMarshaller *m, > > RedDrawablePipeItem *dpi) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > > > @@ -984,7 +984,7 @@ static FillBitsType > > red_marshall_qxl_draw_copy(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi, > > int src_allowed_lossy) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *src_bitmap_out; > > @@ -1010,7 +1010,7 @@ static void > > red_lossy_marshall_qxl_draw_copy(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller, > > RedDrawablePipeItem *dpi) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > int has_mask = !!drawable->u.copy.mask.bitmap; > > @@ -1035,7 +1035,7 @@ static void > > red_marshall_qxl_draw_transparent(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller, > > RedDrawablePipeItem *dpi) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Drawable *item = dpi->drawable; > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *src_bitmap_out; > > @@ -1084,7 +1084,7 @@ static FillBitsType > > red_marshall_qxl_draw_alpha_blend(RedChannelClient *rcc, > > int src_allowed_lossy) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *src_bitmap_out; > > SpiceAlphaBlend alpha_blend; > > @@ -1108,7 +1108,7 @@ static void > > red_lossy_marshall_qxl_draw_alpha_blend(RedChannelClient *rcc, > > RedDrawablePipeItem > > *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int src_is_lossy; > > BitmapData src_bitmap_data; > > @@ -1150,7 +1150,7 @@ static void > > red_lossy_marshall_qxl_copy_bits(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceRect src_rect; > > int horz_offset; > > @@ -1179,7 +1179,7 @@ static void > > red_marshall_qxl_draw_blend(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *src_bitmap_out; > > SpiceMarshaller *mask_bitmap_out; > > @@ -1203,7 +1203,7 @@ static void > > red_lossy_marshall_qxl_draw_blend(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int src_is_lossy; > > BitmapData src_bitmap_data; > > @@ -1265,7 +1265,7 @@ static void > > red_lossy_marshall_qxl_draw_blackness(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int has_mask = !!drawable->u.blackness.mask.bitmap; > > > > @@ -1299,7 +1299,7 @@ static void > > red_lossy_marshall_qxl_draw_whiteness(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int has_mask = !!drawable->u.whiteness.mask.bitmap; > > > > @@ -1339,7 +1339,7 @@ static void red_marshall_qxl_draw_rop3(RedChannelClient > > *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceRop3 rop3; > > SpiceMarshaller *src_bitmap_out; > > @@ -1368,7 +1368,7 @@ static void > > red_lossy_marshall_qxl_draw_rop3(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int src_is_lossy; > > BitmapData src_bitmap_data; > > @@ -1423,7 +1423,7 @@ static void > > red_marshall_qxl_draw_composite(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceMarshaller *src_bitmap_out; > > SpiceMarshaller *mask_bitmap_out; > > @@ -1448,7 +1448,7 @@ static void > > red_lossy_marshall_qxl_draw_composite(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int src_is_lossy; > > BitmapData src_bitmap_data; > > @@ -1504,7 +1504,7 @@ static void > > red_marshall_qxl_draw_stroke(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceStroke stroke; > > SpiceMarshaller *brush_pat_out; > > @@ -1529,7 +1529,7 @@ static void > > red_lossy_marshall_qxl_draw_stroke(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int brush_is_lossy; > > BitmapData brush_bitmap_data; > > @@ -1584,7 +1584,7 @@ static void red_marshall_qxl_draw_text(RedChannelClient > > *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > SpiceText text; > > SpiceMarshaller *brush_pat_out; > > @@ -1611,7 +1611,7 @@ static void > > red_lossy_marshall_qxl_draw_text(RedChannelClient *rcc, > > RedDrawablePipeItem *dpi) > > { > > Drawable *item = dpi->drawable; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedDrawable *drawable = item->red_drawable; > > int fg_is_lossy; > > BitmapData fg_bitmap_data; > > @@ -1683,7 +1683,7 @@ static int red_marshall_stream_data(RedChannelClient > > *rcc, > > SpiceMarshaller *base_marshaller, > > Drawable *drawable) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > DisplayChannel *display = DCC_TO_DC(dcc); > > Stream *stream = drawable->stream; > > SpiceCopy *copy; > > @@ -1833,7 +1833,7 @@ static void > > display_channel_marshall_migrate_data(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller) > > { > > DisplayChannel *display_channel; > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > ImageEncoders *encoders = dcc_get_encoders(dcc); > > SpiceMigrateDataDisplay display_data = {0,}; > > > > @@ -1869,7 +1869,7 @@ static void > > display_channel_marshall_migrate_data(RedChannelClient *rcc, > > static void display_channel_marshall_pixmap_sync(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMsgWaitForChannels wait; > > PixmapCache *pixmap_cache; > > > > @@ -1921,7 +1921,7 @@ static void dcc_pixmap_cache_reset(DisplayChannelClient > > *dcc, SpiceMsgWaitForCha > > static void display_channel_marshall_reset_cache(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMsgWaitForChannels wait; > > > > red_channel_client_init_send_data(rcc, > > SPICE_MSG_DISPLAY_INVAL_ALL_PIXMAPS, NULL); > > @@ -1935,7 +1935,7 @@ static void red_marshall_image(RedChannelClient *rcc, > > SpiceMarshaller *m, > > RedImageItem *item) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > DisplayChannel *display = DCC_TO_DC(dcc); > > SpiceImage red_image; > > SpiceBitmap bitmap; > > @@ -2158,7 +2158,7 @@ static void marshall_qxl_drawable(RedChannelClient > > *rcc, > > static void marshall_stream_start(RedChannelClient *rcc, > > SpiceMarshaller *base_marshaller, > > StreamAgent *agent) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > Stream *stream = agent->stream; > > > > agent->last_send_time = 0; > > @@ -2200,7 +2200,7 @@ static void marshall_stream_clip(RedChannelClient *rcc, > > SpiceMarshaller *base_marshaller, > > RedStreamClipItem *item) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > StreamAgent *agent = item->stream_agent; > > > > spice_return_if_fail(agent->stream); > > @@ -2218,7 +2218,7 @@ static void marshall_stream_clip(RedChannelClient *rcc, > > static void marshall_stream_end(RedChannelClient *rcc, > > SpiceMarshaller *base_marshaller, > > StreamAgent* agent) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMsgDisplayStreamDestroy destroy; > > > > red_channel_client_init_send_data(rcc, SPICE_MSG_DISPLAY_STREAM_DESTROY, > > NULL); > > @@ -2230,7 +2230,7 @@ static void marshall_stream_end(RedChannelClient *rcc, > > static void marshall_upgrade(RedChannelClient *rcc, SpiceMarshaller *m, > > RedUpgradeItem *item) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > RedChannel *channel = red_channel_client_get_channel(rcc); > > RedDrawable *red_drawable; > > SpiceMsgDisplayDrawCopy copy; > > @@ -2260,7 +2260,7 @@ static void marshall_surface_create(RedChannelClient > > *rcc, > > SpiceMarshaller *base_marshaller, > > SpiceMsgSurfaceCreate *surface_create) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > > > region_init(&dcc->priv->surface_client_lossy_region[surface_create->surface_id]); > > red_channel_client_init_send_data(rcc, SPICE_MSG_DISPLAY_SURFACE_CREATE, > > NULL); > > @@ -2271,7 +2271,7 @@ static void marshall_surface_create(RedChannelClient > > *rcc, > > static void marshall_surface_destroy(RedChannelClient *rcc, > > SpiceMarshaller *base_marshaller, > > uint32_t surface_id) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMsgSurfaceDestroy surface_destroy; > > > > region_destroy(&dcc->priv->surface_client_lossy_region[surface_id]); > > @@ -2313,7 +2313,7 @@ static void > > marshall_stream_activate_report(RedChannelClient *rcc, > > SpiceMarshaller > > *base_marshaller, > > uint32_t stream_id) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > StreamAgent *agent = &dcc->priv->stream_agents[stream_id]; > > SpiceMsgDisplayStreamActivateReport msg; > > > > @@ -2329,7 +2329,7 @@ static void marshall_gl_scanout(RedChannelClient *rcc, > > SpiceMarshaller *m, > > RedPipeItem *item) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > DisplayChannel *display_channel = DCC_TO_DC(dcc); > > QXLInstance* qxl = display_channel->common.qxl; > > > > @@ -2354,7 +2354,7 @@ static void marshall_gl_draw(RedChannelClient *rcc, > > > > static void begin_send_message(RedChannelClient *rcc) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > FreeList *free_list = &dcc->priv->send_data.free_list; > > > > if (free_list->res->count) { > > @@ -2389,7 +2389,7 @@ static void reset_send_data(DisplayChannelClient *dcc) > > > > void dcc_send_item(RedChannelClient *rcc, RedPipeItem *pipe_item) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > SpiceMarshaller *m = red_channel_client_get_marshaller(rcc); > > > > reset_send_data(dcc); > > diff --git a/server/dcc.c b/server/dcc.c > > index 9972d75..16d8acd 100644 > > --- a/server/dcc.c > > +++ b/server/dcc.c > > @@ -590,7 +590,7 @@ RedPipeItem *dcc_gl_scanout_item_new(RedChannelClient > > *rcc, void *data, int num) > > > > RedPipeItem *dcc_gl_draw_item_new(RedChannelClient *rcc, void *data, int > > num) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > const SpiceMsgDisplayGlDraw *draw = data; > > RedGlDrawItem *item = spice_new(RedGlDrawItem, 1); > > spice_return_val_if_fail(item != NULL, NULL); > > @@ -990,7 +990,7 @@ static int dcc_handle_gl_draw_done(DisplayChannelClient > > *dcc) > > > > int dcc_handle_message(RedChannelClient *rcc, uint32_t size, uint16_t type, > > void *msg) > > { > > - DisplayChannelClient *dcc = RCC_TO_DCC(rcc); > > + DisplayChannelClient *dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > > > switch (type) { > > case SPICE_MSGC_DISPLAY_INIT: > > @@ -1186,7 +1186,7 @@ int dcc_config_socket(RedChannelClient *rcc) > > RedClient *client = red_channel_client_get_client(rcc); > > MainChannelClient *mcc = red_client_get_main(client); > > > > - RCC_TO_DCC(rcc)->is_low_bandwidth = > > main_channel_client_is_low_bandwidth(mcc); > > + DISPLAY_CHANNEL_CLIENT(rcc)->is_low_bandwidth = > > main_channel_client_is_low_bandwidth(mcc); > > > > return common_channel_config_socket(rcc); > > } > > diff --git a/server/dcc.h b/server/dcc.h > > index 0659ce7..d08e413 100644 > > --- a/server/dcc.h > > +++ b/server/dcc.h > > @@ -60,7 +60,7 @@ typedef struct FreeList { > > typedef struct DisplayChannelClient DisplayChannelClient; > > > > #define DCC_TO_DC(dcc) > > ((DisplayChannel*)red_channel_client_get_channel((RedChannelClient*)dcc)) > > -#define RCC_TO_DCC(rcc) ((DisplayChannelClient*)rcc) > > +#define DISPLAY_CHANNEL_CLIENT(rcc) ((DisplayChannelClient*)rcc) > > > > typedef struct RedSurfaceCreateItem { > > RedPipeItem pipe_item; > > diff --git a/server/display-channel.c b/server/display-channel.c > > index 74c7dc0..108e69b 100644 > > --- a/server/display-channel.c > > +++ b/server/display-channel.c > > @@ -1844,7 +1844,7 @@ static void on_disconnect(RedChannelClient *rcc) > > spice_info(NULL); > > spice_return_if_fail(rcc != NULL); > > > > - dcc = RCC_TO_DCC(rcc); > > + dcc = DISPLAY_CHANNEL_CLIENT(rcc); > > display = DCC_TO_DC(dcc); > > > > dcc_stop(dcc); // TODO: start/stop -> connect/disconnect? > > @@ -1875,7 +1875,7 @@ static uint64_t > > handle_migrate_data_get_serial(RedChannelClient *rcc, uint32_t s > > > > static int handle_migrate_data(RedChannelClient *rcc, uint32_t size, void > > *message) > > { > > - return dcc_handle_migrate_data(RCC_TO_DCC(rcc), size, message); > > + return dcc_handle_migrate_data(DISPLAY_CHANNEL_CLIENT(rcc), size, > > message); > > } > > > > static SpiceCanvas *image_surfaces_get(SpiceImageSurfaces *surfaces, > > uint32_t surface_id) > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel