]> git.sesse.net Git - vlc/blobdiff - modules/gui/wxwindows/playlist.cpp
wx/playlist: never store the pointer to p_item when the playlist lock isn't
[vlc] / modules / gui / wxwindows / playlist.cpp
index 63630dc2730c0a016bc6a3d4267abc53c7980c1f..c0dc98cd55642253a77d1bebec011fc7a1ab7f87 100644 (file)
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * playlist.cpp : wxWindows plugin for vlc
  *****************************************************************************
- * Copyright (C) 2000-2004 VideoLAN
+ * Copyright (C) 2000-2005 VideoLAN
  * $Id$
  *
  * Authors: Olivier Teulière <ipkiss@via.ecp.fr>
@@ -49,8 +49,8 @@
 #include <wx/imaglist.h>
 
 #define HELP_SHUFFLE N_( "Shuffle" )
-#define HELP_LOOP N_( "Loop" )
-#define HELP_REPEAT N_( "Repeat" )
+#define HELP_LOOP N_( "Repeat All" )
+#define HELP_REPEAT N_( "Repeat One" )
 
 /* Callback prototype */
 static int PlaylistChanged( vlc_object_t *, const char *,
@@ -181,13 +181,11 @@ END_EVENT_TABLE()
 class PlaylistItem : public wxTreeItemData
 {
 public:
-    PlaylistItem( playlist_item_t *_p_item ) : wxTreeItemData()
+    PlaylistItem( playlist_item_t *p_item ) : wxTreeItemData()
     {
-        p_item = _p_item;
         i_id = p_item->input.i_id;
     }
 protected:
-    playlist_item_t *p_item;
     int i_id;
 friend class Playlist;
 };
@@ -197,7 +195,7 @@ friend class Playlist;
  *****************************************************************************/
 Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     wxFrame( p_parent, -1, wxU(_("Playlist")), wxDefaultPosition,
-             wxSize(345,400), wxDEFAULT_FRAME_STYLE )
+             wxSize(500,300), wxDEFAULT_FRAME_STYLE )
 {
     vlc_value_t val;
 
@@ -206,6 +204,13 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     i_update_counter = 0;
     i_sort_mode = MODE_NONE;
     b_need_update = VLC_FALSE;
+    p_playlist = (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
+                                                FIND_ANYWHERE );
+    if( p_playlist == NULL )
+    {
+        return;
+    }
+
     SetIcon( *p_intf->p_sys->p_icon );
 
     p_view_menu = NULL;
@@ -369,15 +374,7 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     SetDropTarget( new DragAndDrop( p_intf, VLC_TRUE ) );
 #endif
 
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
-    p_saved_item = NULL;
+    i_saved_id = 0;
 
 
     /* We want to be noticed of playlist changes */
@@ -397,14 +394,13 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     /* Update the playlist */
     Rebuild( VLC_TRUE );
 
-    vlc_object_release( p_playlist );
 }
 
 Playlist::~Playlist()
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
+    if( pp_sds != NULL )
+        free( pp_sds );
+
     if( p_playlist == NULL )
     {
         return;
@@ -423,8 +419,7 @@ Playlist::~Playlist()
  **********************************************************************/
 
 /* Update a node */
-void Playlist::UpdateNode( playlist_t *p_playlist, playlist_item_t *p_node,
-                           wxTreeItemId node )
+void Playlist::UpdateNode( playlist_item_t *p_node, wxTreeItemId node )
 {
     long cookie;
     wxTreeItemId child;
@@ -442,33 +437,28 @@ void Playlist::UpdateNode( playlist_t *p_playlist, playlist_item_t *p_node,
         if( !child.IsOk() )
         {
             /* Not enough children */
-            CreateNode( p_playlist, p_node->pp_children[i], node );
+            CreateNode( p_node->pp_children[i], node );
             /* Keep the tree pointer up to date */
             child = treectrl->GetNextChild( node, cookie );
         }
-        else
-        {
-        }
     }
     treectrl->SetItemImage( node, p_node->input.i_type );
 
 }
 
 /* Creates the node p_node as last child of parent */
-void Playlist::CreateNode( playlist_t *p_playlist, playlist_item_t *p_node,
-                           wxTreeItemId parent )
+void Playlist::CreateNode( playlist_item_t *p_node, wxTreeItemId parent )
 {
     wxTreeItemId node =
         treectrl->AppendItem( parent, wxL2U( p_node->input.psz_name ),
                               -1,-1, new PlaylistItem( p_node ) );
     treectrl->SetItemImage( node, p_node->input.i_type );
 
-    UpdateNodeChildren( p_playlist, p_node, node );
+    UpdateNodeChildren( p_node, node );
 }
 
 /* Update all children (recursively) of this node */
-void Playlist::UpdateNodeChildren( playlist_t *p_playlist,
-                                   playlist_item_t *p_node,
+void Playlist::UpdateNodeChildren( playlist_item_t *p_node,
                                    wxTreeItemId node )
 {
 
@@ -482,34 +472,35 @@ void Playlist::UpdateNodeChildren( playlist_t *p_playlist,
                     wxL2U( p_node->pp_children[i]->input.psz_name ), -1,-1,
                            new PlaylistItem( p_node->pp_children[i]) );
 
-            UpdateTreeItem( p_playlist, item );
+            UpdateTreeItem( item );
         }
         else
         {
-            CreateNode( p_playlist, p_node->pp_children[i],
-                        node );
+            CreateNode( p_node->pp_children[i], node );
         }
     }
 }
 
 /* Update an item in the tree */
