[vlc-devel] httplive bug fixes

[Sergio M. Ammirata, Ph.D.]

Jean-Paul,

I think it is important that this fix makes it to 1.2 (through this patch
or in your own way). Without it, playback from most live sites will not
work!

Regards,

Sergio

On 10/10/11 6:59 AM, "Jean-Paul Saman" <jpsaman at gmail.com> wrote:

>Hi,
>
>On Wed, Oct 5, 2011 at 11:47 AM, Jean-Paul Saman <jpsaman at gmail.com>
>wrote:
>> On Mon, Oct 3, 2011 at 8:02 PM, Sergio M. Ammirata, Ph.D.
>> <sergio at ammirata.net> wrote:
>>>  Here is a modified patch without the streamindex parameter.
>>
>> Thanks, I will have a look at this.
>
>I had a closer look at your patch and concluded that it is not
>correct. You have found several issues which needs correcting, but not
>in the way your patch does.
>
>For instance:
>- reloading the HLS playlist is only done for live streams not for VOD
>streams. Your patch assumes it does that for VOD streams too. This
>assumption is wrong.
>- line 1055: vlc_UrlParse(&segment->url, p->url.psz_path, 0); is not
>correct. It should parse the entire url not only the psz_path part.
>The original line is wrong too.
>  A new URL should be constructed with psz_url = ConstructUrl(&p->url);
>
>Kind Regards,
>
>Jean-Paul Saman
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>http://mailman.videolan.org/listinfo/vlc-devel