[Spice-devel] [PATCH] tests: fix compilation with -Wall -Werror

[Alon Levy]

On Mon, Mar 21, 2011 at 05:24:54PM +0200, Alon Levy wrote:
> On Mon, Mar 21, 2011 at 04:20:29PM +0100, Christophe Fergeau wrote:
> > 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.
> 

btw, while you're at it, mind trying to compile with clang?

> Just one comment inline. Looks good otherwise.
> 
> > ---
> >  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;
> > -}
> > -
> 
> Well, this is test code, I would want to get back to this eventually, so
> could you turn this into a non static (and then the compiler won't complain)
> instead? if that doesn't work even #if 0 is better. (again, test code).
> 
> >  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 at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel