Commit fa3116ec authored by Pierre d'Herbemont's avatar Pierre d'Herbemont

Mac OS X gui: Make sure we don't retain p_input more than once. Don't release it until we're done.

parent cdc2a50d
...@@ -1086,6 +1086,8 @@ static VLCMain *_o_sharedMainInstance = nil; ...@@ -1086,6 +1086,8 @@ static VLCMain *_o_sharedMainInstance = nil;
p_intf->p_sys->b_current_title_update = VLC_TRUE; p_intf->p_sys->b_current_title_update = VLC_TRUE;
p_intf->p_sys->b_intf_update = VLC_TRUE; p_intf->p_sys->b_intf_update = VLC_TRUE;
p_intf->p_sys->b_input_update = VLC_FALSE; p_intf->p_sys->b_input_update = VLC_FALSE;
if( p_intf->p_sys->p_input )
vlc_object_yield( p_intf->p_sys->p_input );
} }
if( p_intf->p_sys->b_intf_update ) if( p_intf->p_sys->b_intf_update )
{ {
...@@ -1103,7 +1105,6 @@ static VLCMain *_o_sharedMainInstance = nil; ...@@ -1103,7 +1105,6 @@ static VLCMain *_o_sharedMainInstance = nil;
if( ( b_input = ( p_intf->p_sys->p_input != NULL ) ) ) if( ( b_input = ( p_intf->p_sys->p_input != NULL ) ) )
{ {
vlc_object_yield( p_intf->p_sys->p_input );
/* seekable streams */ /* seekable streams */
b_seekable = var_GetBool( p_intf->p_sys->p_input, "seekable" ); b_seekable = var_GetBool( p_intf->p_sys->p_input, "seekable" );
...@@ -1112,7 +1113,6 @@ static VLCMain *_o_sharedMainInstance = nil; ...@@ -1112,7 +1113,6 @@ static VLCMain *_o_sharedMainInstance = nil;
/* chapters & titles */ /* chapters & titles */
//b_chapters = p_intf->p_sys->p_input->stream.i_area_nb > 1; //b_chapters = p_intf->p_sys->p_input->stream.i_area_nb > 1;
vlc_object_release( p_intf->p_sys->p_input );
} }
[o_btn_stop setEnabled: b_input]; [o_btn_stop setEnabled: b_input];
...@@ -1551,6 +1551,11 @@ static VLCMain *_o_sharedMainInstance = nil; ...@@ -1551,6 +1551,11 @@ static VLCMain *_o_sharedMainInstance = nil;
vout_thread_t * p_vout; vout_thread_t * p_vout;
int returnedValue = 0; int returnedValue = 0;
if (p_intf->p_sys->p_input)
{
vlc_object_release( p_intf->p_sys->p_input );
p_intf->p_sys->p_input = NULL;
}
/* Stop playback */ /* Stop playback */
p_playlist = pl_Yield( p_intf ); p_playlist = pl_Yield( p_intf );
playlist_Stop( p_playlist ); playlist_Stop( p_playlist );
......
...@@ -367,6 +367,7 @@ static VLCTreeItem *o_root_item = nil; ...@@ -367,6 +367,7 @@ static VLCTreeItem *o_root_item = nil;
break; break;
default: default:
NSLog(@"unknow item in pref");
break; break;
} }
} while( p_item < p_end && p_item++ ); } while( p_item < p_end && p_item++ );
...@@ -531,7 +532,7 @@ static VLCTreeItem *o_root_item = nil; ...@@ -531,7 +532,7 @@ static VLCTreeItem *o_root_item = nil;
o_view = [[VLCFlippedView alloc] initWithFrame: s_vrc]; o_view = [[VLCFlippedView alloc] initWithFrame: s_vrc];
[o_view setAutoresizingMask: NSViewWidthSizable | NSViewMinYMargin | [o_view setAutoresizingMask: NSViewWidthSizable | NSViewMinYMargin |
NSViewMaxYMargin]; NSViewMaxYMargin];
NSLog(@"%s", __FUNCTION__);
/* Create all subviews if it isn't already done because we cannot use */ /* Create all subviews if it isn't already done because we cannot use */
/* setHiden for MacOS < 10.3*/ /* setHiden for MacOS < 10.3*/
if( o_subviews == nil ) if( o_subviews == nil )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment