diff src/audacious/playlist.c @ 2669:0b4949cad842 trunk

[svn] - tuple operations do not change the structure of the playlist and shouldn't block via mutexlock accordingly

--- a/src/audacious/playlist.c	Tue Apr 10 11:12:56 2007 -0700
+++ b/src/audacious/playlist.c	Tue Apr 10 11:20:21 2007 -0700
@@ -1881,10 +1881,7 @@
     if (!playlist)
         return NULL;
 
-    PLAYLIST_LOCK(playlist->mutex);
-
     if (!(node = g_list_nth(playlist->entries, pos))) {
-        PLAYLIST_UNLOCK(playlist->mutex);
         return NULL;
     }
 
@@ -1900,8 +1897,6 @@
         tuple = entry->tuple;
     }
 
-    PLAYLIST_UNLOCK(playlist->mutex);
-
     return tuple;
 }
 
@@ -1915,10 +1910,7 @@
     if (!playlist)
         return -1;
 
-    PLAYLIST_LOCK(playlist->mutex);
-
     if (!(node = g_list_nth(playlist->entries, pos))) {
-        PLAYLIST_UNLOCK(playlist->mutex);
         return -1;
     }
 
@@ -1928,12 +1920,9 @@
 
         if (playlist_entry_get_info(entry))
             song_time = entry->length;
-
-        PLAYLIST_UNLOCK(playlist->mutex);
     }
     else {
         song_time = entry->length;
-        PLAYLIST_UNLOCK(playlist->mutex);
     }
 
     return song_time;