Re: [review] th/keyfile



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

Attachment: signature.asc
Description: This is a digitally signed message part



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]