control.c 19 KB
Newer Older
1
/*****************************************************************************
Clément Stenac's avatar
Clément Stenac committed
2
 * control.c : Handle control of the playlist & running through it
3 4
 *****************************************************************************
 * Copyright (C) 1999-2004 the VideoLAN team
Jean-Paul Saman's avatar
Jean-Paul Saman committed
5
 * $Id$
6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23
 *
 * Authors: Samuel Hocevar <sam@zoy.org>
 *          Clément Stenac <zorglub@videolan.org>
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
 *****************************************************************************/
24 25 26 27
#ifdef HAVE_CONFIG_H
# include "config.h"
#endif

28
#include <vlc_common.h>
29
#include "vlc_playlist.h"
30
#include "playlist_internal.h"
31
#include <assert.h>
32 33 34 35

/*****************************************************************************
 * Local prototypes
 *****************************************************************************/
36
static int PlaylistVAControl( playlist_t * p_playlist, int i_query, va_list args );
37

38
static void PreparseEnqueueItemSub( playlist_t *, playlist_item_t * );
39 40 41 42 43

/*****************************************************************************
 * Playlist control
 *****************************************************************************/

Antoine Cellerier's avatar
Antoine Cellerier committed
44
playlist_t *__pl_Hold( vlc_object_t *p_this )
45
{
46
    playlist_t *pl;
47

48 49
    barrier ();
    pl = libvlc_priv (p_this->p_libvlc)->p_playlist;
50

51
    assert( VLC_OBJECT(pl) != p_this /* This does not make sense to hold the playlist
Antoine Cellerier's avatar
Antoine Cellerier committed
52
    using pl_Hold. use vlc_object_hold in this case */ );
53

54
    if (pl)
55
        vlc_object_hold (pl);
56 57 58 59 60
    return pl;
}

void __pl_Release( vlc_object_t *p_this )
{
61
    playlist_t *pl = libvlc_priv (p_this->p_libvlc)->p_playlist;
62
    assert( pl != NULL );
63

64
    assert( VLC_OBJECT(pl) != p_this /* The rule is that pl_Release() should act on
Antoine Cellerier's avatar
Antoine Cellerier committed
65
    the same object than pl_Hold() */ );
66

67 68 69
    vlc_object_release( pl );
}

70 71 72 73 74 75
void playlist_Signal( playlist_t *p_playlist )
{
    /* TODO: assert playlist lock? */
    vlc_object_signal( p_playlist );
}

76
int playlist_Control( playlist_t * p_playlist, int i_query,
77
                      bool b_locked, ... )
78 79 80
{
    va_list args;
    int i_result;
81
    va_start( args, b_locked );
82
    PL_LOCK_IF( !b_locked );
83 84
    i_result = PlaylistVAControl( p_playlist, i_query, args );
    va_end( args );
85
    PL_UNLOCK_IF( !b_locked );
86 87 88 89

    return i_result;
}

