> > Fix compilation error due to -Werror=maybe-uninitialized: > > CC test-display-base.o > test-display-base.c: In function 'do_wakeup': > test-display-base.c:579:13: error: 'update' may be used uninitialized... > push_command(&update->ext); > > Signed-off-by: Snir Sheriber <ssheribe@xxxxxxxxxx> > --- > server/tests/test-display-base.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/server/tests/test-display-base.c > b/server/tests/test-display-base.c > index 8167df5..1a2a833 100644 > --- a/server/tests/test-display-base.c > +++ b/server/tests/test-display-base.c > @@ -573,7 +573,9 @@ static void produce_command(Test *test) > update = > test_spice_create_update_solid(command->solid.surface_id, > command->solid.bbox, command->solid.color); > break; > - default: /* Just to shut up GCC warning (-Wswitch) */ > + default: > + /* Terminate on unhandled cases - never actually reached > */ > + g_assert_not_reached(); > break; > } > push_command(&update->ext); Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx> I don't understand why g_assert_not_reached is not aligned with the break, I assume some mail artifact. Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel