On Sun, 2015-03-15 at 18:49 +0100, Thomas Haller wrote: > On Fri, 2015-03-13 at 13:20 -0500, Dan Williams wrote: > > Hi Thomas, > > > > About th/keyfile... > > > > > libnm: add _nm_utils_hash_values_to_ptrarray() function > > > > What about g_hash_table_get_values() instead? It returns the values as > > a GList* instead of a GPtrArray, were you just worried about memory > > allocation here? > > I forgot about g_hash_table_get_values(), but yes, the idea was to keep > additional overhead small. > > I refactored it again, so that no malloc is needed. > > Repushed. > > > > The rest looks OK to me. Do we need to update any testcases for the > > sort ordering? > > I think not, they still work for me. The sort order before was > unspecified.
merged as http://cgit.freedesktop.org/NetworkManager/NetworkManager/commit/?id=89c88f24805f6092ea0f9725556fb9944316467f Thomas
signature.asc
Description: This is a digitally signed message part
_______________________________________________ networkmanager-list mailing list networkmanager-list@gnome.org https://mail.gnome.org/mailman/listinfo/networkmanager-list