From 2e326085ad8bcafa356d5a71ceddbbb5fbd09ce6 Mon Sep 17 00:00:00 2001 From: Jonathon Jongsma Date: Thu, 19 May 2016 14:04:57 -0500 Subject: [PATCH] Simplify DisplayChannel hold_item() Since all pipe items implement refcounting now, just call red_pipe_item_ref() and eliminate the switch statement. --- server/display-channel.c | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/server/display-channel.c b/server/display-channel.c index 1487f100..3f414fd9 100644 --- a/server/display-channel.c +++ b/server/display-channel.c @@ -1934,16 +1934,7 @@ static void hold_item(RedChannelClient *rcc, RedPipeItem *item) { spice_return_if_fail(item); - switch (item->type) { - case RED_PIPE_ITEM_TYPE_DRAW: - case RED_PIPE_ITEM_TYPE_IMAGE: - case RED_PIPE_ITEM_TYPE_STREAM_CLIP: - case RED_PIPE_ITEM_TYPE_UPGRADE: - red_pipe_item_ref(item); - break; - default: - spice_warn_if_reached(); - } + red_pipe_item_ref(item); } static int handle_migrate_flush_mark(RedChannelClient *rcc)