90
static int PlaylistVAControl( playlist_t * p_playlist, int i_query, va_list args )
91 92 93 94
{
    playlist_item_t *p_item, *p_node;
    vlc_value_t val;

95 96
    PL_ASSERT_LOCKED;

97 98 99
    if( !vlc_object_alive( p_playlist ) )
        return VLC_EGENERIC;

100
    if( playlist_IsEmpty( p_playlist ) )
101 102 103 104 105
        return VLC_EGENERIC;

    switch( i_query )
    {
    case PLAYLIST_STOP:
106 107 108
        pl_priv(p_playlist)->request.i_status = PLAYLIST_STOPPED;
        pl_priv(p_playlist)->request.b_request = true;
        pl_priv(p_playlist)->request.p_item = NULL;
109 110
        break;

111 112
    // Node can be null, it will keep the same. Use with care ...
    // Item null = take the first child of node
113 114 115 116 117
    case PLAYLIST_VIEWPLAY:
        p_node = (playlist_item_t *)va_arg( args, playlist_item_t * );
        p_item = (playlist_item_t *)va_arg( args, playlist_item_t * );
        if ( p_node == NULL )
        {
118
            p_node = get_current_status_node( p_playlist );
119
            assert( p_node );
120
        }
121 122 123 124 125
        pl_priv(p_playlist)->request.i_status = PLAYLIST_RUNNING;
        pl_priv(p_playlist)->request.i_skip = 0;
        pl_priv(p_playlist)->request.b_request = true;
        pl_priv(p_playlist)->request.p_node = p_node;
        pl_priv(p_playlist)->request.p_item = p_item;
Clément Stenac's avatar
Clément Stenac committed
126
        if( p_item && var_GetBool( p_playlist, "random" ) )
127
            pl_priv(p_playlist)->b_reset_currently_playing = true;
128 129 130
        break;

    case PLAYLIST_PLAY:
131
        if( pl_priv(p_playlist)->p_input )
132 133
        {
            val.i_int = PLAYING_S;
134
            var_Set( pl_priv(p_playlist)->p_input, "state", val );
135 136
            break;
        }
137 138
        else
        {
139 140 141 142 143
            pl_priv(p_playlist)->request.i_status = PLAYLIST_RUNNING;
            pl_priv(p_playlist)->request.b_request = true;
            pl_priv(p_playlist)->request.p_node = get_current_status_node( p_playlist );
            pl_priv(p_playlist)->request.p_item = get_current_status_item( p_playlist );
            pl_priv(p_playlist)->request.i_skip = 0;
144
        }
145 146 147 148
        break;

    case PLAYLIST_PAUSE:
        val.i_int = 0;
149 150
        if( pl_priv(p_playlist)->p_input )
            var_Get( pl_priv(p_playlist)->p_input, "state", &val );
151 152 153

        if( val.i_int == PAUSE_S )
        {
154 155
            pl_priv(p_playlist)->status.i_status = PLAYLIST_RUNNING;
            if( pl_priv(p_playlist)->p_input )
156 157
            {
                val.i_int = PLAYING_S;
158
                var_Set( pl_priv(p_playlist)->p_input, "state", val );
159 160 161 162
            }
        }
        else
        {
163 164
            pl_priv(p_playlist)->status.i_status = PLAYLIST_PAUSED;
            if( pl_priv(p_playlist)->p_input )
165 166
            {
                val.i_int = PAUSE_S;
167
                var_Set( pl_priv(p_playlist)->p_input, "state", val );
168 169 170 171 172
            }
        }
        break;

    case PLAYLIST_SKIP:
173 174 175
        pl_priv(p_playlist)->request.p_node = get_current_status_node( p_playlist );
        pl_priv(p_playlist)->request.p_item = get_current_status_item( p_playlist );
        pl_priv(p_playlist)->request.i_skip = (int) va_arg( args, int );
176
        /* if already running, keep running */
177 178 179
        if( pl_priv(p_playlist)->status.i_status != PLAYLIST_STOPPED )
            pl_priv(p_playlist)->request.i_status = pl_priv(p_playlist)->status.i_status;
        pl_priv(p_playlist)->request.b_request = true;
180 181 182 183 184 185
        break;

    default:
        msg_Err( p_playlist, "unknown playlist query" );
        return VLC_EBADVAR;
    }
186
    vlc_object_signal_unlocked( p_playlist );
187 188 189 190 191 192 193 194 195 196 197

    return VLC_SUCCESS;
}

/*****************************************************************************
 * Preparse control
 *****************************************************************************/
/** Enqueue an item for preparsing */
int playlist_PreparseEnqueue( playlist_t *p_playlist,
                              input_item_t *p_item )
{
198
    playlist_preparse_t *p_preparse = &pl_priv(p_playlist)->preparse;
199

200
    vlc_gc_incref( p_item );
201 202 203 204 205 206

    vlc_mutex_lock( &p_preparse->lock );
    INSERT_ELEM( p_preparse->pp_waiting, p_preparse->i_waiting,
                 p_preparse->i_waiting, p_item );
    vlc_cond_signal( &p_preparse->wait );
    vlc_mutex_unlock( &p_preparse->lock );
207 208 209
    return VLC_SUCCESS;
}

210 211
/** Enqueue a playlist item or a node for preparsing.
 *  This function shall be called without playlist and preparser locks */
