[FFmpeg-devel] [PATCH 1/7] avcodec/adpcmenc: add "block_size" option

[Zane van Iperen]

Ping, I'll apply in a few days if no objections.

On 14/10/20 11:09 pm, Zane van Iperen wrote:
> 
> Signed-off-by: Zane van Iperen <zane at zanevaniperen.com>
> ---
>   libavcodec/adpcmenc.c | 32 ++++++++++++++++++++++++++++++++
>   1 file changed, 32 insertions(+)
> 

> +static const AVOption options[] = {
> +    {
> +        .name        = "block_size",
> +        .help        = "set the block size",
> +        .offset      = offsetof(ADPCMEncodeContext, block_size),
> +        .type        = AV_OPT_TYPE_INT,
> +        .default_val = {.i64 = BLKSIZE},
> +        .min         = 32,
> +        .max         = 8192, /* FIXME: Reasonable upper limit? */

I'd specifically like feedback on whether this limit is reasonable.

Zane