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

[Edward Wang]

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