On Thu, Sep 01, 2016 at 11:22:52AM +0200, Victor Toso wrote:
> Hi,
>
> On Wed, Aug 31, 2016 at 11:54:41AM -0500, Jonathon Jongsma wrote:
> > Prepare to port to GObject by encapsulating all private data
>
> If I'm not mistaken, the private struct now will leak while
> SmartCardChannelClient is not a GObject with its finalize/dispose.
> If that is the case, might point it out in the commit log and/or a FIXME
> on g_new0
>
> Besides that, looks ok to me.
> Reviewed-by: Victor Toso <victort...@redhat.com>

Comment might apply to patches 06 and 07 too
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to