<br>Hi<div><br><div class="gmail_quote">On Fri, Dec 23, 2011 at 8:58 PM, Zeeshan Ali (Khattak) <span dir="ltr"><<a href="mailto:zeeshanak@gnome.org">zeeshanak@gnome.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":1hk">+    data = g_new0(DomainSavedSuspendData, 1);<br></div></blockquote></div><div><div><span style>+    g_slice_free (DomainSavedSuspendData, data);</span><br style></div></div><div><span style><br></span></div>
You are mixing g_new and g_slice, I don't think that's a problem, but that's not consistant and worries me.</div><div><br></div><div>g_object_{set,get}_data could probably use the _full variant, to make sure the data is released on object release.<br clear="all">
<div><font color="#222222" face="arial, sans-serif"><br></font></div><div><font color="#222222" face="arial, sans-serif">Even better would be to follow glib/gio pattern, which seems to use g_simple_async_result_set_op_res_gpointer() instead for passing arguments, although the name function suggest result use case only. (see gio/gfile.c). The existing code could be updated too.</font></div>
<div><font color="#222222" face="arial, sans-serif"><br></font></div><div>-- </div>Marc-André Lureau<br>
</div>