LinGui:

- clear the dvd volume name cache when rescanning the attached dvd devices.
- also protect the cache with a mutex to prevent the unlikely-but-possible
  collision between threads.

git-svn-id: svn://localhost/HandBrake/trunk@2579 b64f7644-9d1e-0410-96f1-a4d463321fa5

diff --git a/gtk/src/callbacks.c b/gtk/src/callbacks.c
index d472889..da6039c 100644 (file)
--- a/gtk/src/callbacks.c
+++ b/gtk/src/callbacks.c
@@ -366,6 +366,7 @@ get_dvd_device_name(GDrive *gd)
 #endif
 
 static GHashTable *volname_hash = NULL;
+static GMutex     *volname_mutex = NULL;
 
 static void
 free_volname_key(gpointer data)
@@ -414,8 +415,9 @@ get_dvd_volume_name(GDrive *gd)
        drive = get_dvd_device_name(gd);
        if (g_drive_has_media (gd))
        {
-               if (volname_hash != NULL)
-                       label = g_strdup(g_hash_table_lookup(volname_hash, drive));
+               g_mutex_lock(volname_mutex);
+               label = g_strdup(g_hash_table_lookup(volname_hash, drive));
+               g_mutex_unlock(volname_mutex);
                if (label != NULL)
                {
                        if (uppers_and_unders(label))
@@ -442,6 +444,14 @@ get_dvd_volume_name(GDrive *gd)
        return result;
 }
 
+void
+ghb_volname_cache_init(void)
+{
+       volname_mutex = g_mutex_new();
+       volname_hash = g_hash_table_new_full(g_str_hash, g_str_equal,
+                                                                               free_volname_key, free_volname_value);
+}
+
 gpointer
 ghb_cache_volnames(signal_user_data_t *ud)
 {
@@ -452,11 +462,8 @@ ghb_cache_volnames(signal_user_data_t *ud)
        if (drives == NULL)
                return NULL;
 
-       if (volname_hash == NULL)
-       {
-               volname_hash = g_hash_table_new_full(g_str_hash, g_str_equal,
-                                                                               free_volname_key, free_volname_value);
-       }
+       g_mutex_lock(volname_mutex);
+       g_hash_table_remove_all(volname_hash);
        while (link != NULL)
        {
                gchar *drive = get_dvd_device_name(link->data);
@@ -476,6 +483,7 @@ ghb_cache_volnames(signal_user_data_t *ud)
                g_object_unref(link->data);
                link = link->next;
        }
+       g_mutex_unlock(volname_mutex);
 
        g_list_free(drives);
 

diff --git a/gtk/src/callbacks.h b/gtk/src/callbacks.h
index 787aa44..22de6ef 100644 (file)
--- a/gtk/src/callbacks.h
+++ b/gtk/src/callbacks.h
@@ -59,6 +59,7 @@ void ghb_inhibit_gpm(void);
 void wm_drive_changed(MSG *msg, signal_user_data_t *ud);
 #endif
 gpointer ghb_cache_volnames(signal_user_data_t *ud);
+void ghb_volname_cache_init(void);
 
 #endif // _CALLBACKS_H_
 

diff --git a/gtk/src/main.c b/gtk/src/main.c
index fc0a78d..2294666 100644 (file)
--- a/gtk/src/main.c
+++ b/gtk/src/main.c
@@ -788,6 +788,7 @@ main (int argc, char *argv[])
        g_timeout_add (500, ghb_timer_cb, (gpointer)ud);
 
        // Add dvd devices to File menu
+       ghb_volname_cache_init();
        g_thread_create((GThreadFunc)ghb_cache_volnames, ud, FALSE, NULL);
 
        GtkStatusIcon *si;