[FFmpeg-devel] [PATCH v5] lavc/libvpxenc: add screen-content-mode option

[James Zern]

On Mon, Feb 12, 2024 at 10:34 PM Dariusz Marcinkiewicz via
ffmpeg-devel <ffmpeg-devel at ffmpeg.org> wrote:
>
> This exposes VP8E_SET_SCREEN_CONTENT_MODE option from libvpx.
>
> Co-authored-by: Erik Språng <sprang at webrtc.org>
> Signed-off-by: Dariusz Marcinkiewicz <darekm at google.com>
> ---
>  doc/encoders.texi      |  6 ++++++
>  libavcodec/libvpxenc.c | 13 +++++++++++++
>  libavcodec/version.h   |  2 +-
>  3 files changed, 20 insertions(+), 1 deletion(-)
>

lgtm. I'll submit this soon if there are no other comments.

> diff --git a/doc/encoders.texi b/doc/encoders.texi
> index c9fe6d6143..13a7084512 100644
> --- a/doc/encoders.texi
> +++ b/doc/encoders.texi
> @@ -2237,6 +2237,12 @@ the two temporal layer 2 frames within the temporal period.
>  @end table
>  @end table
>
> + at item VP8-specific options
> + at table @option
> + at item screen-content-mode
> +Screen content mode, one of: (0) off, (1) screen, (2) screen with more aggressive rate control.

I swapped the ()s: 0 (off) ...