On Sat, Feb 10, 2024 at 12:22 AM Dariusz Marcinkiewicz via ffmpeg-devel <ffmpeg-devel@ffmpeg.org> wrote: > > This exposes VP8E_SET_SCREEN_CONTENT_MODE option from libvpx. > > Co-authored-by: Erik Språng <spr...@webrtc.org> > Signed-off-by: Dariusz Marcinkiewicz <dar...@google.com> > --- > doc/encoders.texi | 3 +++ > libavcodec/libvpxenc.c | 11 +++++++++++ > libavcodec/version.h | 2 +- > 3 files changed, 15 insertions(+), 1 deletion(-) > > diff --git a/doc/encoders.texi b/doc/encoders.texi > index c9fe6d6143..0868aa66db 100644 > --- a/doc/encoders.texi > +++ b/doc/encoders.texi > @@ -2150,6 +2150,9 @@ of quality. > Set a change threshold on blocks below which they will be skipped by the > encoder. > > +@item screen-content-mode > +Screen content mode, one of: off (0), screen (1), screen with more > aggressive rate control (2). > +
Move this above 'VP9-specific options' and add a section for VP8. Also reword this to '...one of: 0 (off), 1 (screen), ...'. The parameter is an integer so should list that first, similar to aq-mode. > @item slices (@emph{token-parts}) > Note that FFmpeg's @option{slices} option gives the total number of > partitions, > while @command{vpxenc}'s @option{token-parts} is given as > diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c > index 80988a2608..0d507beaae 100644 > --- a/libavcodec/libvpxenc.c > +++ b/libavcodec/libvpxenc.c > @@ -102,6 +102,7 @@ typedef struct VPxEncoderContext { > #define VP8F_AUTO_ALT_REF 0x00000002 ///< Enable automatic alternate > reference frame generation > > int auto_alt_ref; > + int screen_content_mode; > Move this above '// VP9-only' and add '// VP8-only'. > [...] > @@ -1946,6 +1956,7 @@ static const AVOption vp8_options[] = { > { "auto-alt-ref", "Enable use of alternate reference " > "frames (2-pass only)", > OFFSET(auto_alt_ref), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 2, VE}, > { "cpu-used", "Quality/Speed ratio modifier", > OFFSET(cpu_used), AV_OPT_TYPE_INT, {.i64 = 1}, -16, 16, VE}, > + { "screen-content-mode", "Encoder screen content mode", > OFFSET(screen_content_mode), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 2, VE, > "screen_content_mode"}, Remove the last item, "screen_content_mode". This no longer describes a group of options. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".