From faf0947ebbcdbd2088d6349e936a23b24d09153b Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Mon, 12 Dec 2011 17:53:04 +0000 Subject: Remove useless if() before free() The free() function allows NULL to be passed in, so any code which puts a if() before free() is wasting time --- common/sw_canvas.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'common/sw_canvas.c') diff --git a/common/sw_canvas.c b/common/sw_canvas.c index 02934174..8eabc896 100644 --- a/common/sw_canvas.c +++ b/common/sw_canvas.c @@ -1166,9 +1166,7 @@ static void canvas_destroy(SpiceCanvas *spice_canvas) } pixman_image_unref(canvas->image); canvas_base_destroy(&canvas->base); - if (canvas->private_data) { - free(canvas->private_data); - } + free(canvas->private_data); free(canvas); } -- cgit