On Fri, Nov 27, 2015 at 03:41:20PM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau <marcandre.lur...@redhat.com> > > The prepare callback needs to be implemented with glib < 2.36. > > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
But nothing prevents us from implementing it for all versions. > --- > tests/vhost-user-test.c | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/tests/vhost-user-test.c b/tests/vhost-user-test.c > index 29205ed..4ab48e4 100644 > --- a/tests/vhost-user-test.c > +++ b/tests/vhost-user-test.c > @@ -506,8 +506,19 @@ test_migrate_source_check(GSource *source) > return FALSE; > } > > +#if !GLIB_CHECK_VERSION(2,36,0) > +static gboolean > +test_migrate_source_prepare(GSource *source, gint *timeout) > +{ > + *timeout = -1; > + return FALSE; > +} > +#endif > + > GSourceFuncs test_migrate_source_funcs = { > - NULL, > +#if !GLIB_CHECK_VERSION(2,36,0) > + test_migrate_source_prepare, > +#endif > test_migrate_source_check, So now for version 2.36 check will be used instead of prepare. That's clearly wrong. I'll switch this to named initializers to avoid the mess. > NULL, > NULL > -- > 2.5.0