diff options
| author | Alon Levy <alevy@redhat.com> | 2011-01-25 00:38:16 +0200 |
|---|---|---|
| committer | Alon Levy <alevy@redhat.com> | 2011-01-27 12:48:35 +0200 |
| commit | 143452477df2a095406f69a154d1056953fda229 (patch) | |
| tree | d48e096b6b9851d25aeafb9f30b5855dc62f5489 | |
| parent | bd1ae4a6e14b0b141ce4ecacbc7b148ec4658129 (diff) | |
| download | spice-143452477df2a095406f69a154d1056953fda229.tar.gz spice-143452477df2a095406f69a154d1056953fda229.tar.xz spice-143452477df2a095406f69a154d1056953fda229.zip | |
common/sw_canvas: remove unused error val
This is the only unused var change I'll want to revisit eventually,
I'm submitting anyway since it doesn't change current behavior. I'm
talking about ignoring the return value from canvas creation. Adding
a print is possible but I didn't test (may be too verbose, also
preferable to be a debug print if so, and we don't have that option
in the code atm - probably an environment variable will do, or adding
some spice_server_set_logging_level api, maybe even
spice_server_set_logging_fd?)
| -rw-r--r-- | common/sw_canvas.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/common/sw_canvas.c b/common/sw_canvas.c index f579b4cb..9891d851 100644 --- a/common/sw_canvas.c +++ b/common/sw_canvas.c @@ -1180,7 +1180,6 @@ static SpiceCanvas *canvas_create_common(pixman_image_t *image, ) { SwCanvas *canvas; - int init_ok; if (need_init) { return NULL; @@ -1189,7 +1188,7 @@ static SpiceCanvas *canvas_create_common(pixman_image_t *image, spice_surface_format_to_pixman (format)); canvas = spice_new0(SwCanvas, 1); - init_ok = canvas_base_init(&canvas->base, &sw_canvas_ops, + canvas_base_init(&canvas->base, &sw_canvas_ops, pixman_image_get_width (image), pixman_image_get_height (image), format |