212 213 214
int playlist_PreparseEnqueueItem( playlist_t *p_playlist,
                                  playlist_item_t *p_item )
{
215
    playlist_preparse_t *p_preparse = &pl_priv(p_playlist)->preparse;
216

217
    vlc_object_lock( p_playlist );
218
    vlc_mutex_lock( &p_preparse->lock );
219
    PreparseEnqueueItemSub( p_playlist, p_item );
220
    vlc_mutex_unlock( &p_preparse->lock );
221
    vlc_object_unlock( p_playlist );
222 223 224
    return VLC_SUCCESS;
}

225
int playlist_AskForArtEnqueue( playlist_t *p_playlist,
226
                               input_item_t *p_item )
227
{
228
    playlist_fetcher_t *p_fetcher = &pl_priv(p_playlist)->fetcher;
229

230
    vlc_gc_incref( p_item );
231 232

    vlc_mutex_lock( &p_fetcher->lock );
233 234
    INSERT_ELEM( p_fetcher->pp_waiting, p_fetcher->i_waiting,
                 p_fetcher->i_waiting, p_item );
235 236
    vlc_cond_signal( &p_fetcher->wait );
    vlc_mutex_unlock( &p_fetcher->lock );
237 238 239
    return VLC_SUCCESS;
}

240 241
static void PreparseEnqueueItemSub( playlist_t *p_playlist,
                                    playlist_item_t *p_item )
242
{
243
    playlist_preparse_t *p_preparse = &pl_priv(p_playlist)->preparse;
244

245 246
    if( p_item->i_children == -1 )
    {
247
        /* Leaf item */
248
        vlc_gc_incref( p_item->p_input );
249 250 251
        INSERT_ELEM( p_preparse->pp_waiting,
                     p_preparse->i_waiting,
                     p_preparse->i_waiting,
252 253 254 255
                     p_item->p_input );
    }
    else
    {
256 257
        /* Non-leaf item: recurse */
        for( int i = 0; i < p_item->i_children; i++)
258
        {
259
            PreparseEnqueueItemSub( p_playlist, p_item->pp_children[i] );
260 261 262 263 264 265 266 267
        }
    }
}

/*****************************************************************************
 * Playback logic
 *****************************************************************************/

Rémi Duraffort's avatar
Rémi Duraffort committed
268 269 270 271 272 273 274 275 276
/**
 * Synchronise the current index of the playlist
 * to match the index of the current item.
 *
 * \param p_playlist the playlist structure
 * \param p_cur the current playlist item
 * \return nothing
 */
static void ResyncCurrentIndex( playlist_t *p_playlist, playlist_item_t *p_cur )
Clément Stenac's avatar
Clément Stenac committed
277
{
Rémi Duraffort's avatar
Rémi Duraffort committed
278
     PL_DEBUG( "resyncing on %s", PLI_NAME( p_cur ) );
Clément Stenac's avatar
Clément Stenac committed
279 280 281 282 283
     /* Simply resync index */
     int i;
     p_playlist->i_current_index = -1;
     for( i = 0 ; i< p_playlist->current.i_size; i++ )
     {
Rémi Duraffort's avatar
Rémi Duraffort committed
284
          if( ARRAY_VAL( p_playlist->current, i ) == p_cur )
Clément Stenac's avatar
Clément Stenac committed
285 286 287 288 289
          {
              p_playlist->i_current_index = i;
              break;
          }
     }
Rémi Duraffort's avatar
Rémi Duraffort committed
290
     PL_DEBUG( "%s is at %i", PLI_NAME( p_cur ), p_playlist->i_current_index );
Clément Stenac's avatar
Clément Stenac committed
291 292
}

293
void ResetCurrentlyPlaying( playlist_t *p_playlist, bool b_random,
294
                            playlist_item_t *p_cur )
