[libvirt] [libvirt-glib] API to save and suspend
Zeeshan Ali (Khattak)
zeeshanak at gnome.org
Tue Jan 3 15:40:11 UTC 2012
On Tue, Jan 3, 2012 at 5:35 PM, Marc-André Lureau
<marcandre.lureau at gmail.com> wrote:
>
>
> On Tue, Jan 3, 2012 at 4:22 PM, Zeeshan Ali (Khattak) <zeeshanak at gnome.org>
> wrote:
>>
>> On Tue, Jan 3, 2012 at 5:08 PM, Zeeshan Ali (Khattak)
>> <zeeshanak at gnome.org> wrote:
>> > From: "Zeeshan Ali (Khattak)" <zeeshanak at 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.
Why? gslice is *the* glib way to allocate/free fixed-sized structures.
> fyi, none of the gio data
> structs used in this case are allocated with gslice .
Weird! Perhaps there is a reason for them to not use gslice there?
> Tbh, I think it's better having the small wrapper if we keep gslice.
Why? I don't see any possible leaks here.
--
Regards,
Zeeshan Ali (Khattak)
FSF member#5124
More information about the libvir-list
mailing list