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

Michael Niedermayer michael at niedermayer.cc
Sun Dec 11 00:01:04 EET 2016


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;
 
-- 
2.11.0



More information about the ffmpeg-devel mailing list