I found that Wocky has several memory leaks that are visible while running the unit tests. As this makes it harder to check that new code does not introduce new leaks, I've fixed all the one I found. Only 1 is remaining which seems to be in GnuTLS. Fixes are found in: http://git.collabora.co.uk/?p=user/nicolas/wocky.git;a=shortlog;h=refs/heads/leak-fix
Looks good, apart from style! :) + if (!g_cancellable_is_cancelled (priv->cancellable)) + { + g_cancellable_cancel (priv->cancellable); + } + g_object_unref (priv->cancellable); + priv->cancellable = NULL; I think the if statement here should not use braces. + if (priv->conn) This should check != NULL explicitly. There are loads of other cases of this in this branch. English pedantry in a commit message for future reference: in “Most call where already implemented correctly”, you mean “were”.
(In reply to comment #1) > Looks good, apart from style! :) Sorry for that, I'm stepping from project all the time and I often loose track of right coding style. > > + if (!g_cancellable_is_cancelled (priv->cancellable)) > + { > + g_cancellable_cancel (priv->cancellable); > + } > + g_object_unref (priv->cancellable); > + priv->cancellable = NULL; > > I think the if statement here should not use braces. Fixed. > > + if (priv->conn) > > This should check != NULL explicitly. There are loads of other cases of this in > this branch. Ok, since it was simpler to search them all I've end up fixing 30 of them. > > English pedantry in a commit message for future reference: in “Most call where > already implemented correctly”, you mean “were”. Fixed/Rebased So I'll merge and push this branch soon.
Merged upstream.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.