From 31bdf4d2c9a7d9611347cc6d8f5aabd6b7169f4e Mon Sep 17 00:00:00 2001 From: Cedric BAIL Date: Tue, 15 May 2012 06:43:07 +0000 Subject: [PATCH] eet: make eet_dictionary thread safe. SVN revision: 71094 --- legacy/eet/ChangeLog | 4 + legacy/eet/NEWS | 1 + legacy/eet/src/lib/Eet_private.h | 1 + legacy/eet/src/lib/eet_dictionary.c | 149 ++++++++++++++++++++-------- 4 files changed, 113 insertions(+), 42 deletions(-) diff --git a/legacy/eet/ChangeLog b/legacy/eet/ChangeLog index 26d4870612..43e85b2e45 100644 --- a/legacy/eet/ChangeLog +++ b/legacy/eet/ChangeLog @@ -586,3 +586,7 @@ 2012-05-11 Cedric Bail * Force destruction of all pending file when shuting down eet. + +2012-05-15 Cedric Bail + + * Make eet_dictionary thread safe. diff --git a/legacy/eet/NEWS b/legacy/eet/NEWS index bff1bff2c3..5d11d3ee9d 100644 --- a/legacy/eet/NEWS +++ b/legacy/eet/NEWS @@ -5,6 +5,7 @@ Changes since Eet 1.6.0: Fixes: * Force destruction of all pending file when shuting down eet. + * Make eet_dictionary thread safe. Eet 1.6.0 diff --git a/legacy/eet/src/lib/Eet_private.h b/legacy/eet/src/lib/Eet_private.h index c04daf0239..afd4685dbe 100644 --- a/legacy/eet/src/lib/Eet_private.h +++ b/legacy/eet/src/lib/Eet_private.h @@ -41,6 +41,7 @@ struct _Eet_Dictionary { Eet_String *all; Eina_Hash *converts; + Eina_Lock mutex; int size; int offset; diff --git a/legacy/eet/src/lib/eet_dictionary.c b/legacy/eet/src/lib/eet_dictionary.c index c79239bc5d..f307d4ddbe 100644 --- a/legacy/eet/src/lib/eet_dictionary.c +++ b/legacy/eet/src/lib/eet_dictionary.c @@ -21,6 +21,7 @@ eet_dictionary_add(void) return NULL; memset(new->hash, -1, sizeof (int) * 256); + eina_lock_new(&new->mutex); return new; } @@ -32,6 +33,8 @@ eet_dictionary_free(Eet_Dictionary *ed) if (!ed) return; + eina_lock_free(&ed->mutex); + for (i = 0; i < ed->count; ++i) if (ed->all[i].allocated) eina_stringshare_del(ed->all[i].str); @@ -94,12 +97,17 @@ eet_dictionary_string_add(Eet_Dictionary *ed, hash = _eet_hash_gen(string, 8); len = strlen(string) + 1; + eina_lock_take(&ed->mutex); + idx = _eet_dictionary_lookup(ed, string, len, hash); if (idx != -1) { if (ed->all[idx].str && (ed->all[idx].str == string || strcmp(ed->all[idx].str, string) == 0)) - return idx; + { + eina_lock_release(&ed->mutex); + return idx; + } } if (ed->total == ed->count) @@ -110,16 +118,14 @@ eet_dictionary_string_add(Eet_Dictionary *ed, total = ed->total + 8; new = realloc(ed->all, total * sizeof(Eet_String)); - if (!new) - return -1; + if (!new) goto on_error; ed->all = new; ed->total = total; } str = eina_stringshare_add(string); - if (!str) - return -1; + if (!str) goto on_error; current = ed->all + ed->count; @@ -150,23 +156,33 @@ eet_dictionary_string_add(Eet_Dictionary *ed, ed->hash[hash] = ed->count; } + eina_lock_release(&ed->mutex); return ed->count++; + + on_error: + eina_lock_release(&ed->mutex); + return -1; } int eet_dictionary_string_get_size(const Eet_Dictionary *ed, int idx) { - if (!ed) - return 0; + int length = 0; - if (idx < 0) - return 0; + if (!ed) goto done; + + if (idx < 0) goto done; + + eina_lock_take((Eina_Lock*) &ed->mutex); if (idx < ed->count) - return ed->all[idx].len; + length = ed->all[idx].len; - return 0; + eina_lock_release((Eina_Lock*) &ed->mutex); + + done: + return length; } EAPI int @@ -179,27 +195,34 @@ int eet_dictionary_string_get_hash(const Eet_Dictionary *ed, int idx) { - if (!ed) - return -1; + int hash = -1; - if (idx < 0) - return -1; + if (!ed) goto done; + + if (idx < 0) goto done; + + eina_lock_take((Eina_Lock*) &ed->mutex); if (idx < ed->count) - return ed->all[idx].hash; + hash = ed->all[idx].hash; - return -1; + eina_lock_release((Eina_Lock*) &ed->mutex); + + done: + return hash; } const char * eet_dictionary_string_get_char(const Eet_Dictionary *ed, int idx) { - if (!ed) - return NULL; + const char *s = NULL; - if (idx < 0) - return NULL; + if (!ed) goto done; + + if (idx < 0) goto done; + + eina_lock_take((Eina_Lock*) &ed->mutex); if (idx < ed->count) { @@ -211,10 +234,13 @@ eet_dictionary_string_get_char(const Eet_Dictionary *ed, ed->all[idx].allocated = EINA_TRUE; } #endif /* ifdef _WIN32 */ - return ed->all[idx].str; + s = ed->all[idx].str; } - return NULL; + eina_lock_release((Eina_Lock*) &ed->mutex); + + done: + return s; } static inline Eina_Bool @@ -281,19 +307,25 @@ _eet_dictionary_test(const Eet_Dictionary *ed, int idx, void *result) { - if (!result) - return EINA_FALSE; + Eina_Bool limit = EINA_FALSE; - if (!ed) - return EINA_FALSE; + if (!result) goto done; - if (idx < 0) - return EINA_FALSE; + if (!ed) goto done; - if (!(idx < ed->count)) - return EINA_FALSE; + if (idx < 0) goto done; - return EINA_TRUE; + eina_lock_take((Eina_Lock*) &ed->mutex); + + if (!(idx < ed->count)) goto unlock_done; + + limit = EINA_TRUE; + + unlock_done: + eina_lock_release((Eina_Lock*) &ed->mutex); + + done: + return limit; } static Eet_Convert * @@ -303,6 +335,8 @@ eet_dictionary_convert_get(const Eet_Dictionary *ed, { Eet_Convert *result; + eina_lock_take((Eina_Lock*) &ed->mutex); + *str = ed->all[idx].str; if (!ed->converts) @@ -313,12 +347,16 @@ eet_dictionary_convert_get(const Eet_Dictionary *ed, } result = eina_hash_find(ed->converts, &idx); - if (result) return result; + if (result) goto done; -add_convert: + add_convert: result = calloc(1, sizeof (Eet_Convert)); eina_hash_add(ed->converts, &idx, result); + + done: + eina_lock_release((Eina_Lock*) &ed->mutex); + return result; } @@ -338,6 +376,7 @@ eet_dictionary_string_get_float(const Eet_Dictionary *ed, if (!(convert->type & EET_D_FLOAT)) { + eina_lock_take((Eina_Lock*) &ed->mutex); if (!_eet_dictionary_string_get_float_cache(str, ed->all[idx].len, &convert->f)) { @@ -346,10 +385,14 @@ eet_dictionary_string_get_float(const Eet_Dictionary *ed, if (eina_convert_atod(str, ed->all[idx].len, &mantisse, &exponent) == EINA_FALSE) - return EINA_FALSE; + { + eina_lock_release((Eina_Lock*) &ed->mutex); + return EINA_FALSE; + } convert->f = ldexpf((float)mantisse, exponent); } + eina_lock_release((Eina_Lock*) &ed->mutex); convert->type |= EET_D_FLOAT; } @@ -374,6 +417,8 @@ eet_dictionary_string_get_double(const Eet_Dictionary *ed, if (!(convert->type & EET_D_DOUBLE)) { + eina_lock_take((Eina_Lock*) &ed->mutex); + if (!_eet_dictionary_string_get_double_cache(str, ed->all[idx].len, &convert->d)) { @@ -382,10 +427,14 @@ eet_dictionary_string_get_double(const Eet_Dictionary *ed, if (eina_convert_atod(str, ed->all[idx].len, &mantisse, &exponent) == EINA_FALSE) - return EINA_FALSE; + { + eina_lock_release((Eina_Lock*) &ed->mutex); + return EINA_FALSE; + } convert->d = ldexp((double)mantisse, exponent); } + eina_lock_release((Eina_Lock*) &ed->mutex); convert->type |= EET_D_DOUBLE; } @@ -412,8 +461,13 @@ eet_dictionary_string_get_fp(const Eet_Dictionary *ed, { Eina_F32p32 fp; + eina_lock_take((Eina_Lock*) &ed->mutex); if (!eina_convert_atofp(str, ed->all[idx].len, &fp)) - return EINA_FALSE; + { + eina_lock_release((Eina_Lock*) &ed->mutex); + return EINA_FALSE; + } + eina_lock_release((Eina_Lock*) &ed->mutex); convert->fp = fp; convert->type |= EET_D_FIXED_POINT; @@ -427,18 +481,29 @@ EAPI int eet_dictionary_string_check(Eet_Dictionary *ed, const char *string) { + int res = 0; int i; if ((!ed) || (!string)) return 0; + eina_lock_take(&ed->mutex); + if ((ed->start <= string) && (string < ed->end)) - return 1; + res = 1; - for (i = 0; i < ed->count; ++i) - if ((ed->all[i].allocated) && ed->all[i].str == string) - return 1; + if (!res) + { + for (i = 0; i < ed->count; ++i) + if ((ed->all[i].allocated) && ed->all[i].str == string) + { + res = 1; + break; + } + } - return 0; + eina_lock_release(&ed->mutex); + + return res; }