luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2024-11-10 11:26:16 +01:00
Code Issues Packages Projects Releases Wiki Activity

keyfile: avoid needless allocations on file load

Browse Source 
When loading a key file, the keys and values of individual lines
are allocated once when copied and trimmed from the parse buffer
and allocated/copied again when added to the lookup map.

This commit avoids the second pair of allocations by introducing
a new function g_key_file_add_key_value_pair that gives the
lookup map direct ownership of the key and value copied from the
parse buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=650211
This commit is contained in:
John Lindgren 2011-05-16 22:27:11 -04:00 committed by Ray Strode
parent f96dc555d5
commit 9c1a44cf32
1 changed files with 29 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
glib/gkeyfile.c
View File

@ -139,6 +139,9 @@ static void g_key_file_remove_key_value_pair_node (GKeyFile
GKeyFileGroup *group,
GList *pair_node);
static void g_key_file_add_key_value_pair (GKeyFile *key_file,
GKeyFileGroup *group,
GKeyFileKeyValuePair *pair);
static void g_key_file_add_key (GKeyFile *key_file,
GKeyFileGroup *group,
const gchar *key,
@ -912,13 +915,24 @@ g_key_file_parse_key_value_pair (GKeyFile *key_file,
locale = key_get_locale (key);
if (locale == NULL || g_key_file_locale_is_interesting (key_file, locale))
g_key_file_add_key (key_file, key_file->current_group, key, value);
{
GKeyFileKeyValuePair *pair;
g_free (locale);
pair = g_slice_new (GKeyFileKeyValuePair);
pair->key = key;
pair->value = value;
g_key_file_add_key_value_pair (key_file, key_file->current_group, pair);
}
else
{
g_free (key);
g_free (value);
}
g_free (locale);
}
static gchar *
key_get_locale (const gchar *key)
{
@ -3329,6 +3343,17 @@ g_key_file_remove_group (GKeyFile *key_file,
return TRUE;
}
static void
g_key_file_add_key_value_pair (GKeyFile *key_file,
GKeyFileGroup *group,
GKeyFileKeyValuePair *pair)
{
g_hash_table_replace (group->lookup_map, pair->key, pair);
group->key_value_pairs = g_list_prepend (group->key_value_pairs, pair);
group->has_trailing_blank_line = FALSE;
key_file->approximate_size += strlen (pair->key) + strlen (pair->value) + 2;
}
static void
g_key_file_add_key (GKeyFile *key_file,
GKeyFileGroup *group,
@ -3341,10 +3366,7 @@ g_key_file_add_key (GKeyFile *key_file,
pair->key = g_strdup (key);
pair->value = g_strdup (value);
g_hash_table_replace (group->lookup_map, pair->key, pair);
group->key_value_pairs = g_list_prepend (group->key_value_pairs, pair);
group->has_trailing_blank_line = FALSE;
key_file->approximate_size += strlen (key) + strlen (value) + 2;
g_key_file_add_key_value_pair (key_file, group, pair);
}
/**