-void Playlist::UpdateTreeItem( playlist_t *p_playlist, wxTreeItemId item )
+void Playlist::UpdateTreeItem( wxTreeItemId item )
 {
     if( ! item.IsOk() ) return;
 
-    playlist_item_t *p_item  =
-            ((PlaylistItem *)treectrl->GetItemData( item ))->p_item;
-
-    if( !p_item )
-    {
-        return;
-    }
+    wxTreeItemData *p_data = treectrl->GetItemData( item );
+    if( !p_data ) return;
+    
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    playlist_item_t *p_item = playlist_ItemGetById( p_playlist,
+                                          ((PlaylistItem *)p_data)->i_id );
+    if( !p_item ) return;
 
     wxString msg;
     wxString duration = wxU( "" );
     char *psz_author = vlc_input_item_GetInfo( &p_item->input,
                                                      _("Meta-information"),
                                                      _("Artist"));
+    if( psz_author == NULL )
+        return;
     char psz_duration[MSTRTIME_MAX_SIZE];
     mtime_t dur = p_item->input.i_duration;
 
@@ -529,6 +520,7 @@ void Playlist::UpdateTreeItem( playlist_t *p_playlist, wxTreeItemId item )
         msg.Printf( wxString(wxU( psz_author )) + wxT(" - ") +
                     wxString(wxU(p_item->input.psz_name)) + duration );
     }
+    free( psz_author );
     treectrl->SetItemText( item , msg );
     treectrl->SetItemImage( item, p_item->input.i_type );
 
@@ -541,29 +533,22 @@ void Playlist::UpdateTreeItem( playlist_t *p_playlist, wxTreeItemId item )
     {
         treectrl->SetItemBold( item, false );
     }
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }
 
-/* Process a AppendItem request */
+/* Process a AppendIt em request */
 void Playlist::AppendItem( wxCommandEvent& event )
 {
     playlist_add_t *p_add = (playlist_add_t *)event.GetClientData();
 
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
     wxTreeItemId item,node;
-    if( p_playlist == NULL )
-    {
-        event.Skip();
-        return;
-    }
 
-    if( p_add->i_view != i_current_view )
+    if( p_add->i_view != i_current_view || !p_add->p_node )
     {
         goto update;
     }
 
-    node = FindItem( treectrl->GetRootItem(), p_add->p_node );
+    node = FindItem( treectrl->GetRootItem(), p_add->p_node->input.i_id );
     if( !node.IsOk() )
     {
         goto update;
@@ -576,7 +561,7 @@ void Playlist::AppendItem( wxCommandEvent& event )
 
     if( item.IsOk() && p_add->p_item->i_children == -1 )
     {
-        UpdateTreeItem( p_playlist, item );
+        UpdateTreeItem( item );
     }
 
 update:
@@ -601,7 +586,6 @@ update:
                                   p_playlist->i_size ), 0 );
     }
 
-    vlc_object_release( p_playlist );
     return;
 }
 
@@ -609,27 +593,13 @@ update:
 void Playlist::UpdateItem( int i )
 {
     if( i < 0 ) return; /* Sanity check */
-    playlist_item_t *p_item;
-
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
-    p_item = playlist_LockItemGetById( p_playlist, i );
-
-    wxTreeItemId item = FindItem( treectrl->GetRootItem(), p_item);
+    
+    wxTreeItemId item = FindItem( treectrl->GetRootItem(), i );
 
     if( item.IsOk() )
     {
-        UpdateTreeItem( p_playlist, item );
+        UpdateTreeItem( item );
     }
-
-    vlc_object_release(p_playlist);
 }
 
 void Playlist::RemoveItem( int i )
@@ -649,58 +619,6 @@ void Playlist::RemoveItem( int i )
  * Search functions (internal)
  **********************************************************************/
 
-/* Find a wxItem from a playlist_item */
-wxTreeItemId Playlist::FindItem( wxTreeItemId root, playlist_item_t *p_item )
-{
-    long cookie;
-    PlaylistItem *p_wxcurrent;
-    wxTreeItemId search;
-    wxTreeItemId item = treectrl->GetFirstChild( root, cookie );
-    wxTreeItemId child;
-
-    if( p_item == p_saved_item && saved_tree_item.IsOk() )
-    {
-        return saved_tree_item;
-    }
-
-    p_wxcurrent = (PlaylistItem *)treectrl->GetItemData( root );
-
-    if( !p_item || !p_wxcurrent )
-    {
-        wxTreeItemId dummy;
-        return dummy;
-    }
-
-    if( p_wxcurrent->p_item == p_item )
-    {
-        return root;
-    }
-
-    while( item.IsOk() )
-    {
-        p_wxcurrent = (PlaylistItem *)treectrl->GetItemData( item );
-        if( p_wxcurrent->p_item == p_item )
-        {
-            saved_tree_item = item;
-            p_saved_item = p_item;
-            return item;
-        }
-        if( treectrl->ItemHasChildren( item ) )
-        {
-            wxTreeItemId search = FindItem( item, p_item );
-            if( search.IsOk() )
-            {
-                saved_tree_item = search;
-                p_saved_item = p_item;
-                return search;
-            }
-        }
-        item = treectrl->GetNextChild( root, cookie );
-    }
-    /* Not found */
-    wxTreeItemId dummy;
-    return dummy;
-}
 /* Find a wxItem from a playlist id */
 wxTreeItemId Playlist::FindItem( wxTreeItemId root, int i_id )
 {
@@ -756,16 +674,22 @@ int Playlist::CountItems( wxTreeItemId root )
     long cookie;
     int count = 0;
     wxTreeItemId item = treectrl->GetFirstChild( root, cookie );
-
+    
     while( item.IsOk() )
     {
         if( treectrl->ItemHasChildren( item ) )
         {
             count += CountItems( item );
         }
-        else if( ( (PlaylistItem *)treectrl->GetItemData( item ) )->
-                            p_item->i_children == -1 )
-            count++;
+        else
+        {
+            playlist_item_t *p_item;
+            LockPlaylist( p_intf->p_sys, p_playlist );
+            p_item = playlist_ItemGetById( p_playlist, ((PlaylistItem *)treectrl->GetItemData( item ))->i_id );
+            if( p_item && p_item->i_children == -1 )
+                count++;
+            UnlockPlaylist( p_intf->p_sys, p_playlist );
+        }
         item = treectrl->GetNextChild( root, cookie );
     }
     return count;
@@ -815,13 +739,6 @@ wxTreeItemId Playlist::FindItemByName( wxTreeItemId root, wxString search_string
 void Playlist::Rebuild( vlc_bool_t b_root )
 {
     playlist_view_t *p_view;
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
 
     /* We can remove the callbacks before locking, anyway, we won't
      * miss anything */
@@ -834,19 +751,18 @@ void Playlist::Rebuild( vlc_bool_t b_root )
         var_DelCallback( p_playlist, "item-deleted", ItemDeleted, this );
 
         /* ...and rebuild it */
-        vlc_mutex_lock( &p_playlist->object_lock );
+        LockPlaylist( p_intf->p_sys, p_playlist );
     }
 
     p_view = playlist_ViewFind( p_playlist, i_current_view ); /* FIXME */
 
     /* HACK we should really get new*/
-    msg_Dbg( p_intf, "rebuilding tree for view %i", i_current_view );
     treectrl->DeleteAllItems();
     treectrl->AddRoot( wxU(_("root" )), -1, -1,
                          new PlaylistItem( p_view->p_root) );
 
     wxTreeItemId root = treectrl->GetRootItem();
-    UpdateNode( p_playlist, p_view->p_root, root );
+    UpdateNode( p_view->p_root, root );
 
     int i_count = CountItems( treectrl->GetRootItem() );
 
@@ -879,9 +795,8 @@ void Playlist::Rebuild( vlc_bool_t b_root )
         var_AddCallback( p_playlist, "item-append", ItemAppended, this );
         var_AddCallback( p_playlist, "item-deleted", ItemDeleted, this );
 
-        vlc_mutex_unlock( &p_playlist->object_lock );
+        UnlockPlaylist( p_intf->p_sys, p_playlist );
     }
-    vlc_object_release( p_playlist );
 }
 
 
@@ -908,16 +823,6 @@ void Playlist::UpdatePlaylist()
 
     /* Updating the playing status every 0.5s is enough */
     if( i_update_counter % 5 ) return;
-
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
-    vlc_object_release( p_playlist );
 }
 
 /*****************************************************************************
@@ -925,36 +830,16 @@ void Playlist::UpdatePlaylist()
  *****************************************************************************/
 void Playlist::DeleteItem( int item_id )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
     playlist_LockDelete( p_playlist, item_id );
-
-    vlc_object_release( p_playlist );
 }
 
 void Playlist::DeleteNode( playlist_item_t *p_item )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
     playlist_NodeDelete( p_playlist, p_item, VLC_TRUE , VLC_FALSE );
-
-    vlc_object_release( p_playlist );
 }
 
 
-void Playlist::OnClose( wxCommandEvent& WXUNUSED(event) )
+void Playlist::OnClose( wxCloseEvent& WXUNUSED(event) )
 {
     Hide();
 }
@@ -970,19 +855,10 @@ void Playlist::OnSave( wxCommandEvent& WXUNUSED(event) )
 
     wxString filter = wxT("");
 
-    playlist_t * p_playlist =
-                (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                               FIND_ANYWHERE );
-
-    if( ! p_playlist )
-    {
-        return;
-    }
     if( p_playlist->i_size == 0 )
     {
         wxMessageBox( wxU(_("Playlist is empty") ), wxU(_("Can't save")),
                       wxICON_WARNING | wxOK, this );
-        vlc_object_release( p_playlist );
         return;
     }
 
@@ -1005,20 +881,10 @@ void Playlist::OnSave( wxCommandEvent& WXUNUSED(event) )
         }
     }
 
-    vlc_object_release( p_playlist );
-
 }
 
 void Playlist::OnOpen( wxCommandEvent& WXUNUSED(event) )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
     wxFileDialog dialog( this, wxU(_("Open playlist")), wxT(""), wxT(""),
         wxT("All playlists|*.pls;*.m3u;*.asx;*.b4s|M3U files|*.m3u"), wxOPEN );
 
@@ -1026,8 +892,6 @@ void Playlist::OnOpen( wxCommandEvent& WXUNUSED(event) )
     {
         playlist_Import( p_playlist, dialog.GetPath().mb_str() );
     }
-
-    vlc_object_release( p_playlist );
 }
 
 void Playlist::OnAddFile( wxCommandEvent& WXUNUSED(event) )
