[vlc-devel] [PATCH 2/3] dash added byte range requests

Christopher Müller christopher.mueller at itec.uni-klu.ac.at
Sun Jan 29 10:51:11 CET 2012

> Von: vlc-devel-bounces at videolan.org [mailto:vlc-devel-
> bounces at videolan.org] Im Auftrag von Hugo Beauzée-Luyssen
> Gesendet: Sonntag, 29. Jänner 2012 01:57
> An: Mailing list for VLC media player developers
> Betreff: Re: [vlc-devel] [PATCH 2/3] dash added byte range requests
> 2012/1/28 Christopher Müller <christopher.mueller at itec.uni-klu.ac.at>:
> >
> >
> >
> Hello,
> If I'm not mistaking, this patch makes HttpConnection use net_* instead
> of the old stream_* Maybe this should be split in another patch. Also,
> some attributes seems to be unused afterward
> (HttpConnection::urlStream) you should probably remove them.
> When you delete this->peekBuffer, you must use delete[] as it was
> allocated with new[] The class destructor should probably ensure the
> connection is closed, if closeSocket wasn't called, it feels more
> natural.

Yes it uses net_* due to the fact that stream_* does not support byte ranges. Comments are added thx.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-dash-added-byte-range-requests.patch
Type: application/octet-stream
Size: 11295 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120129/d424a7b2/attachment.obj>

More information about the vlc-devel mailing list