changeset 1351:fbb38017d144 trunk

[svn] - why are we requiring a lock in functions that are in the nolock namespace?
author nenolod
date Wed, 28 Jun 2006 13:50:30 -0700
parents ca5d03c4b3f1
children 8b249765fdd1
files ChangeLog audacious/playlist.c
diffstat 2 files changed, 9 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Wed Jun 28 13:16:32 2006 -0700
+++ b/ChangeLog	Wed Jun 28 13:50:30 2006 -0700
@@ -1,3 +1,12 @@
+2006-06-28 20:16:32 +0000  William Pitcock <nenolod@nenolod.net>
+  revision [1612]
+  - extra sanity checking keeps the double-free away
+  
+
+  Changes:        Modified:
+  +25 -9          trunk/libaudacious/titlestring.c  
+
+
 2006-06-28 20:02:34 +0000  Tony Vroon <chainsaw@gentoo.org>
   revision [1610]
   AltiVec-accelerated DCT64 pinched from mplayer SVN. Needs to be integrated into the build system.
--- a/audacious/playlist.c	Wed Jun 28 13:16:32 2006 -0700
+++ b/audacious/playlist.c	Wed Jun 28 13:50:30 2006 -0700
@@ -1215,7 +1215,6 @@
 gint
 playlist_get_length_nolock(void)
 {
-    REQUIRE_STATIC_LOCK(playlist);
     return g_list_length(playlist);
 }
 
@@ -1600,8 +1599,6 @@
 gint
 playlist_get_position_nolock(void)
 {
-    REQUIRE_STATIC_LOCK(playlist);
-
     if (playlist && playlist_position)
         return g_list_index(playlist, playlist_position);
     return 0;