@@ -1053,30 +917,22 @@ void Playlist::OnAddMRL( wxCommandEvent& WXUNUSED(event) )
  ********************************************************************/
 void Playlist::OnSort( wxCommandEvent& event )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
     PlaylistItem *p_wxitem;
     p_wxitem = (PlaylistItem *)treectrl->GetItemData( treectrl->GetRootItem() );
 
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-    vlc_mutex_lock( &p_playlist->object_lock );
+    LockPlaylist( p_intf->p_sys, p_playlist );
     switch( event.GetId() )
     {
         case SortTitle_Event:
-            playlist_RecursiveNodeSort( p_playlist, p_wxitem->p_item,
+            playlist_RecursiveNodeSort( p_playlist, playlist_ItemGetById( p_playlist, p_wxitem->i_id ),
                                         SORT_TITLE_NODES_FIRST, ORDER_NORMAL );
             break;
         case RSortTitle_Event:
-            playlist_RecursiveNodeSort( p_playlist, p_wxitem->p_item,
+            playlist_RecursiveNodeSort( p_playlist, playlist_ItemGetById( p_playlist, p_wxitem->i_id ),
                                         SORT_TITLE_NODES_FIRST, ORDER_REVERSE );
     }
-    vlc_mutex_unlock( &p_playlist->object_lock );
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 
-    vlc_object_release( p_playlist );
     Rebuild( VLC_TRUE );
 }
 
@@ -1140,45 +996,21 @@ void Playlist::OnRandom( wxCommandEvent& event )
 {
     vlc_value_t val;
     val.b_bool = event.IsChecked();
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
     var_Set( p_playlist, "random", val);
-    vlc_object_release( p_playlist );
 }
 
 void Playlist::OnLoop( wxCommandEvent& event )
 {
     vlc_value_t val;
     val.b_bool = event.IsChecked();
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
     var_Set( p_playlist, "loop", val);
-    vlc_object_release( p_playlist );
 }
 
 void Playlist::OnRepeat( wxCommandEvent& event )
 {
     vlc_value_t val;
     val.b_bool = event.IsChecked();
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
     var_Set( p_playlist, "repeat", val);
-    vlc_object_release( p_playlist );
 }
 
 /********************************************************************
@@ -1186,9 +1018,7 @@ void Playlist::OnRepeat( wxCommandEvent& event )
  ********************************************************************/
 void Playlist::OnActivateItem( wxTreeEvent& event )
 {
-    playlist_item_t *p_item,*p_node;
-    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
-                                    VLC_OBJECT_PLAYLIST,FIND_ANYWHERE );
+    playlist_item_t *p_item,*p_node,*p_item2,*p_node2;
 
     PlaylistItem *p_wxitem = (PlaylistItem *)treectrl->GetItemData(
                                                             event.GetItem() );
@@ -1196,23 +1026,22 @@ void Playlist::OnActivateItem( wxTreeEvent& event )
 
     PlaylistItem *p_wxparent = (PlaylistItem *)treectrl->GetItemData( parent );
 
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
-    if( p_wxitem->p_item->i_children == -1 )
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    
+    p_item2 = playlist_ItemGetById(p_playlist, p_wxitem->i_id);
+    p_node2 = playlist_ItemGetById(p_playlist, p_wxparent->i_id);
+    if( p_item2 && p_item2->i_children == -1 )
     {
-        p_node = p_wxparent->p_item;
-        p_item = p_wxitem->p_item;
+        p_node = p_node2;
+        p_item = p_item2;
     }
     else
     {
-        p_node = p_wxitem->p_item;
-        if( p_wxitem->p_item->i_children > 0 &&
-            p_wxitem->p_item->pp_children[0]->i_children == -1)
+        p_node = p_item2;
+        if( p_node && p_node->i_children > 0 &&
+            p_node->pp_children[0]->i_children == -1)
         {
-            p_item = p_wxitem->p_item->pp_children[0];
+            p_item = p_node->pp_children[0];
         }
         else
         {
@@ -1222,8 +1051,7 @@ void Playlist::OnActivateItem( wxTreeEvent& event )
 
     playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, i_current_view,
                       p_node, p_item );
-
-    vlc_object_release( p_playlist );
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }
 
 void Playlist::OnKeyDown( wxTreeEvent& event )
@@ -1239,15 +1067,7 @@ void Playlist::OnKeyDown( wxTreeEvent& event )
 
 void Playlist::OnEnDis( wxCommandEvent& event )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
     msg_Warn( p_intf, "not implemented" );
-    vlc_object_release( p_playlist );
 }
 
 /**********************************************************************
@@ -1271,18 +1091,9 @@ void Playlist::OnMenuOpen( wxMenuEvent& event)
 
 void Playlist::OnMenuEvent( wxCommandEvent& event )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-
     if( event.GetId() < FirstView_Event )
     {
         event.Skip();
-        vlc_object_release( p_playlist );
         return;
     }
     else if( event.GetId() < LastView_Event )
@@ -1298,7 +1109,6 @@ void Playlist::OnMenuEvent( wxCommandEvent& event )
             i_current_view = i_new_view;
             playlist_ViewUpdate( p_playlist, i_new_view );
             Rebuild( VLC_TRUE );
-            vlc_object_release( p_playlist );
             return;
         }
         else if( i_new_view >= VIEW_FIRST_SORTED &&
@@ -1329,19 +1139,10 @@ void Playlist::OnMenuEvent( wxCommandEvent& event )
             wxMutexGuiEnter();
         }
     }
-    vlc_object_release( p_playlist );
 }
 
 wxMenu * Playlist::ViewMenu()
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return NULL;
-    }
-
     if( !p_view_menu )
     {
         p_view_menu = new wxMenu;
@@ -1359,28 +1160,19 @@ wxMenu * Playlist::ViewMenu()
 
     /* FIXME : have a list of "should have" views */
     p_view_menu->Append( FirstView_Event + VIEW_CATEGORY,
-                           wxU(_("By category") ) );
-    p_view_menu->Append( FirstView_Event + VIEW_SIMPLE,
+                           wxU(_("Normal") ) );
+/*    p_view_menu->Append( FirstView_Event + VIEW_SIMPLE,
                            wxU(_("Manually added") ) );
     p_view_menu->Append( FirstView_Event + VIEW_ALL,
-                           wxU(_("All items, unsorted") ) );
+                           wxU(_("All items, unsorted") ) ); */
     p_view_menu->Append( FirstView_Event + VIEW_S_AUTHOR,
-                           wxU(_("Sorted by author") ) );
-
-    vlc_object_release( p_playlist);
+                           wxU(_("Sorted by artist") ) );
 
     return p_view_menu;
 }
 
 wxMenu *Playlist::SDMenu()
 {
-
-    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
-                              VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
-    if( !p_playlist )
-    {
-        return NULL;
-    }
     p_sd_menu = new wxMenu;
 
     vlc_list_t *p_list = vlc_list_find( p_playlist, VLC_OBJECT_MODULE,
@@ -1391,12 +1183,22 @@ wxMenu *Playlist::SDMenu()
     {
         module_t * p_parser = (module_t *)p_list->p_values[i_index].p_object ;
 
+        if( !strcmp( p_parser->psz_capability, "services_discovery" ) )
+            i_number++;
+    }
+    if( i_number ) pp_sds = (char **)calloc( i_number, sizeof(void *) );
+
+    i_number = 0;
+    for( int i_index = 0; i_index < p_list->i_count; i_index++ )
+    {
+        module_t * p_parser = (module_t *)p_list->p_values[i_index].p_object ;
+
         if( !strcmp( p_parser->psz_capability, "services_discovery" ) )
         {
             p_sd_menu->AppendCheckItem( FirstSD_Event + i_number ,
-                       wxU( p_parser->psz_longname ? p_parser->psz_longname :
-                            ( p_parser->psz_shortname ?
-                            p_parser->psz_shortname :p_parser->psz_object_name)) );
+                wxU( p_parser->psz_longname ? p_parser->psz_longname :
+                     (p_parser->psz_shortname ?
+                      p_parser->psz_shortname : p_parser->psz_object_name) ) );
 
             if( playlist_IsServicesDiscoveryLoaded( p_playlist,
                                     p_parser->psz_object_name ) )
@@ -1404,12 +1206,10 @@ wxMenu *Playlist::SDMenu()
                 p_sd_menu->Check( FirstSD_Event + i_number, TRUE );
             }
 
-            INSERT_ELEM( (void**)pp_sds, i_number, i_number,
-                         (void*)p_parser->psz_object_name );
+            pp_sds[i_number++] = p_parser->psz_object_name;
         }
     }
     vlc_list_release( p_list );
