X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fgui%2Fmacosx%2Fplaylist.m;h=d9f137089516f811bc0a3a673e69a568d2812319;hb=857b51f9c9ace345a6b6a9a45a20829101830abb;hp=9bf6fd8d9f5aa0c93bb9380d325f5e48ee00df75;hpb=b58dfe3584f7a60fcd5afbdac903b513f7082804;p=vlc diff --git a/modules/gui/macosx/playlist.m b/modules/gui/macosx/playlist.m index 9bf6fd8d9f..d9f1370895 100644 --- a/modules/gui/macosx/playlist.m +++ b/modules/gui/macosx/playlist.m @@ -1,7 +1,7 @@ /***************************************************************************** * playlist.m: MacOS X interface module ***************************************************************************** -* Copyright (C) 2002-2005 the VideoLAN team +* Copyright (C) 2002-2006 the VideoLAN team * $Id$ * * Authors: Jon Lech Johansen @@ -25,9 +25,7 @@ /* TODO * add 'icons' for different types of nodes? (http://www.cocoadev.com/index.pl?IconAndTextInTableCell) - * create a new search field build with pictures from the 'regular' search field, so it can be emulated on 10.2 * create toggle buttons for the shuffle, repeat one, repeat all functions. - * implement drag and drop and item reordering. * reimplement enable/disable item * create a new 'tool' button (see the gear button in the Finder window) for 'actions' (adding service discovery, other views, new node/playlist, save node/playlist) stuff like that @@ -44,13 +42,15 @@ #include #include -#include "intf.h" +#import "intf.h" #import "wizard.h" #import "bookmarks.h" -#include "playlist.h" -#include "controls.h" -#include "vlc_osd.h" -#include "misc.h" +#import "playlistinfo.h" +#import "playlist.h" +#import "controls.h" +#import "vlc_osd.h" +#import "misc.h" +#import /***************************************************************************** * VLCPlaylistView implementation @@ -111,14 +111,10 @@ o_outline_dict = [[NSMutableDictionary alloc] init]; } return self; -} +} - (void)awakeFromNib { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - i_current_view = VIEW_CATEGORY; - playlist_ViewUpdate( p_playlist, i_current_view ); - + playlist_t * p_playlist = pl_Yield( VLCIntf ); [o_outline_view setTarget: self]; [o_outline_view setDelegate: self]; [o_outline_view setDataSource: self]; @@ -153,10 +149,8 @@ - (int)outlineView:(NSOutlineView *)outlineView numberOfChildrenOfItem:(id)item { int i_return = 0; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - return 0; + playlist_item_t *p_item = NULL; + playlist_t * p_playlist = pl_Yield( VLCIntf ); if( outlineView != o_outline_view ) { vlc_object_release( p_playlist ); @@ -166,77 +160,51 @@ if( item == nil ) { /* root object */ - playlist_view_t *p_view; - p_view = playlist_ViewFind( p_playlist, i_current_view ); - if( p_view && p_view->p_root ) - { - i_return = p_view->p_root->i_children; - - if( i_current_view == VIEW_CATEGORY ) - { - i_return--; /* remove the GENERAL item from the list */ - i_return += p_playlist->p_general->i_children; /* add the items of the general node */ - } - } + p_item = p_playlist->p_root_category; } else { - playlist_item_t *p_item = (playlist_item_t *)[item pointerValue]; - if( p_item ) - i_return = p_item->i_children; + p_item = (playlist_item_t *)[item pointerValue]; } + if( p_item ) + i_return = p_item->i_children; vlc_object_release( p_playlist ); - + if( i_return <= 0 ) i_return = 0; - +NSLog( @"%d children for %s", i_return, p_item->p_input->psz_name ); return i_return; } /* return the child at index for the Obj-C pointer item */ /* DONE */ - (id)outlineView:(NSOutlineView *)outlineView child:(int)index ofItem:(id)item { - playlist_item_t *p_return = NULL; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_item_t *p_return = NULL, *p_item = NULL; NSValue *o_value; - - if( p_playlist == NULL ) - return nil; + playlist_t * p_playlist = pl_Yield( VLCIntf ); if( item == nil ) { /* root object */ - playlist_view_t *p_view; - p_view = playlist_ViewFind( p_playlist, i_current_view ); - if( p_view && p_view->p_root ) p_return = p_view->p_root->pp_children[index]; - - if( i_current_view == VIEW_CATEGORY ) - { - if( p_playlist->p_general->i_children && index >= 0 && index < p_playlist->p_general->i_children ) - { - p_return = p_playlist->p_general->pp_children[index]; - } - else if( p_view && p_view->p_root && index >= 0 && index - p_playlist->p_general->i_children < p_view->p_root->i_children ) - { - p_return = p_view->p_root->pp_children[index - p_playlist->p_general->i_children + 1]; - } - } + p_item = p_playlist->p_root_category; } else { - playlist_item_t *p_item = (playlist_item_t *)[item pointerValue]; - if( p_item && index < p_item->i_children && index >= 0 ) - p_return = p_item->pp_children[index]; + p_item = (playlist_item_t *)[item pointerValue]; } + if( p_item && index < p_item->i_children && index >= 0 ) + p_return = p_item->pp_children[index]; - vlc_object_release( p_playlist ); o_value = [o_outline_dict objectForKey:[NSString stringWithFormat: @"%p", p_return]]; + #if 0 + NSLog( @"%s", p_return->p_input->psz_name); + #endif if( o_value == nil ) { o_value = [[NSValue valueWithPointer: p_return] retain]; + msg_Err( VLCIntf, "missing playlist item's pointer value" ); } return o_value; } @@ -245,22 +213,14 @@ - (BOOL)outlineView:(NSOutlineView *)outlineView isItemExpandable:(id)item { int i_return = 0; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - return NO; + playlist_t *p_playlist = pl_Yield( VLCIntf ); if( item == nil ) { /* root object */ - playlist_view_t *p_view; - p_view = playlist_ViewFind( p_playlist, i_current_view ); - if( p_view && p_view->p_root ) i_return = p_view->p_root->i_children; - - if( i_current_view == VIEW_CATEGORY ) + if( p_playlist->p_root_category ) { - i_return--; - i_return += p_playlist->p_general->i_children; + i_return = p_playlist->p_root_category->i_children; } } else @@ -271,6 +231,7 @@ } vlc_object_release( p_playlist ); +NSLog( @"expandable" ); if( i_return <= 0 ) return NO; else @@ -282,55 +243,38 @@ { id o_value = nil; intf_thread_t *p_intf = VLCIntf; - playlist_t *p_playlist; playlist_item_t *p_item; if( item == nil || ![item isKindOfClass: [NSValue class]] ) return( @"error" ); - p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - { - return( @"error" ); - } - p_item = (playlist_item_t *)[item pointerValue]; - if( p_item == NULL ) { - vlc_object_release( p_playlist ); return( @"error"); } - +//NSLog( @"values for %p", p_item ); + if( [[o_tc identifier] isEqualToString:@"1"] ) { o_value = [NSString stringWithUTF8String: - p_item->input.psz_name]; + p_item->p_input->psz_name]; if( o_value == NULL ) o_value = [NSString stringWithCString: - p_item->input.psz_name]; + p_item->p_input->psz_name]; } - else if( [[o_tc identifier] isEqualToString:@"2"] ) + else if( [[o_tc identifier] isEqualToString:@"2"] && p_item->p_input->p_meta && + p_item->p_input->p_meta->psz_artist && *p_item->p_input->p_meta->psz_artist ) { - char *psz_temp; - psz_temp = vlc_input_item_GetInfo( &p_item->input ,_("Meta-information"),_("Artist") ); - - if( psz_temp == NULL ) - o_value = @""; - else - { - o_value = [NSString stringWithUTF8String: psz_temp]; - if( o_value == NULL ) - { - o_value = [NSString stringWithCString: psz_temp]; - } - free( psz_temp ); - } + o_value = [NSString stringWithUTF8String: + p_item->p_input->p_meta->psz_artist]; + if( o_value == NULL ) + o_value = [NSString stringWithCString: + p_item->p_input->p_meta->psz_artist]; } else if( [[o_tc identifier] isEqualToString:@"3"] ) { char psz_duration[MSTRTIME_MAX_SIZE]; - mtime_t dur = p_item->input.i_duration; + mtime_t dur = p_item->p_input->i_duration; if( dur != -1 ) { secstotimestr( psz_duration, dur/1000000 ); @@ -341,7 +285,6 @@ o_value = @"-:--:--"; } } - vlc_object_release( p_playlist ); return( o_value ); } @@ -365,6 +308,19 @@ @end +/***************************************************************************** + * extension to NSOutlineView's interface to fix compilation warnings + * and let us access these 2 functions properly + * this uses a private Apple-API, but works fine on all current OSX releases + * keep checking for compatiblity with future releases though + *****************************************************************************/ + +@interface NSOutlineView (UndocumentedSortImages) ++ (NSImage *)_defaultTableHeaderSortImage; ++ (NSImage *)_defaultTableHeaderReverseSortImage; +@end + + /***************************************************************************** * VLCPlaylist implementation *****************************************************************************/ @@ -383,8 +339,7 @@ - (void)awakeFromNib { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( VLCIntf ); vlc_list_t *p_list = vlc_list_find( p_playlist, VLC_OBJECT_MODULE, FIND_ANYWHERE ); @@ -399,26 +354,11 @@ @"VLCPlaylistItemPboardType", nil]]; [o_outline_view setIntercellSpacing: NSMakeSize (0.0, 1.0)]; -/* We need to check whether _defaultTableHeaderSortImage exists, since it -belongs to an Apple hidden private API, and then can "disapear" at any time*/ - - if( [[NSOutlineView class] respondsToSelector:@selector(_defaultTableHeaderSortImage)] ) - { - o_ascendingSortingImage = [[NSOutlineView class] _defaultTableHeaderSortImage]; - } - else - { - o_ascendingSortingImage = nil; - } - - if( [[NSOutlineView class] respondsToSelector:@selector(_defaultTableHeaderReverseSortImage)] ) - { - o_descendingSortingImage = [[NSOutlineView class] _defaultTableHeaderReverseSortImage]; - } - else - { - o_descendingSortingImage = nil; - } + /* this uses private Apple API which works fine until 10.4, + * but keep checking in the future! + * These methods are being added artificially to NSOutlineView's interface above */ + o_ascendingSortingImage = [[NSOutlineView class] _defaultTableHeaderSortImage]; + o_descendingSortingImage = [[NSOutlineView class] _defaultTableHeaderReverseSortImage]; o_tc_sortColumn = nil; @@ -480,23 +420,23 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_mi_delete setTitle: _NS("Delete")]; [o_mi_recursive_expand setTitle: _NS("Expand Node")]; [o_mi_selectall setTitle: _NS("Select All")]; - [o_mi_info setTitle: _NS("Properties")]; - [o_mi_preparse setTitle: _NS("Preparse")]; + [o_mi_info setTitle: _NS("Information")]; + [o_mi_preparse setTitle: _NS("Get Stream Information")]; [o_mi_sort_name setTitle: _NS("Sort Node by Name")]; [o_mi_sort_author setTitle: _NS("Sort Node by Author")]; [o_mi_services setTitle: _NS("Services discovery")]; [o_status_field setStringValue: [NSString stringWithFormat: _NS("No items in the playlist")]]; - [o_random_ckb setTitle: _NS("Random")]; #if 0 [o_search_button setTitle: _NS("Search")]; #endif [o_search_field setToolTip: _NS("Search in Playlist")]; - [[o_loop_popup itemAtIndex:0] setTitle: _NS("Standard Play")]; - [[o_loop_popup itemAtIndex:1] setTitle: _NS("Repeat One")]; - [[o_loop_popup itemAtIndex:2] setTitle: _NS("Repeat All")]; [o_mi_addNode setTitle: _NS("Add Folder to Playlist")]; + + [o_save_accessory_text setStringValue: _NS("File Format:")]; + [[o_save_accessory_popup itemAtIndex:0] setTitle: _NS("Extended M3U")]; + [[o_save_accessory_popup itemAtIndex:1] setTitle: _NS("XML Shareable Playlist Format (XSPF)")]; } - (void)playlistUpdated @@ -517,66 +457,62 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_outline_view reloadData]; [[[[VLCMain sharedInstance] getWizard] getPlaylistWizard] reloadOutlineView]; [[[[VLCMain sharedInstance] getBookmarks] getDataTable] reloadData]; + + playlist_t *p_playlist = pl_Yield( VLCIntf ); + + if( p_playlist->i_size >= 2 ) + { + [o_status_field setStringValue: [NSString stringWithFormat: + _NS("%i items in the playlist"), p_playlist->i_size]]; + } + else + { + if( p_playlist->i_size == 0 ) + { + [o_status_field setStringValue: _NS("No items in the playlist")]; + } + else + { + [o_status_field setStringValue: _NS("1 item in the playlist")]; + } + } + vlc_object_release( p_playlist ); } - (void)playModeUpdated { - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); vlc_value_t val, val2; - if( p_playlist == NULL ) - { - return; - } - var_Get( p_playlist, "loop", &val2 ); var_Get( p_playlist, "repeat", &val ); if( val.b_bool == VLC_TRUE ) { - [o_loop_popup selectItemAtIndex: 1]; + [[[VLCMain sharedInstance] getControls] repeatOne]; } else if( val2.b_bool == VLC_TRUE ) { - [o_loop_popup selectItemAtIndex: 2]; + [[[VLCMain sharedInstance] getControls] repeatAll]; } else { - [o_loop_popup selectItemAtIndex: 0]; + [[[VLCMain sharedInstance] getControls] repeatOff]; } - var_Get( p_playlist, "random", &val ); - [o_random_ckb setState: val.b_bool]; + [[[VLCMain sharedInstance] getControls] shuffle]; vlc_object_release( p_playlist ); } -- (playlist_item_t *)parentOfItem:(playlist_item_t *)p_item -{ - int i; - for( i = 0 ; i < p_item->i_parents; i++ ) - { - if( p_item->pp_parents[i]->i_view == i_current_view ) - { - return p_item->pp_parents[i]->p_parent; - } - } - return NULL; -} - - (void)updateRowSelection { int i_row; unsigned int j; - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); playlist_item_t *p_item, *p_temp_item; NSMutableArray *o_array = [NSMutableArray array]; - if( p_playlist == NULL ) - return; - p_item = p_playlist->status.p_item; if( p_item == NULL ) { @@ -585,11 +521,10 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } p_temp_item = p_item; - while( p_temp_item->i_parents > 0 ) + while( p_temp_item->p_parent ) { [o_array insertObject: [NSValue valueWithPointer: p_temp_item] atIndex: 0]; - - p_temp_item = [self parentOfItem: p_temp_item]; + p_temp_item = p_temp_item->p_parent; /*for (i = 0 ; i < p_temp_item->i_parents ; i++) { if( p_temp_item->pp_parents[i]->i_view == i_current_view ) @@ -600,13 +535,15 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ }*/ } - for (j = 0 ; j < [o_array count] - 1 ; j++) + for( j = 0; j < [o_array count] - 1; j++ ) { id o_item; if( ( o_item = [o_outline_dict objectForKey: [NSString stringWithFormat: @"%p", [[o_array objectAtIndex:j] pointerValue]]] ) != nil ) + { [o_outline_view expandItem: o_item]; + } } @@ -616,7 +553,10 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_outline_view selectRow: i_row byExtendingSelection: NO]; [o_outline_view scrollRowToVisible: i_row]; - vlc_object_release(p_playlist); + vlc_object_release( p_playlist ); + + /* update our info-panel to reflect the new item */ + [[[VLCMain sharedInstance] getInfo] updatePanel]; } /* Check if p_item is a child of p_node recursively. We need to check the item @@ -628,15 +568,9 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ checkItemExistence:(BOOL)b_check { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( VLCIntf ); playlist_item_t *p_temp_item = p_item; - if( p_playlist == NULL ) - { - return NO; - } - if( p_node == p_item ) { vlc_object_release(p_playlist); @@ -671,33 +605,15 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } } - while( p_temp_item->i_parents > 0 ) + while( p_temp_item ) { - p_temp_item = [self parentOfItem: p_temp_item]; + p_temp_item = p_temp_item->p_parent; if( p_temp_item == p_node ) { vlc_mutex_unlock( &p_playlist->object_lock ); vlc_object_release( p_playlist ); return YES; } - -/* for( i = 0; i < p_temp_item->i_parents ; i++ ) - { - if( p_temp_item->pp_parents[i]->i_view == i_current_view ) - { - if( p_temp_item->pp_parents[i]->p_parent == p_node ) - { - vlc_mutex_unlock( &p_playlist->object_lock ); - vlc_object_release( p_playlist ); - return YES; - } - else - { - p_temp_item = p_temp_item->pp_parents[i]->p_parent; - break; - } - } - }*/ } vlc_mutex_unlock( &p_playlist->object_lock ); } @@ -737,67 +653,98 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (IBAction)savePlaylist:(id)sender { intf_thread_t * p_intf = VLCIntf; - playlist_t * p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( p_intf ); NSSavePanel *o_save_panel = [NSSavePanel savePanel]; - NSString * o_name = [NSString stringWithFormat: @"%@.m3u", _NS("Untitled")]; + NSString * o_name = [NSString stringWithFormat: @"%@", _NS("Untitled")]; + + //[o_save_panel setAllowedFileTypes: [NSArray arrayWithObjects: @"m3u", @"xpf", nil] ]; [o_save_panel setTitle: _NS("Save Playlist")]; [o_save_panel setPrompt: _NS("Save")]; + [o_save_panel setAccessoryView: o_save_accessory_view]; if( [o_save_panel runModalForDirectory: nil file: o_name] == NSOKButton ) { - playlist_Export( p_playlist, [[o_save_panel filename] fileSystemRepresentation], "export-m3u" ); + NSString *o_filename = [o_save_panel filename]; + + if( [o_save_accessory_popup indexOfSelectedItem] == 1 ) + { + NSString * o_real_filename; + NSRange range; + range.location = [o_filename length] - [@".xspf" length]; + range.length = [@".xspf" length]; + + if( [o_filename compare:@".xspf" options: NSCaseInsensitiveSearch + range: range] != NSOrderedSame ) + { + o_real_filename = [NSString stringWithFormat: @"%@.xspf", o_filename]; + } + else + { + o_real_filename = o_filename; + } + playlist_Export( p_playlist, + [o_real_filename fileSystemRepresentation], + p_playlist->p_local_category, "export-xspf" ); + } + else + { + NSString * o_real_filename; + NSRange range; + range.location = [o_filename length] - [@".m3u" length]; + range.length = [@".m3u" length]; + + if( [o_filename compare:@".m3u" options: NSCaseInsensitiveSearch + range: range] != NSOrderedSame ) + { + o_real_filename = [NSString stringWithFormat: @"%@.m3u", o_filename]; + } + else + { + o_real_filename = o_filename; + } + playlist_Export( p_playlist, + [o_real_filename fileSystemRepresentation], + p_playlist->p_local_category, "export-m3u" ); + } } vlc_object_release( p_playlist ); } - /* When called retrieves the selected outlineview row and plays that node or item */ - (IBAction)playItem:(id)sender { intf_thread_t * p_intf = VLCIntf; - playlist_t * p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( p_intf ); - if( p_playlist != NULL ) - { - playlist_item_t *p_item; - playlist_item_t *p_node = NULL; + playlist_item_t *p_item; + playlist_item_t *p_node = NULL; - p_item = [[o_outline_view itemAtRow:[o_outline_view selectedRow]] pointerValue]; + p_item = [[o_outline_view itemAtRow:[o_outline_view selectedRow]] pointerValue]; - if( p_item ) + if( p_item ) + { + if( p_item->i_children == -1 ) { - if( p_item->i_children == -1 ) - { - p_node = [self parentOfItem: p_item]; + p_node = p_item->p_parent; -/* for( i = 0 ; i < p_item->i_parents ; i++ ) - { - if( p_item->pp_parents[i]->i_view == i_current_view ) - { - p_node = p_item->pp_parents[i]->p_parent; - } - }*/ + } + else + { + p_node = p_item; + if( p_node->i_children > 0 && p_node->pp_children[0]->i_children == -1 ) + { + p_item = p_node->pp_children[0]; } else { - p_node = p_item; - if( p_node->i_children > 0 && p_node->pp_children[0]->i_children == -1 ) - { - p_item = p_node->pp_children[0]; - } - else - { - p_item = NULL; - } + p_item = NULL; } - playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, i_current_view, p_node, p_item ); } - vlc_object_release( p_playlist ); + playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, p_node, p_item ); } + vlc_object_release( p_playlist ); } /* When called retrieves the selected outlineview row and plays that node or item */ @@ -806,40 +753,36 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ int i_count; NSMutableArray *o_to_preparse; intf_thread_t * p_intf = VLCIntf; - playlist_t * p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( p_intf ); o_to_preparse = [NSMutableArray arrayWithArray:[[o_outline_view selectedRowEnumerator] allObjects]]; i_count = [o_to_preparse count]; - if( p_playlist != NULL ) + int i, i_row; + NSNumber *o_number; + playlist_item_t *p_item = NULL; + + for( i = 0; i < i_count; i++ ) { - int i, i_row; - NSNumber *o_number; - playlist_item_t *p_item = NULL; + o_number = [o_to_preparse lastObject]; + i_row = [o_number intValue]; + p_item = [[o_outline_view itemAtRow:i_row] pointerValue]; + [o_to_preparse removeObject: o_number]; + [o_outline_view deselectRow: i_row]; - for( i = 0; i < i_count; i++ ) + if( p_item ) { - o_number = [o_to_preparse lastObject]; - i_row = [o_number intValue]; - p_item = [[o_outline_view itemAtRow:i_row] pointerValue]; - [o_to_preparse removeObject: o_number]; - [o_outline_view deselectRow: i_row]; - - if( p_item ) + if( p_item->i_children == -1 ) { - if( p_item->i_children == -1 ) - { - playlist_PreparseEnqueue( p_playlist, &p_item->input ); - } - else - { - msg_Dbg( p_intf, "preparse of nodes not yet implemented" ); - } + playlist_PreparseEnqueue( p_playlist, p_item->p_input ); + } + else + { + msg_Dbg( p_intf, "preparse of nodes not yet implemented" ); } } - vlc_object_release( p_playlist ); } + vlc_object_release( p_playlist ); [self playlistUpdated]; } @@ -847,8 +790,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ { NSMenuItem *o_mi = (NSMenuItem *)sender; NSString *o_string = [o_mi representedObject]; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( VLCIntf ); if( !playlist_IsServicesDiscoveryLoaded( p_playlist, [o_string cString] ) ) playlist_ServicesDiscoveryAdd( p_playlist, [o_string cString] ); else @@ -857,8 +799,6 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_mi setState: playlist_IsServicesDiscoveryLoaded( p_playlist, [o_string cString] ) ? YES : NO]; - i_current_view = VIEW_CATEGORY; - playlist_ViewUpdate( p_playlist, i_current_view ); vlc_object_release( p_playlist ); [self playlistUpdated]; return; @@ -878,13 +818,8 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ playlist_t * p_playlist; intf_thread_t * p_intf = VLCIntf; - p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + p_playlist = pl_Yield( p_intf ); - if ( p_playlist == NULL ) - { - return; - } o_to_delete = [NSMutableArray arrayWithArray:[[o_outline_view selectedRowEnumerator] allObjects]]; i_count = [o_to_delete count]; @@ -915,14 +850,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } else { - if( p_playlist->status.i_status != PLAYLIST_STOPPED && - p_playlist->status.p_item == [[o_outline_view itemAtRow: i_row] pointerValue] ) - { - playlist_Stop( p_playlist ); - } - vlc_mutex_lock( &p_playlist->object_lock ); - playlist_Delete( p_playlist, p_item->input.i_id ); - vlc_mutex_unlock( &p_playlist->object_lock ); + playlist_LockDelete( p_playlist, p_item->i_id ); } } [self playlistUpdated]; @@ -936,20 +864,14 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (IBAction)sortNodeByAuthor:(id)sender { - [self sortNode: SORT_AUTHOR]; + [self sortNode: SORT_ARTIST]; } - (void)sortNode:(int)i_mode { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( VLCIntf ); playlist_item_t * p_item; - if (p_playlist == NULL) - { - return; - } - if( [o_outline_view selectedRow] > -1 ) { p_item = [[o_outline_view itemAtRow: [o_outline_view selectedRow]] @@ -958,8 +880,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ else /*If no item is selected, sort the whole playlist*/ { - playlist_view_t * p_view = playlist_ViewFind( p_playlist, i_current_view ); - p_item = p_view->p_root; + p_item = p_playlist->p_root_category; } if( p_item->i_children > -1 ) // the item is a node @@ -970,35 +891,21 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } else { - int i; - - for( i = 0 ; i < p_item->i_parents ; i++ ) - { - if( p_item->pp_parents[i]->i_view == i_current_view ) - { - vlc_mutex_lock( &p_playlist->object_lock ); - playlist_RecursiveNodeSort( p_playlist, - p_item->pp_parents[i]->p_parent, i_mode, ORDER_NORMAL ); - vlc_mutex_unlock( &p_playlist->object_lock ); - break; - } - } + vlc_mutex_lock( &p_playlist->object_lock ); + playlist_RecursiveNodeSort( p_playlist, + p_item->p_parent, i_mode, ORDER_NORMAL ); + vlc_mutex_unlock( &p_playlist->object_lock ); } vlc_object_release( p_playlist ); [self playlistUpdated]; } -- (playlist_item_t *)createItem:(NSDictionary *)o_one_item +- (input_item_t *)createItem:(NSDictionary *)o_one_item { intf_thread_t * p_intf = VLCIntf; - playlist_t * p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( p_intf ); - if( p_playlist == NULL ) - { - return NULL; - } - playlist_item_t *p_item; + input_item_t *p_input; int i; BOOL b_rem = FALSE, b_dir = FALSE; NSString *o_uri, *o_name; @@ -1056,15 +963,15 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ o_uri = o_temp; } - p_item = playlist_ItemNew( p_intf, [o_uri fileSystemRepresentation], [o_name UTF8String] ); - if( !p_item ) + p_input = input_ItemNew( p_playlist, [o_uri fileSystemRepresentation], [o_name UTF8String] ); + if( !p_input ) return NULL; if( o_options ) { for( i = 0; i < (int)[o_options count]; i++ ) { - playlist_ItemAddOption( p_item, strdup( [[o_options objectAtIndex:i] UTF8String] ) ); + input_ItemAddOption( p_input, strdup( [[o_options objectAtIndex:i] UTF8String] ) ); } } @@ -1077,135 +984,96 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } vlc_object_release( p_playlist ); - return p_item; + return p_input; } - (void)appendArray:(NSArray*)o_array atPos:(int)i_position enqueue:(BOOL)b_enqueue { int i_item; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - { - return; - } + playlist_t * p_playlist = pl_Yield( VLCIntf ); for( i_item = 0; i_item < (int)[o_array count]; i_item++ ) { - playlist_item_t *p_item; + input_item_t *p_input; NSDictionary *o_one_item; /* Get the item */ o_one_item = [o_array objectAtIndex: i_item]; - p_item = [self createItem: o_one_item]; - if( !p_item ) + p_input = [self createItem: o_one_item]; + if( !p_input ) { continue; } /* Add the item */ - playlist_AddItem( p_playlist, p_item, PLAYLIST_INSERT, i_position == -1 ? PLAYLIST_END : i_position + i_item ); + playlist_PlaylistAddInput( p_playlist, p_input, PLAYLIST_INSERT, + i_position == -1 ? PLAYLIST_END : i_position + i_item ); if( i_item == 0 && !b_enqueue ) { - playlist_Control( p_playlist, PLAYLIST_ITEMPLAY, p_item ); + playlist_item_t *p_item; + p_item = playlist_ItemGetByInput( p_playlist, p_input ); + playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, NULL, p_item ); + } + else + { + playlist_item_t *p_item; + p_item = playlist_ItemGetByInput( p_playlist, p_input ); + playlist_Control( p_playlist, PLAYLIST_PREPARSE, p_item ); } } + [self playlistUpdated]; vlc_object_release( p_playlist ); } -- (void)appendNodeArray:(NSArray*)o_array inNode:(playlist_item_t *)p_node atPos:(int)i_position inView:(int)i_view enqueue:(BOOL)b_enqueue +- (void)appendNodeArray:(NSArray*)o_array inNode:(playlist_item_t *)p_node atPos:(int)i_position enqueue:(BOOL)b_enqueue { int i_item; - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - { - return; - } + playlist_t * p_playlist = pl_Yield( VLCIntf ); for( i_item = 0; i_item < (int)[o_array count]; i_item++ ) { - playlist_item_t *p_item; + input_item_t *p_input; NSDictionary *o_one_item; /* Get the item */ o_one_item = [o_array objectAtIndex: i_item]; - p_item = [self createItem: o_one_item]; - if( !p_item ) + p_input = [self createItem: o_one_item]; + if( !p_input ) { continue; } /* Add the item */ - playlist_NodeAddItem( p_playlist, p_item, i_view, p_node, PLAYLIST_INSERT, i_position + i_item ); + playlist_NodeAddInput( p_playlist, p_input, p_node, + PLAYLIST_INSERT, + i_position == -1 ? + PLAYLIST_END : i_position + i_item ); + if( i_item == 0 && !b_enqueue ) { - playlist_Control( p_playlist, PLAYLIST_ITEMPLAY, p_item ); + playlist_item_t *p_item; + p_item = playlist_ItemGetByInput( p_playlist, p_input ); + playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, NULL, p_item ); + } + else + { + playlist_item_t *p_item; + p_item = playlist_ItemGetByInput( p_playlist, p_input ); + playlist_Control( p_playlist, PLAYLIST_PREPARSE, p_item ); } } + [self playlistUpdated]; vlc_object_release( p_playlist ); - -} - -- (IBAction)handlePopUp:(id)sender - -{ - intf_thread_t * p_intf = VLCIntf; - vlc_value_t val1,val2; - playlist_t * p_playlist = vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( p_playlist == NULL ) - { - return; - } - - switch( [o_loop_popup indexOfSelectedItem] ) - { - case 1: - - val1.b_bool = 0; - var_Set( p_playlist, "loop", val1 ); - val1.b_bool = 1; - var_Set( p_playlist, "repeat", val1 ); - vout_OSDMessage( p_intf, DEFAULT_CHAN, _( "Repeat One" ) ); - break; - - case 2: - val1.b_bool = 0; - var_Set( p_playlist, "repeat", val1 ); - val1.b_bool = 1; - var_Set( p_playlist, "loop", val1 ); - vout_OSDMessage( p_intf, DEFAULT_CHAN, _( "Repeat All" ) ); - break; - - default: - var_Get( p_playlist, "repeat", &val1 ); - var_Get( p_playlist, "loop", &val2 ); - if( val1.b_bool || val2.b_bool ) - { - val1.b_bool = 0; - var_Set( p_playlist, "repeat", val1 ); - var_Set( p_playlist, "loop", val1 ); - vout_OSDMessage( p_intf, DEFAULT_CHAN, _( "Repeat Off" ) ); - } - break; - } - vlc_object_release( p_playlist ); - [self playlistUpdated]; } - (NSMutableArray *)subSearchItem:(playlist_item_t *)p_item { - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); playlist_item_t *p_selected_item; int i_current, i_selected_row; - if( !p_playlist ) - return NULL; - i_selected_row = [o_outline_view selectedRow]; if (i_selected_row < 0) i_selected_row = 0; @@ -1220,8 +1088,8 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ vlc_mutex_lock( &p_playlist->object_lock ); o_current_name = [NSString stringWithUTF8String: - p_item->pp_children[i_current]->input.psz_name]; - psz_temp = vlc_input_item_GetInfo( &p_item->input , + p_item->pp_children[i_current]->p_input->psz_name]; + psz_temp = input_ItemGetInfo( p_item->p_input , _("Meta-information"),_("Artist") ); o_current_author = [NSString stringWithUTF8String: psz_temp]; free( psz_temp); @@ -1269,9 +1137,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (IBAction)searchItem:(id)sender { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - playlist_view_t * p_view; + playlist_t * p_playlist = pl_Yield( VLCIntf ); id o_result; unsigned int i; @@ -1279,67 +1145,52 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ b_selected_item_met = NO; - if( p_playlist == NULL ) - return; - p_view = playlist_ViewFind( p_playlist, i_current_view ); - - if( p_view ) - { /*First, only search after the selected item:* *(b_selected_item_met = NO) */ - o_result = [self subSearchItem:p_view->p_root]; - if( o_result == NULL ) - { - /* If the first search failed, search again from the beginning */ - o_result = [self subSearchItem:p_view->p_root]; - } - if( o_result != NULL ) - { - int i_start; - if( [[o_result objectAtIndex: 0] pointerValue] == - p_playlist->p_general ) - i_start = 1; - else - i_start = 0; + o_result = [self subSearchItem:p_playlist->p_root_category]; + if( o_result == NULL ) + { + /* If the first search failed, search again from the beginning */ + o_result = [self subSearchItem:p_playlist->p_root_category]; + } + if( o_result != NULL ) + { + int i_start; + if( [[o_result objectAtIndex: 0] pointerValue] == + p_playlist->p_local_category ) + i_start = 1; + else + i_start = 0; - for( i = i_start ; i < [o_result count] - 1 ; i++ ) - { - [o_outline_view expandItem: [o_outline_dict objectForKey: - [NSString stringWithFormat: @"%p", - [[o_result objectAtIndex: i] pointerValue]]]]; - } - i_row = [o_outline_view rowForItem: [o_outline_dict objectForKey: - [NSString stringWithFormat: @"%p", - [[o_result objectAtIndex: [o_result count] - 1 ] - pointerValue]]]]; - } - if( i_row > -1 ) + for( i = i_start ; i < [o_result count] - 1 ; i++ ) { - [o_outline_view selectRow:i_row byExtendingSelection: NO]; - [o_outline_view scrollRowToVisible: i_row]; + [o_outline_view expandItem: [o_outline_dict objectForKey: + [NSString stringWithFormat: @"%p", + [[o_result objectAtIndex: i] pointerValue]]]]; } + i_row = [o_outline_view rowForItem: [o_outline_dict objectForKey: + [NSString stringWithFormat: @"%p", + [[o_result objectAtIndex: [o_result count] - 1 ] + pointerValue]]]]; + } + if( i_row > -1 ) + { + [o_outline_view selectRow:i_row byExtendingSelection: NO]; + [o_outline_view scrollRowToVisible: i_row]; } vlc_object_release( p_playlist ); } - (IBAction)recursiveExpandNode:(id)sender { - int i; id o_item = [o_outline_view itemAtRow: [o_outline_view selectedRow]]; playlist_item_t *p_item = (playlist_item_t *)[o_item pointerValue]; if( ![[o_outline_view dataSource] outlineView: o_outline_view isItemExpandable: o_item] ) { - for( i = 0 ; i < p_item->i_parents ; i++ ) - { - if( p_item->pp_parents[i]->i_view == i_current_view ) - { - o_item = [o_outline_dict objectForKey: [NSString - stringWithFormat: @"%p", p_item->pp_parents[i]->p_parent]]; - break; - } - } + o_item = [o_outline_dict objectForKey: [NSString + stringWithFormat: @"%p", p_item->p_parent]]; } /* We need to collapse the node first, since OSX refuses to recursively @@ -1377,14 +1228,8 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ { int i_mode = 0, i_type; intf_thread_t *p_intf = VLCIntf; - 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; - } + playlist_t *p_playlist = pl_Yield( p_intf ); /* Check whether the selected table column header corresponds to a sortable table column*/ @@ -1394,8 +1239,6 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ return; } - p_view = playlist_ViewFind( p_playlist, i_current_view ); - if( o_tc_sortColumn == o_tc ) { b_isSortDescending = !b_isSortDescending; @@ -1411,7 +1254,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } else if( o_tc == o_tc_author ) { - i_mode = SORT_AUTHOR; + i_mode = SORT_ARTIST; } if( b_isSortDescending ) @@ -1424,7 +1267,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ } vlc_mutex_lock( &p_playlist->object_lock ); - playlist_RecursiveNodeSort( p_playlist, p_view->p_root, i_mode, i_type ); + playlist_RecursiveNodeSort( p_playlist, p_playlist->p_root_category, i_mode, i_type ); vlc_mutex_unlock( &p_playlist->object_lock ); vlc_object_release( p_playlist ); @@ -1451,13 +1294,10 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ forTableColumn:(NSTableColumn *)tableColumn item:(id)item { - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); id o_playing_item; - if( !p_playlist ) return; - o_playing_item = [o_outline_dict objectForKey: [NSString stringWithFormat:@"%p", p_playlist->status.p_item]]; @@ -1476,24 +1316,39 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (IBAction)addNode:(id)sender { + /* we have to create a new thread here because otherwise we would block the + * interface since the interaction-stuff and this code would run in the same + * thread */ + [NSThread detachNewThreadSelector: @selector(addNodeThreadedly) + toTarget: self withObject:nil]; + [self playlistUpdated]; +} + +- (void)addNodeThreadedly +{ + NSAutoreleasePool * ourPool = [[NSAutoreleasePool alloc] init]; + /* simply adds a new node to the end of the playlist */ - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); - if( !p_playlist ) - { - msg_Err( VLCIntf, "Uh Oh! Unable to find playlist!" ); - return; - } + playlist_t * p_playlist = pl_Yield( VLCIntf ); + vlc_thread_set_priority( p_playlist, VLC_THREAD_PRIORITY_LOW ); - playlist_item_t * p_item = playlist_NodeCreate( p_playlist, VIEW_CATEGORY, - _("Empty Folder"), p_playlist->p_general ); + int ret_v; + char *psz_name = NULL; + playlist_item_t * p_item; + ret_v = intf_UserStringInput( p_playlist, _("New Node"), + _("Please enter a name for the new node."), &psz_name ); + if( psz_name != NULL && psz_name != "" ) + p_item = playlist_NodeCreate( p_playlist, psz_name, + p_playlist->p_local_category ); + else + p_item = playlist_NodeCreate( p_playlist, _("Empty Folder"), + p_playlist->p_local_category ); if(! p_item ) - msg_Warn( VLCIntf, "node creation failed, fix VLC!" ); - - playlist_ViewUpdate( p_playlist, VIEW_CATEGORY ); - + msg_Warn( VLCIntf, "node creation failed" ); + vlc_object_release( p_playlist ); + [ourPool release]; } @end @@ -1503,10 +1358,10 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (id)outlineView:(NSOutlineView *)outlineView child:(int)index ofItem:(id)item { id o_value = [super outlineView: outlineView child: index ofItem: item]; - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); - if( !p_playlist ) return nil; + /* FIXME: playlist->i_size doesn't provide the correct number of items anymore + * check the playlist API for the fixed function, once zorglub implemented it -- fpk, 9/17/06 */ if( p_playlist->i_size >= 2 ) { @@ -1528,7 +1383,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_outline_dict setObject:o_value forKey:[NSString stringWithFormat:@"%p", [o_value pointerValue]]]; - + msg_Dbg( VLCIntf, "adding item %p", [o_value pointerValue] ); return o_value; } @@ -1537,16 +1392,13 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (BOOL)outlineView:(NSOutlineView *)outlineView writeItems:(NSArray *)items toPasteboard:(NSPasteboard *)pboard { unsigned int i; - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); /* First remove the items that were moved during the last drag & drop operation */ [o_items_array removeAllObjects]; [o_nodes_array removeAllObjects]; - if( !p_playlist ) return NO; - for( i = 0 ; i < [items count] ; i++ ) { id o_item = [items objectAtIndex: i]; @@ -1554,7 +1406,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ /* Refuse to move items that are not in the General Node (Service Discovery) */ if( ![self isItem: [o_item pointerValue] inNode: - p_playlist->p_general checkItemExistence: NO]) + p_playlist->p_local_category checkItemExistence: NO]) { vlc_object_release(p_playlist); return NO; @@ -1584,8 +1436,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ - (NSDragOperation)outlineView:(NSOutlineView *)outlineView validateDrop:(id )info proposedItem:(id)item proposedChildIndex:(int)index { - playlist_t *p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t *p_playlist = pl_Yield( VLCIntf ); NSPasteboard *o_pasteboard = [info draggingPasteboard]; if( !p_playlist ) return NSDragOperationNone; @@ -1604,7 +1455,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ /* We refuse to drop an item in anything else than a child of the General Node. We still accept items that would be root nodes of the outlineview however, to allow drop in an empty playlist. */ - if( !([self isItem: [item pointerValue] inNode: p_playlist->p_general + if( !([self isItem: [item pointerValue] inNode: p_playlist->p_local_category checkItemExistence: NO] || item == nil) ) { vlc_object_release( p_playlist ); @@ -1626,28 +1477,25 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ return NSDragOperationNone; } } - vlc_object_release(p_playlist); + vlc_object_release( p_playlist ); return NSDragOperationMove; } /* Drop from the Finder */ else if( [[o_pasteboard types] containsObject: NSFilenamesPboardType] ) { - vlc_object_release(p_playlist); + vlc_object_release( p_playlist ); return NSDragOperationGeneric; } - vlc_object_release(p_playlist); + vlc_object_release( p_playlist ); return NSDragOperationNone; } - (BOOL)outlineView:(NSOutlineView *)outlineView acceptDrop:(id )info item:(id)item childIndex:(int)index { - playlist_t * p_playlist = vlc_object_find( VLCIntf, VLC_OBJECT_PLAYLIST, - FIND_ANYWHERE ); + playlist_t * p_playlist = pl_Yield( VLCIntf ); NSPasteboard *o_pasteboard = [info draggingPasteboard]; - if( !p_playlist ) return NO; - /* Drag & Drop inside the playlist */ if( [[o_pasteboard types] containsObject: @"VLCPlaylistItemPboardType"] ) { @@ -1659,7 +1507,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ /* If the item is to be dropped as root item of the outline, make it a child of the General node. Else, choose the proposed parent as parent. */ - if( item == nil ) p_new_parent = p_playlist->p_general; + if( item == nil ) p_new_parent = p_playlist->p_local_category; else p_new_parent = [item pointerValue]; /* Make sure the proposed parent is a node. @@ -1676,7 +1524,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ int i_old_index = 0; p_item = [[o_all_items objectAtIndex:i] pointerValue]; - p_old_parent = [self parentOfItem: p_item]; + p_old_parent = p_item->p_parent; if( !p_old_parent ) continue; /* We may need the old index later */ @@ -1696,8 +1544,6 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ vlc_mutex_lock( &p_playlist->object_lock ); // Acually detach the item from the old position if( playlist_NodeRemoveItem( p_playlist, p_item, p_old_parent ) == - VLC_SUCCESS && - playlist_NodeRemoveParent( p_playlist, p_item, p_old_parent ) == VLC_SUCCESS ) { int i_new_index; @@ -1716,8 +1562,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ i_new_index = index + i - i_removed_from_node; } // Reattach the item to the new position - playlist_NodeInsert( p_playlist, i_current_view, p_item, - p_new_parent, i_new_index ); + playlist_NodeInsert( p_playlist, p_item, p_new_parent, i_new_index ); } vlc_mutex_unlock( &p_playlist->object_lock ); } @@ -1736,7 +1581,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ [o_outline_view selectRow: i_row byExtendingSelection: NO]; [o_outline_view scrollRowToVisible: i_row]; - vlc_object_release(p_playlist); + vlc_object_release( p_playlist ); return YES; } @@ -1772,7 +1617,7 @@ belongs to an Apple hidden private API, and then can "disapear" at any time*/ else { [self appendNodeArray: o_array inNode: p_node - atPos: index inView: i_current_view enqueue: YES]; + atPos: index enqueue: YES]; } vlc_object_release( p_playlist ); return YES;