I'd just squash this one in the previous patch (if you think it's needed, with a note in the commit log that the added basic_event_loop_destroy() addition allows to see that this leak is gone). But feel free to keep it separate if you think it's best. Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> Christophe On Wed, Mar 01, 2017 at 05:09:55PM +0000, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > --- > server/tests/test-leaks.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/server/tests/test-leaks.c b/server/tests/test-leaks.c > index cd96bc8..b8521f4 100644 > --- a/server/tests/test-leaks.c > +++ b/server/tests/test-leaks.c > @@ -47,6 +47,7 @@ static void leaks(void) > g_assert_cmpint(result, ==, 0); > > spice_server_destroy(server); > + basic_event_loop_destroy(); > } > > int main(int argc, char *argv[]) > -- > 2.9.3 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel