[vlc-devel] [PATCH] HLS: hls_Download: don't put stream_Read in a loop
fyhuel at viotech.net
Fri Mar 30 11:16:43 CEST 2012
On Fri, Mar 30, 2012 at 11:02 AM, Jean-Paul Saman <jpsaman at videolan.org> wrote:
> On Fri, Mar 30, 2012 at 10:48 AM, Frederic YHUEL <fyhuel at viotech.net> wrote:
>> On Fri, Mar 30, 2012 at 10:24 AM, Jean-Paul Saman <jpsaman at videolan.org> wrote:
>>> On Thu, Mar 29, 2012 at 2:34 PM, Frédéric Yhuel <fyhuel at viotech.net> wrote:
>>>> Maybe I missed something, but it seems to me that either that patch must
>>>> be applied, either "stream_Read" documentation must be fixed.
>>> May I ask why this is necessary?
>> Not necessary but cleaner.
>>> I recall that some HLS servers report
>>> 0 as segment size.
>> Ok my bad, I did not understand. Would you put that explanation as a
>> comment? It would be useful for those who read the code IMHO.
> Would this suffice?
> /* NOTE: Beware the size reported for a segment by the HLS
> server may not
> * be correct, when downloading the segment data. There for
> check the size
> * and enlarge the segment data block if necessary.
Except for the typo (s/There for/Therefore) I think it's fine. Thanks!
More information about the vlc-devel