[vlc-devel] [PATCH 1/2] decoder: adding missing continue's

Thomas Guillem thomas at gllm.fr
Mon Feb 10 11:33:31 CET 2020


+1

On Mon, Feb 10, 2020, at 08:02, Steve Lhomme wrote:
> LGTM
> 
> On 2020-02-09 14:34, RĂ©mi Denis-Courmont wrote:
> > If the FIFO was unlocked, the state is no longer known and must be
> > rechecked from the beginning. This is consistent with surrounding code
> > blocks.
> > ---
> >   src/input/decoder.c | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/src/input/decoder.c b/src/input/decoder.c
> > index bc279208ab..e2eac473c7 100644
> > --- a/src/input/decoder.c
> > +++ b/src/input/decoder.c
> > @@ -1695,6 +1695,7 @@ static void *DecoderThread( void *p_data )
> >   
> >               vlc_restorecancel( canc );
> >               vlc_fifo_Lock( p_owner->p_fifo );
> > +            continue;
> >           }
> >   
> >           if( delay != p_owner->delay )
> > @@ -1708,6 +1709,7 @@ static void *DecoderThread( void *p_data )
> >   
> >               vlc_restorecancel( canc );
> >               vlc_fifo_Lock( p_owner->p_fifo );
> > +            continue;
> >           }
> >   
> >           if( p_owner->paused && p_owner->frames_countdown == 0 )
> > -- 
> > 2.25.0
> > 
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
> > 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list