[FFmpeg-devel] [PATCH] avcodec: Require avoptions for the user to set max_pixels.

wm4 nfxjfg at googlemail.com
Sun Dec 11 14:54:28 EET 2016


On Sat, 10 Dec 2016 23:01:04 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> When we will backport this, it will be inevitably in a different location
> in AVCodecContext in each release and master. 3.0, 3.1, 3.2 and master
> use the same soname though and must have a binary compatible interface.
> It thus can only saftely be accessed through AVOptions
> 
> It may be enough to require this only in the releases but that could be
> rather confusing.
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/avcodec.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 02234aee67..8123092ac0 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -3573,8 +3573,8 @@ typedef struct AVCodecContext {
>      /**
>       * The number of pixels per image to maximally accept.
>       *
> -     * - decoding: set by user
> -     * - encoding: set by user
> +     * - decoding: set by user through AVOptions (NO direct access)
> +     * - encoding: set by user through AVOptions (NO direct access)
>       */
>      int64_t max_pixels;
>  

Why? We gave up the Libav ABI compat. abomination.


More information about the ffmpeg-devel mailing list