Code review comment for lp:~kamstrup/unity/home-lenses

Revision history for this message
Neil J. Patel (njpatel) wrote :

237 (and other places): Instead of handling the gchar* yourself, use glib::String from UnityCore/GLibWrapper.h, it autofree's it when it's popped from the stack, so you don't need to worry about g_free in multiple places.

330: Evil, evil kamstrup.

Apart from that, excellent work :)

« Back to merge proposal