From 3ace9a3333c72495b2ea674fc663473611d8144d Mon Sep 17 00:00:00 2001 From: Alon Levy Date: Fri, 17 May 2013 10:36:13 -0400 Subject: server/red_worker: simplify monitors_config update --- server/red_worker.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'server') diff --git a/server/red_worker.c b/server/red_worker.c index 14d219b0..f047d299 100644 --- a/server/red_worker.c +++ b/server/red_worker.c @@ -1279,6 +1279,9 @@ static MonitorsConfig *monitors_config_getref(MonitorsConfig *monitors_config) static void monitors_config_decref(MonitorsConfig *monitors_config) { + if (!monitors_config) { + return; + } if (--monitors_config->refs > 0) { return; } @@ -11339,9 +11342,7 @@ static void worker_update_monitors_config(RedWorker *worker, MonitorsConfig *monitors_config; int i; - if (worker->monitors_config) { - monitors_config_decref(worker->monitors_config); - } + monitors_config_decref(worker->monitors_config); spice_debug("monitors config %d(%d)", dev_monitors_config->count, @@ -11396,12 +11397,10 @@ static void set_monitors_config_to_primary(RedWorker *worker) DrawContext *context; if (!worker->surfaces[0].context.canvas) { - spice_warning("%s: no primary surface", __FUNCTION__); + spice_warning("no primary surface"); return; } - if (worker->monitors_config) { - monitors_config_decref(worker->monitors_config); - } + monitors_config_decref(worker->monitors_config); context = &worker->surfaces[0].context; worker->monitors_config = spice_malloc(sizeof(*worker->monitors_config) + sizeof(QXLHead)); -- cgit