[FFmpeg-devel] [PATCH 1/3] avformat/dvdvideodec: add CLUT utilities and subtitle palette support

[Stefano Sabatini]

On date Wednesday 2024-03-06 01:19:11 -0600, Marth64 wrote:
> Since last iteration: removes unused variable in dvdclut.c
> 
> Signed-off-by: Marth64 <marth64 at proxyid.net>
> ---
>  doc/demuxers.texi         |  5 +++
>  libavformat/Makefile      |  2 +-
>  libavformat/dvdclut.c     | 75 +++++++++++++++++++++++++++++++++++++++
>  libavformat/dvdclut.h     | 37 +++++++++++++++++++
>  libavformat/dvdvideodec.c | 15 ++++++++
>  5 files changed, 133 insertions(+), 1 deletion(-)
>  create mode 100644 libavformat/dvdclut.c
>  create mode 100644 libavformat/dvdclut.h
> 
> diff --git a/doc/demuxers.texi b/doc/demuxers.texi
> index f4bac8f3b3..1a17c6db16 100644
> --- a/doc/demuxers.texi
> +++ b/doc/demuxers.texi
> @@ -394,6 +394,11 @@ often with junk data intended for controlling a real DVD player's
>  buffering speed and with no other material data value.
>  Default is 1, true.
>  

> + at item clut_rgb @var{bool}
> +Output subtitle palettes (CLUTs) as RGB, required for Matroska.
> +Disable to output the palette in its original YUV colorspace.
> +Default is 1, true.

Can you expand about this? When is matroska used in a DVD?

Can we make the setting automatic in case matroska is detected?

[...]

LGTM otherwise.