Clément Stenac's avatar
Clément Stenac committed
295 296
{
    playlist_item_t *p_next = NULL;
297 298
    stats_TimerStart( p_playlist, "Items array build",
                      STATS_TIMER_PLAYLIST_BUILD );
Rémi Duraffort's avatar
Rémi Duraffort committed
299
    PL_DEBUG( "rebuilding array of current - root %s",
300
              PLI_NAME( pl_priv(p_playlist)->status.p_node ) );
Rémi Duraffort's avatar
Rémi Duraffort committed
301
    ARRAY_RESET( p_playlist->current );
Clément Stenac's avatar
Clément Stenac committed
302 303 304 305 306
    p_playlist->i_current_index = -1;
    while( 1 )
    {
        /** FIXME: this is *slow* */
        p_next = playlist_GetNextLeaf( p_playlist,
307
                                       pl_priv(p_playlist)->status.p_node,
308
                                       p_next, true, false );
Clément Stenac's avatar
Clément Stenac committed
309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327
        if( p_next )
        {
            if( p_next == p_cur )
                p_playlist->i_current_index = p_playlist->current.i_size;
            ARRAY_APPEND( p_playlist->current, p_next);
        }
        else break;
    }
    PL_DEBUG("rebuild done - %i items, index %i", p_playlist->current.i_size,
                                                  p_playlist->i_current_index);
    if( b_random )
    {
        /* Shuffle the array */
        srand( (unsigned int)mdate() );
        int j;
        for( j = p_playlist->current.i_size - 1; j > 0; j-- )
        {
            int i = rand() % (j+1); /* between 0 and j */
            playlist_item_t *p_tmp;
Rémi Duraffort's avatar
Rémi Duraffort committed
328
            /* swap the two items */
Clément Stenac's avatar
Clément Stenac committed
329 330 331 332 333
            p_tmp = ARRAY_VAL(p_playlist->current, i);
            ARRAY_VAL(p_playlist->current,i) = ARRAY_VAL(p_playlist->current,j);
            ARRAY_VAL(p_playlist->current,j) = p_tmp;
        }
    }
334
    pl_priv(p_playlist)->b_reset_currently_playing = false;
335
    stats_TimerStop( p_playlist, STATS_TIMER_PLAYLIST_BUILD );
Clément Stenac's avatar
Clément Stenac committed
336 337
}

Rémi Duraffort's avatar
Rémi Duraffort committed
338 339 340 341 342 343 344
/**
 * Compute the next playlist item depending on
 * the playlist course mode (forward, backward, random, view,...).
 *
 * \param p_playlist the playlist object
 * \return nothing
 */
345 346 347
playlist_item_t * playlist_NextItem( playlist_t *p_playlist )
{
    playlist_item_t *p_new = NULL;
348
    int i_skip = 0, i;
349

350 351 352 353
    bool b_loop = var_GetBool( p_playlist, "loop" );
    bool b_random = var_GetBool( p_playlist, "random" );
    bool b_repeat = var_GetBool( p_playlist, "repeat" );
    bool b_playstop = var_GetBool( p_playlist, "play-and-stop" );
354 355 356

    /* Handle quickly a few special cases */
    /* No items to play */
Clément Stenac's avatar
Clément Stenac committed
357
    if( p_playlist->items.i_size == 0 )
358 359 360 361 362 363
    {
        msg_Info( p_playlist, "playlist is empty" );
        return NULL;
    }

    /* Repeat and play/stop */
364
    if( !pl_priv(p_playlist)->request.b_request && b_repeat == true &&
365
         get_current_status_item( p_playlist ) )
366 367
    {
        msg_Dbg( p_playlist,"repeating item" );
368
        return get_current_status_item( p_playlist );
369
    }
370
    if( !pl_priv(p_playlist)->request.b_request && b_playstop == true )
371
    {
Rémi Duraffort's avatar
Rémi Duraffort committed
372
        msg_Dbg( p_playlist,"stopping (play and stop)" );
373 374 375
        return NULL;
    }

376
    if( !pl_priv(p_playlist)->request.b_request &&
377
        get_current_status_item( p_playlist ) )
378
    {
379
        playlist_item_t *p_parent = get_current_status_item( p_playlist );
380 381 382 383 384 385 386 387 388
        while( p_parent )
        {
            if( p_parent->i_flags & PLAYLIST_SKIP_FLAG )
            {
                msg_Dbg( p_playlist, "blocking item, stopping") ;
                return NULL;
            }
            p_parent = p_parent->p_parent;
        }
389 390 391
    }

    /* Start the real work */
392
    if( pl_priv(p_playlist)->request.b_request )
393
    {
394 395
        p_new = pl_priv(p_playlist)->request.p_item;
        i_skip = pl_priv(p_playlist)->request.i_skip;
Clément Stenac's avatar
Clément Stenac committed
396
        PL_DEBUG( "processing request item %s node %s skip %i",
397 398
                        PLI_NAME( pl_priv(p_playlist)->request.p_item ),
                        PLI_NAME( pl_priv(p_playlist)->request.p_node ), i_skip );
399

400 401
        if( pl_priv(p_playlist)->request.p_node &&
            pl_priv(p_playlist)->request.p_node != get_current_status_node( p_playlist ) )
Clément Stenac's avatar
Clément Stenac committed
402
        {
403

404 405
            set_current_status_node( p_playlist, pl_priv(p_playlist)->request.p_node );
            pl_priv(p_playlist)->request.p_node = NULL;
406
            pl_priv(p_playlist)->b_reset_currently_playing = true;
Clément Stenac's avatar
Clément Stenac committed
407
        }
408

409
        /* If we are asked for a node, go to it's first child */
410
        if( i_skip == 0 && ( p_new == NULL || p_new->i_children != -1 ) )
411
        {
412
            i_skip++;
413 414 415 416 417 418 419 420 421 422 423 424 425
            if( p_new != NULL )
            {
                p_new = playlist_GetNextLeaf( p_playlist, p_new, NULL, true, false );
                for( i = 0; i < p_playlist->current.i_size; i++ )
                {
                    if( p_new == ARRAY_VAL( p_playlist->current, i ) )
                    {
                        p_playlist->i_current_index = i;
                        i_skip = 0;
                    }
                }
            }
        }
426

427
        if( pl_priv(p_playlist)->b_reset_currently_playing )
428
            /* A bit too bad to reset twice ... */
Clément Stenac's avatar
Clément Stenac committed
429 430 431 432 433 434
            ResetCurrentlyPlaying( p_playlist, b_random, p_new );
        else if( p_new )
            ResyncCurrentIndex( p_playlist, p_new );
        else
            p_playlist->i_current_index = -1;

435
        if( p_playlist->current.i_size && (i_skip > 0) )
436
        {
437 438
            if( p_playlist->i_current_index < -1 )
                p_playlist->i_current_index = -1;
439 440
            for( i = i_skip; i > 0 ; i-- )
            {
Clément Stenac's avatar
Clément Stenac committed
441
                p_playlist->i_current_index++;
442
                if( p_playlist->i_current_index >= p_playlist->current.i_size )
443
                {
444
                    PL_DEBUG( "looping - restarting at beginning of node" );
Clément Stenac's avatar
Clément Stenac committed
445
                    p_playlist->i_current_index = 0;
446 447
                }
            }
Clément Stenac's avatar
Clément Stenac committed
448 449
            p_new = ARRAY_VAL( p_playlist->current,
                               p_playlist->i_current_index );
450
        }
451
        else if( p_playlist->current.i_size && (i_skip < 0) )
452 453 454
        {
            for( i = i_skip; i < 0 ; i++ )
            {
Clément Stenac's avatar
Clément Stenac committed
455
                p_playlist->i_current_index--;
456
                if( p_playlist->i_current_index <= -1 )
457
                {
458
                    PL_DEBUG( "looping - restarting at end of node" );
Clément Stenac's avatar
Clément Stenac committed
459
                    p_playlist->i_current_index = p_playlist->current.i_size-1;
460 461
                }
            }
Clément Stenac's avatar
Clément Stenac committed
462 463
            p_new = ARRAY_VAL( p_playlist->current,
                               p_playlist->i_current_index );
464 465
        }
        /* Clear the request */
466
        pl_priv(p_playlist)->request.b_request = false;
467 468 469 470
    }
    /* "Automatic" item change ( next ) */
    else
    {
Clément Stenac's avatar
Clément Stenac committed
471 472
        PL_DEBUG( "changing item without a request (current %i/%i)",
                  p_playlist->i_current_index, p_playlist->current.i_size );
473
        /* Cant go to next from current item */
474 475
        if( get_current_status_item( p_playlist ) &&
            get_current_status_item( p_playlist )->i_flags & PLAYLIST_SKIP_FLAG )
476 477
            return NULL;

478
        if( pl_priv(p_playlist)->b_reset_currently_playing )
Clément Stenac's avatar
Clément Stenac committed
479
            ResetCurrentlyPlaying( p_playlist, b_random,
480
                                   get_current_status_item( p_playlist ) );
Clément Stenac's avatar
Clément Stenac committed
481 482

        p_playlist->i_current_index++;
Rafaël Carré's avatar
Rafaël Carré committed
483
        assert( p_playlist->i_current_index <= p_playlist->current.i_size );
Clément Stenac's avatar
Clément Stenac committed
484
        if( p_playlist->i_current_index == p_playlist->current.i_size )
485
        {
Clément Stenac's avatar
Clément Stenac committed
486 487
            if( !b_loop || p_playlist->current.i_size == 0 ) return NULL;
            p_playlist->i_current_index = 0;
488
        }
Clément Stenac's avatar
Clément Stenac committed
489 490 491 492
        PL_DEBUG( "using item %i", p_playlist->i_current_index );
        if ( p_playlist->current.i_size == 0 ) return NULL;

        p_new = ARRAY_VAL( p_playlist->current, p_playlist->i_current_index );
493 494 495 496 497 498 499
        /* The new item can't be autoselected  */
        if( p_new != NULL && p_new->i_flags & PLAYLIST_SKIP_FLAG )
            return NULL;
    }
    return p_new;
}

Rémi Duraffort's avatar
Rémi Duraffort committed
500 501 502
/**
 * Start the input for an item
 *
Antoine Cellerier's avatar
Antoine Cellerier committed
503
 * \param p_playlist the playlist object
Rémi Duraffort's avatar
Rémi Duraffort committed
504 505 506
 * \param p_item the item to play
 * \return nothing
 */
507 508
int playlist_PlayItem( playlist_t *p_playlist, playlist_item_t *p_item )
{
509
    input_item_t *p_input = p_item->p_input;
510
    sout_instance_t **pp_sout = &pl_priv(p_playlist)->p_sout;
Rémi Duraffort's avatar
Rémi Duraffort committed
511
    int i_activity = var_GetInteger( p_playlist, "activity" ) ;
512 513 514

    msg_Dbg( p_playlist, "creating new input thread" );

515
    p_input->i_nb_played++;
516
    set_current_status_item( p_playlist, p_item );
517

518
    pl_priv(p_playlist)->status.i_status = PLAYLIST_RUNNING;
519 520 521

    var_SetInteger( p_playlist, "activity", i_activity +
                    DEFAULT_INPUT_ACTIVITY );
522 523

    input_thread_t * p_input_thread =
524
        input_CreateThreadExtended( p_playlist, p_input, NULL, *pp_sout );
525 526 527
    playlist_set_current_input( p_playlist, p_input_thread );
    vlc_object_release( p_input_thread );

528
    *pp_sout = NULL;
529

530 531 532 533 534 535 536 537 538
    char *psz_uri = input_item_GetURI( p_item->p_input );
    if( psz_uri && ( !strncmp( psz_uri, "directory:", 10 ) ||
                     !strncmp( psz_uri, "vlc:", 4 ) ) )
    {
        free( psz_uri );
        return VLC_SUCCESS;
    }
    free( psz_uri );

539
    if( pl_priv(p_playlist)->fetcher.i_art_policy == ALBUM_ART_WHEN_PLAYED )
540
    {
541
        bool b_has_art;
542

543 544 545 546
        char *psz_arturl, *psz_name;
        psz_arturl = input_item_GetArtURL( p_input );
        psz_name = input_item_GetName( p_input );

Rémi Duraffort's avatar
Rémi Duraffort committed
547
        /* p_input->p_meta should not be null after a successfull CreateThread */
548
        b_has_art = !EMPTY_STR( psz_arturl );
549

550
        if( !b_has_art || strncmp( psz_arturl, "attachment://", 13 ) )
551
        {
552
            PL_DEBUG( "requesting art for %s", psz_name );
553 554
            playlist_AskForArtEnqueue( p_playlist, p_input );
        }
555 556
        free( psz_arturl );
        free( psz_name );
557
    }
558

559
    PL_UNLOCK;
Rémi Denis-Courmont's avatar
Rémi Denis-Courmont committed
560
    var_SetInteger( p_playlist, "playlist-current", p_input->i_id );
561
    PL_LOCK;
562 563 564

    return VLC_SUCCESS;
}