diff options
author | Tim Janik <timj@gtk.org> | 2001-08-12 00:55:38 +0000 |
---|---|---|
committer | Tim Janik <timj@src.gnome.org> | 2001-08-12 00:55:38 +0000 |
commit | 4bc2b2c3b4bee766422c04569f3c23daed1b453e (patch) | |
tree | 75e9478cf8ac75249eb34a80690088b44a70083a /gobject/gvaluearray.c | |
parent | 6132f0496a9f27a234c44cfa10988fa44f1c7dc9 (diff) | |
download | glib-4bc2b2c3b4bee766422c04569f3c23daed1b453e.tar.gz glib-4bc2b2c3b4bee766422c04569f3c23daed1b453e.tar.bz2 glib-4bc2b2c3b4bee766422c04569f3c23daed1b453e.zip |
fix preallocation logic, support DISABLE_MEM_POOLS properly, group value
Sun Aug 12 02:07:10 2001 Tim Janik <timj@gtk.org>
* gvaluearray.[hc]: fix preallocation logic, support DISABLE_MEM_POOLS
properly, group value allocations.
(g_value_array_new): fix semantic of n_prealloced argument, so it's
really just about preallocation space.
Diffstat (limited to 'gobject/gvaluearray.c')
-rw-r--r-- | gobject/gvaluearray.c | 65 |
1 files changed, 54 insertions, 11 deletions
diff --git a/gobject/gvaluearray.c b/gobject/gvaluearray.c index 30bc6b9e6..44685482e 100644 --- a/gobject/gvaluearray.c +++ b/gobject/gvaluearray.c @@ -25,6 +25,11 @@ #include <string.h> #include <stdlib.h> /* qsort() */ +#ifdef DISABLE_MEM_POOLS +# define GROUP_N_VALUES (1) /* power of 2 !! */ +#else +# define GROUP_N_VALUES (8) /* power of 2 !! */ +#endif /* --- functions --- */ @@ -38,14 +43,49 @@ g_value_array_get_nth (GValueArray *value_array, return value_array->values + index; } +static inline void +value_array_grow (GValueArray *value_array, + guint n_values, + gboolean zero_init) +{ + g_return_if_fail (n_values >= value_array->n_values); + + value_array->n_values = n_values; + if (value_array->n_values > value_array->n_prealloced) + { + guint i = value_array->n_prealloced; + + value_array->n_prealloced = (value_array->n_values + GROUP_N_VALUES - 1) & ~(GROUP_N_VALUES - 1); + value_array->values = g_renew (GValue, value_array->values, value_array->n_prealloced); + if (!zero_init) + i = value_array->n_values; + memset (value_array->values + i, 0, + (value_array->n_prealloced - i) * sizeof (value_array->values[0])); + } +} + +static inline void +value_array_shrink (GValueArray *value_array) +{ +#ifdef DISABLE_MEM_POOLS + if (value_array->n_prealloced >= value_array->n_values + GROUP_N_VALUES) + { + value_array->n_prealloced = (value_array->n_values + GROUP_N_VALUES - 1) & ~(GROUP_N_VALUES - 1); + value_array->values = g_renew (GValue, value_array->values, value_array->n_prealloced); + } +#endif +} + GValueArray* g_value_array_new (guint n_prealloced) { - GValueArray *value_array = g_new0 (GValueArray, 1); + GValueArray *value_array = g_new (GValueArray, 1); - value_array->n_values = n_prealloced; - value_array->values = g_renew (GValue, value_array->values, value_array->n_values); - memset (value_array->values, 0, value_array->n_values * sizeof (value_array->values[0])); + value_array->n_values = 0; + value_array->n_prealloced = 0; + value_array->values = NULL; + value_array_grow (value_array, n_prealloced, TRUE); + value_array->n_values = 0; return value_array; } @@ -76,10 +116,11 @@ g_value_array_copy (const GValueArray *value_array) g_return_val_if_fail (value_array != NULL, NULL); - new_array = g_new0 (GValueArray, 1); - new_array->n_values = value_array->n_values; - new_array->values = g_renew (GValue, new_array->values, new_array->n_values); - memset (new_array->values, 0, new_array->n_values * sizeof (new_array->values[0])); + new_array = g_new (GValueArray, 1); + new_array->n_values = 0; + new_array->values = NULL; + new_array->n_prealloced = 0; + value_array_grow (new_array, value_array->n_values, TRUE); for (i = 0; i < new_array->n_values; i++) if (G_VALUE_TYPE (value_array->values + i) != 0) { @@ -121,8 +162,8 @@ g_value_array_insert (GValueArray *value_array, /* we support NULL for "value" as a shortcut for an unset value */ - i = value_array->n_values++; - value_array->values = g_renew (GValue, value_array->values, value_array->n_values); + i = value_array->n_values; + value_array_grow (value_array, value_array->n_values + 1, FALSE); if (index + 1 < value_array->n_values) g_memmove (value_array->values + index + 1, value_array->values + index, (i - index) * sizeof (value_array->values[0])); @@ -148,7 +189,9 @@ g_value_array_remove (GValueArray *value_array, if (index < value_array->n_values) g_memmove (value_array->values + index, value_array->values + index + 1, (value_array->n_values - index) * sizeof (value_array->values[0])); - value_array->values = g_renew (GValue, value_array->values, value_array->n_values); + value_array_shrink (value_array); + if (value_array->n_prealloced > value_array->n_values) + memset (value_array->values + value_array->n_values, 0, sizeof (value_array->values[0])); return value_array; } |