-    vlc_object_release( p_playlist );
     return p_sd_menu;
 }
 
@@ -1420,35 +1220,48 @@ wxMenu *Playlist::SDMenu()
 void Playlist::OnPopup( wxContextMenuEvent& event )
 {
     wxPoint pt = event.GetPosition();
+    playlist_item_t *p_item;
 
-    i_popup_item = treectrl->HitTest( ScreenToClient( pt ) );
-    if( i_popup_item.IsOk() )
+    i_wx_popup_item = treectrl->HitTest( ScreenToClient( pt ) );
+    if( i_wx_popup_item.IsOk() )
     {
         PlaylistItem *p_wxitem = (PlaylistItem *)treectrl->GetItemData(
-                                                            i_popup_item );
-        PlaylistItem *p_wxparent= (PlaylistItem *) treectrl->GetItemData(
-                                   treectrl->GetItemParent( i_popup_item ) );
-        p_popup_item = p_wxitem->p_item;
-        p_popup_parent = p_wxparent->p_item;
-        treectrl->SelectItem( i_popup_item );
-        if( p_popup_item->i_children == -1 )
+                                                            i_wx_popup_item );
+        PlaylistItem *p_wxparent= (PlaylistItem *)treectrl->GetItemData(
+                                  treectrl->GetItemParent( i_wx_popup_item ) );
+        i_popup_item = p_wxitem->i_id;
+        i_popup_parent = p_wxparent->i_id;
+        treectrl->SelectItem( i_wx_popup_item );
+
+        LockPlaylist( p_intf->p_sys, p_playlist );
+        p_item = playlist_ItemGetById( p_playlist, i_popup_item );
+
+        if( !p_item )
+        {
+            UnlockPlaylist( p_intf->p_sys, p_playlist );
+            return;
+        }
+        if( p_item->i_children == -1 )
+        {
+            UnlockPlaylist( p_intf->p_sys, p_playlist );
             Playlist::PopupMenu( item_popup,
                                  ScreenToClient( wxGetMousePosition() ) );
+        }
         else
+        {
+            UnlockPlaylist( p_intf->p_sys, p_playlist );
             Playlist::PopupMenu( node_popup,
                                  ScreenToClient( wxGetMousePosition() ) );
+        }
     }
 }
 
