When compiling spice with make CFLAGS="-g3 -ggdb3 -O0 -Wall -Werror", the build broken because of a few unused variables/missing returns. This patch fixes these warnings. --- server/tests/basic_event_loop.c | 2 +- server/tests/test_display_base.c | 16 +--------------- server/tests/test_empty_success.c | 3 +++ 3 files changed, 5 insertions(+), 16 deletions(-)
diff --git a/server/tests/basic_event_loop.c b/server/tests/basic_event_loop.c index 7e8a1df..2fe1b69 100644 --- a/server/tests/basic_event_loop.c +++ b/server/tests/basic_event_loop.c @@ -319,7 +319,7 @@ void basic_event_loop_mainloop(void) if ((next_timer = get_next_timer()) != NULL) { calc_next_timeout(next_timer, &next_timer_timeout); timeout = &next_timer_timeout; - DPRINTF(1, "timeout of %d.%06d", + DPRINTF(1, "timeout of %zd.%06zd", timeout->tv_sec, timeout->tv_usec); } else { timeout = NULL; diff --git a/server/tests/test_display_base.c b/server/tests/test_display_base.c index c87d2c7..9cb7ad0 100644 --- a/server/tests/test_display_base.c +++ b/server/tests/test_display_base.c @@ -40,7 +40,6 @@ static void test_spice_destroy_update(SimpleSpiceUpdate *update) #define SINGLE_PART 8 static const int angle_parts = 64 / SINGLE_PART; -static int angle = 0; static int unique = 1; static int color = -1; static int c_i = 0; @@ -354,7 +353,6 @@ static int num_simple_commands = 0; static void produce_command() { static int target_surface = 0; - static int simple_command_index = 0; static int cmd_index = 0; ASSERT(num_simple_commands); @@ -449,19 +447,6 @@ static struct { uint8_t data[CURSOR_WIDTH * CURSOR_HEIGHT * 4]; // 32bit per pixel } cursor; -static void init_cursor() -{ - cursor.cursor.header.unique = 0; // TODO ?? - cursor.cursor.header.type = SPICE_CURSOR_TYPE_COLOR32; - cursor.cursor.header.width = CURSOR_WIDTH; - cursor.cursor.header.height = CURSOR_HEIGHT; - cursor.cursor.header.hot_spot_x = 0; - cursor.cursor.header.hot_spot_y = 0; - cursor.cursor.data_size = CURSOR_WIDTH * CURSOR_HEIGHT; - cursor.cursor.chunk.data_size = cursor.cursor.data_size; - cursor.cursor.chunk.prev_chunk = cursor.cursor.chunk.next_chunk = 0; -} - static int get_cursor_command(QXLInstance *qin, struct QXLCommandExt *ext) { static int color = 0; @@ -515,6 +500,7 @@ static void notify_update(QXLInstance *qin, uint32_t update_id) static int flush_resources(QXLInstance *qin) { printf("%s\n", __func__); + return TRUE; } QXLInterface display_sif = { diff --git a/server/tests/test_empty_success.c b/server/tests/test_empty_success.c index e747e40..97aa772 100644 --- a/server/tests/test_empty_success.c +++ b/server/tests/test_empty_success.c @@ -1,8 +1,10 @@ +#include <stdlib.h> #include <strings.h> #include <spice.h> SpiceTimer* timer_add(SpiceTimerFunc func, void *opaque) { + return NULL; } void timer_start(SpiceTimer *timer, uint32_t ms) @@ -19,6 +21,7 @@ void timer_remove(SpiceTimer *timer) SpiceWatch *watch_add(int fd, int event_mask, SpiceWatchFunc func, void *opaque) { + return NULL; } void watch_update_mask(SpiceWatch *watch, int event_mask) -- 1.7.4 _______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel