[vlc-devel] [PATCH] fix 3932 reshuffle after playthrough

[Alex Warhawk]

code after line 405 is only executed when the player advances the playlist
automatically (as far as i know)

if you hit the next button, it will be handled as request - putting it
after 405 was my first intention but it didn't work out for hitting the
next button.


On Mon, Sep 30, 2013 at 3:54 AM, Edward Wang
<edward.c.wang at compdigitec.com>wrote:

> On 2013-09-29 21:40, Alex Warhawk <ax.warhawk at gmail.com> wrote:
>
>> +    /* reshuffle (3932) */
>> +    if( p_playlist->i_current_index == p_playlist->current.i_size - 1 ) {
>> +        if( var_GetBool( p_playlist, "random" ) ) {
>> +            PL_DEBUG( "reshuffle playlist" );
>> +            ResetCurrentlyPlaying( p_playlist,
>> +                                   get_current_status_item( p_playlist )
>> );
>> +        }
>> +    }
>>
> Is it possible to have it done at line 405 instead?
>
> > if( p_playlist->i_current_index == p_playlist->current.i_size )
>
> That way the extraneous if statement is not needed and won't affect other
> stuff in the playback than needed.
>
> Regards,
>         Edward Wang
> ______________________________**_________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/**listinfo/vlc-devel<https://mailman.videolan.org/listinfo/vlc-devel>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20130930/7ad49f8e/attachment.html>