-void Playlist::OnPopupPlay( wxMenuEvent& event )
+void Playlist::OnPopupPlay( wxCommandEvent& event )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
+    playlist_item_t *p_popup_item, *p_popup_parent;
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    p_popup_item = playlist_ItemGetById( p_playlist, i_popup_item );
+    p_popup_parent = playlist_ItemGetById( p_playlist, i_popup_parent );
     if( p_popup_item != NULL )
     {
         if( p_popup_item->i_children > -1 )
@@ -1476,24 +1289,20 @@ void Playlist::OnPopupPlay( wxMenuEvent& event )
             }
         }
     }
-    vlc_object_release( p_playlist );
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }
 
-void Playlist::OnPopupPreparse( wxMenuEvent& event )
+void Playlist::OnPopupPreparse( wxCommandEvent& event )
 {
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-    Preparse( p_playlist );
-    vlc_object_release( p_playlist );
+    Preparse();
 }
 
-void Playlist::Preparse( playlist_t *p_playlist )
+void Playlist::Preparse()
 {
+    playlist_item_t *p_popup_item;
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    p_popup_item = playlist_ItemGetById( p_playlist, i_popup_item );
+    
     if( p_popup_item != NULL )
     {
         if( p_popup_item->i_children == -1 )
@@ -1507,68 +1316,78 @@ void Playlist::Preparse( playlist_t *p_playlist )
             for( i = 0; i< p_parent->i_children ; i++ )
             {
                 wxMenuEvent dummy;
-                i_popup_item = FindItem( treectrl->GetRootItem(),
-                                         p_parent->pp_children[i] );
-                p_popup_item = p_parent->pp_children[i];
-                Preparse( p_playlist );
+                i_wx_popup_item = FindItem( treectrl->GetRootItem(),
+                                         p_parent->pp_children[i]->input.i_id );
+                i_popup_item = p_parent->pp_children[i]->input.i_id;
+                Preparse();
             }
         }
     }
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }
 
