On Tue, Jan 3, 2012 at 4:22 PM, Zeeshan Ali (Khattak)
<zeesha...@gnome.org>wrote:

> On Tue, Jan 3, 2012 at 5:08 PM, Zeeshan Ali (Khattak)
> <zeesha...@gnome.org> wrote:
> > From: "Zeeshan Ali (Khattak)" <zeesha...@gnome.org>
> > +    res = g_simple_async_result_new(G_OBJECT(dom),
> > +                                    callback,
> > +                                    user_data,
> > +                                    gvir_domain_saved_suspend);
> > +    g_simple_async_result_set_op_res_gpointer (res, data, NULL);
>
>   The reason I'm not using destroy notify argument here is that I
> can't: g_slice_free*() takes 2 arguments and thats not what
> DestroyNotify expects. I could write a wrapper function but I don't
> see the point.


Oh, I see. Perhaps we don't need gslice then. fyi, none of the gio data
structs used in this case are allocated with gslice  .

Tbh, I think it's better having the small wrapper if we keep gslice.

-- 
Marc-André Lureau
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to