<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 14, 2014 at 11:53 AM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div id=":k95" class="" style="overflow:hidden">ACK to the patch apart from these cosmetic issues. There are still<br>
dozens of calls to g_signal_connect in the codebase after this patch,<br>
why are these special?</div></blockquote></div><br></div><div class="gmail_extra">There was already a mix of both. No particular reason except that <span class="im"> virt_viewer_signal_connect_object() is easier to deal with when handler callback is a object. I was tracking memory issues with Spice, I didn't see the need to replace it elsewhere.<br></span></div><div class="gmail_extra"><br clear="all"><br>-- <br><div class="gmail_signature">Marc-André Lureau</div>
</div></div>