summaryrefslogtreecommitdiffstats
path: root/common/gdi_canvas.c
diff options
context:
space:
mode:
authorDaniel P. Berrange <berrange@redhat.com>2011-12-12 17:53:04 +0000
committerMarc-André Lureau <marcandre.lureau@gmail.com>2012-03-20 15:25:55 +0100
commit4a9f7971fceb1296922740aab27df819fce101bf (patch)
treeada969e1ffcee83d8b173e638cc29582194aacac /common/gdi_canvas.c
parentbcdf929c79e681eedd757a9bf2219f6607dac0fd (diff)
downloadspice-common-4a9f7971fceb1296922740aab27df819fce101bf.tar.gz
spice-common-4a9f7971fceb1296922740aab27df819fce101bf.tar.xz
spice-common-4a9f7971fceb1296922740aab27df819fce101bf.zip
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
Diffstat (limited to 'common/gdi_canvas.c')
-rw-r--r--common/gdi_canvas.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/common/gdi_canvas.c b/common/gdi_canvas.c
index 1a46ac1..5c9cdf0 100644
--- a/common/gdi_canvas.c
+++ b/common/gdi_canvas.c
@@ -1773,9 +1773,7 @@ static void gdi_canvas_draw_stroke(SpiceCanvas *spice_canvas, SpiceRect *bbox, S
}
#endif
- if (user_style) {
- free(user_style);
- }
+ free(user_style);
}
static void gdi_canvas_clear(SpiceCanvas *spice_canvas)