# HG changeset patch
# User nenolod
# Date 1151527830 25200
# Node ID fbb38017d1441ef0d4f4e283257b2ebfddc2dea1
# Parent  ca5d03c4b3f1c1caf85fe27034c476484a38e20f
[svn] - why are we requiring a lock in functions that are in the nolock namespace?

diff -r ca5d03c4b3f1 -r fbb38017d144 ChangeLog
--- 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.
diff -r ca5d03c4b3f1 -r fbb38017d144 audacious/playlist.c
--- 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;