-void Playlist::OnPopupDel( wxMenuEvent& event )
+void Playlist::OnPopupDel( wxCommandEvent& event )
 {
     PlaylistItem *p_wxitem;
+    playlist_item_t *p_item;
 
-    p_wxitem = (PlaylistItem *)treectrl->GetItemData( i_popup_item );
-
-    if( p_wxitem->p_item->i_children == -1 )
+    p_wxitem = (PlaylistItem *)treectrl->GetItemData( i_wx_popup_item );
+    
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    p_item = playlist_ItemGetById( p_playlist, p_wxitem->i_id );
+    if( !p_item )
+    {
+        UnlockPlaylist( p_intf->p_sys, p_playlist );
+        return;
+    }
+    
+    if( p_item->i_children == -1 )
     {
-        DeleteItem( p_wxitem->p_item->input.i_id );
+        UnlockPlaylist( p_intf->p_sys, p_playlist );
+        DeleteItem( p_item->input.i_id );
     }
     else
     {
-        DeleteNode( p_wxitem->p_item );
+        UnlockPlaylist( p_intf->p_sys, p_playlist );
+        DeleteNode( p_item );
     }
 }
 
-void Playlist::OnPopupSort( wxMenuEvent& event )
+void Playlist::OnPopupSort( wxCommandEvent& event )
 {
     PlaylistItem *p_wxitem;
+    playlist_item_t *p_item;
 
-    p_wxitem = (PlaylistItem *)treectrl->GetItemData( i_popup_item );
+    p_wxitem = (PlaylistItem *)treectrl->GetItemData( i_wx_popup_item );
+    LockPlaylist( p_intf->p_sys, p_playlist );
 
-    if( p_wxitem->p_item->i_children >= 0 )
+    p_item = playlist_ItemGetById( p_playlist, p_wxitem->i_id );
+    if( p_item->i_children >= 0 )
     {
-        playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
-                                        VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
-
-        if( p_playlist )
-        {
-            vlc_mutex_lock( &p_playlist->object_lock );
-            playlist_RecursiveNodeSort( p_playlist, p_wxitem->p_item,
-                                        SORT_TITLE_NODES_FIRST, ORDER_NORMAL );
-            vlc_mutex_unlock( &p_playlist->object_lock );
-
-            treectrl->DeleteChildren( i_popup_item );
-            UpdateNodeChildren( p_playlist, p_wxitem->p_item, i_popup_item );
+        playlist_RecursiveNodeSort( p_playlist, p_item,
+                                    SORT_TITLE_NODES_FIRST, ORDER_NORMAL );
+        
+        treectrl->DeleteChildren( i_wx_popup_item );
+        UpdateNodeChildren( p_item, i_wx_popup_item );
 
-            vlc_object_release( p_playlist );
-        }
     }
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }
 
-void Playlist::OnPopupInfo( wxMenuEvent& event )
+void Playlist::OnPopupInfo( wxCommandEvent& event )
 {
+    LockPlaylist( p_intf->p_sys, p_playlist );
+    playlist_item_t *p_popup_item = playlist_ItemGetById( p_playlist, i_popup_item );
     if( p_popup_item )
     {
         iteminfo_dialog = new ItemInfoDialog( p_intf, p_popup_item, this );
         if( iteminfo_dialog->ShowModal() == wxID_OK )
         {
-            UpdateItem( i_popup_item );
+            UpdateItem( i_wx_popup_item );
         }
         delete iteminfo_dialog;
     }
+    UnlockPlaylist( p_intf->p_sys, p_playlist );
 }