[vlc-devel] [PATCH] HLS: minor fix

Hugo Beauzée-Luyssen hugo at beauzee.fr
Mon Jun 25 19:01:21 CEST 2012


On Tue, Jun 19, 2012 at 5:56 PM, Frédéric Yhuel <fyhuel at viotech.net> wrote:
> It seems to me it makes more sense like that.
> ---
>  modules/stream_filter/httplive.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/modules/stream_filter/httplive.c b/modules/stream_filter/httplive.c
> index 5758189..2c0d603 100644
> --- a/modules/stream_filter/httplive.c
> +++ b/modules/stream_filter/httplive.c
> @@ -2180,13 +2180,13 @@ static ssize_t hls_Read(stream_t *s, uint8_t *p_read, unsigned int i_read)
>             else
>                 segment_RestorePos(segment);
>
> -            p_sys->playback.segment++;
>             vlc_mutex_unlock(&segment->lock);
>
>             /* signal download thread */
>             vlc_mutex_lock(&p_sys->download.lock_wait);
> -            vlc_cond_signal(&p_sys->download.wait);
> +            p_sys->playback.segment++;
>             vlc_mutex_unlock(&p_sys->download.lock_wait);
> +            vlc_cond_signal(&p_sys->download.wait);
>             continue;
>         }
>
> --
> 1.7.9.5
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel

Works for me.

However, playback is still accessed way too much in an non thread-safe
manner. But that's a start!

Regards,

-- 
Hugo Beauzée-Luyssen
mail: hugo at beauzee.fr
skype: beauze.h



More information about the vlc-devel mailing list