[vlc-devel] [PATCH 2/8] modules/stream_extractor: added libarchive extractor

Filip Roséen filip at atch.se
Tue Jan 24 03:38:22 CET 2017


Hi again,

On 2017-01-23 15:58, Filip Roséen wrote:

> > > +    uint8_t buffer[ 8096 ];
> > 
> > Any reason for this value?
> 
> Other than being `2^N` and a reasonable size in terms of buffering,
> not really. Do you have some other buffer size in mind?

I just got an email from *Nick Briggs* that noted that `8096` of
course isn't `2^N`, the value should be `8192` for the reasoning to
make sense.

I honestly did not notice the typo related to `4096` and `8192`,
though I will reply with a revised patch shortly (that includes the
necessary change in question).

Best Regards,\
Filip Roséen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170124/c17113e9/attachment.html>


More information about the vlc-devel mailing list