Justin Ruggles <justin.ruggles@...> writes:

> 
> ---
> Added some better error code handling for known libwebp errors.
> Added a suggestion to the user to pass in a different pixel
> format instead of allowing libwebp to convert it.
> 
>  Changelog               |   1 +
>  configure               |   4 +
>  doc/general.texi        |   4 +-
>  libavcodec/Makefile     |   1 +
>  libavcodec/allcodecs.c  |   1 +
>  libavcodec/libwebpenc.c | 224 
++++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/version.h    |   2 +-
>  libavformat/img2enc.c   |   2 +-
>  8 files changed, 235 insertions(+), 4 deletions(-)
>  create mode 100644 libavcodec/libwebpenc.c

Maybe you'd want to add a Texinfo documentation to doc/encoders.texi?

[...]

> +#define OFFSET(x) offsetof(LibWebPContext, x)
> +#define VE AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_ENCODING_PARAM
> +static const AVOption options[] = {
> +    { "lossless",   "Use lossless mode",       OFFSET(lossless), 
AV_OPT_TYPE_INT,   { .i64 =  0 },  0, 1,                           VE           
},

> +    { "method",     "quality/speed trade-off", OFFSET(method),   
AV_OPT_TYPE_INT,   { .i64 =  4 },  0, 6,                           VE           
},
> +    { "quality",    "lossy quality",           OFFSET(quality),  
AV_OPT_TYPE_FLOAT, { .dbl = 75 },  0, 100,                         VE           
},

Why don't you use avctx->compression_level and a avctx->global_quality
for these 2 options?

[...]

Timothy

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to