[FFmpeg-devel] [PATCH] avformat/async: allow to set buffer size [v2]

Michael Niedermayer michael at niedermayer.cc
Sun Sep 24 02:58:50 EEST 2017


On Fri, Sep 22, 2017 at 04:25:33PM +0900, Takayuki 'January June' Suwa wrote:
> - add "-asyncbufsize" option to async:
> - refactor async.c
>   - remove READ_BACK_CAPACITY and the 3rd arg of ring_init()
>   - replace some magicnums with symconst macros

This should be several patches as they are independant changes

[...]
> @@ -466,6 +469,7 @@ static int64_t async_seek(URLContext *h, int64_t pos, int whence)
>  #define D AV_OPT_FLAG_DECODING_PARAM
>  
>  static const AVOption options[] = {

> +    { "asyncbufsize", "Amount in bytes that may be used for asynchronous data buffering", OFFSET(asyncbufsize), AV_OPT_TYPE_INT, { .i64 = BUFFER_CAPACITY }, 0, INT_MAX, D },

the minimum should probably be BUFFER_CAPACITY_MIN and not 0

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170924/945e6bf6/attachment.sig>


More information about the ffmpeg-devel mailing list