diff options
| author | Jeffrey Stedfast <fejj@ximian.com> | 2001-07-05 01:22:12 +0800 |
|---|---|---|
| committer | Jeffrey Stedfast <fejj@src.gnome.org> | 2001-07-05 01:22:12 +0800 |
| commit | d6e309cac3804858878039f3283b2bcff24658a1 (patch) | |
| tree | 3af63cfbf5a18ecf70f535f705ec9f91917e1364 /mail/mail-config.c | |
| parent | 37069d45fd05fd66f985b64d4364e6ef73026a18 (diff) | |
| download | gsoc2013-evolution-d6e309cac3804858878039f3283b2bcff24658a1.tar.gz gsoc2013-evolution-d6e309cac3804858878039f3283b2bcff24658a1.tar.zst gsoc2013-evolution-d6e309cac3804858878039f3283b2bcff24658a1.zip | |
Revert Sam's changes.
2001-07-04 Jeffrey Stedfast <fejj@ximian.com>
* mail-config.c (mail_config_set_show_preview): Revert Sam's changes.
Note: Uhm, g_hash_table_lookup_extended gives us a pointer to the
original value which we can just change to update the hash table
without the need to re-insert - it's already there!! Also, you
don't want to g_hash_table_insert the new value with the same key
without at least first removing the existing bucket. And, you
certainly don't want to g_free() the key after you insert it -
eek!!
svn path=/trunk/; revision=10780
Diffstat (limited to 'mail/mail-config.c')
| -rw-r--r-- | mail/mail-config.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mail/mail-config.c b/mail/mail-config.c index 28cbca3f17..6a60a4855e 100644 --- a/mail/mail-config.c +++ b/mail/mail-config.c @@ -923,8 +923,7 @@ mail_config_set_show_preview (const char *uri, gboolean value) config->preview_hash = g_hash_table_new (g_str_hash, g_str_equal); if (g_hash_table_lookup_extended (config->preview_hash, dbkey, &key, &val)) { - g_hash_table_insert (config->preview_hash, dbkey, - GINT_TO_POINTER (value)); + val = GINT_TO_POINTER (value); g_free (dbkey); } else { g_hash_table_insert (config->preview_hash, dbkey, |