On Tue, 2016-10-18 at 10:38 +0200, Francois Gouget wrote:
> On Tue, 18 Oct 2016, Pavel Grunt wrote:
> [...]
> > > +#if !GLIB_CHECK_VERSION(2,30,0)
> > > +static inline gboolean
> > > +g_queue_remove_boolean(GQueue *queue, gconstpointer data)
> > > +{
> > > + GList *link = g_queue_find(queue, data);
> > > + if (!link) {
> > > + return FALSE;
> > > + }
> > > + g_queue_unlink(queue, link);
> > > + return TRUE;
> > > +}
> > > +#define g_queue_remove g_queue_remove_boolean
> >
> > glib changed the function ?
>
> g_queue_remove() used to return void, thus causing a compilation
> error
> where we assigned / returned its value.
>
I see, I don't like these api changes.. Even worse without updating
the documentation for the function.
Ack for the patch
(I don't have a el6 machine where i could test these stuff)
Pavel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel