diff mbox series

[FFmpeg-devel,3/3] avformat/riff: map AYUV fourcc to RAWVIDEO decoder

Message ID 20220807003456.54933-2-jamrial@gmail.com
State New
Headers show
Series [FFmpeg-devel] swscale/output: add VUYA output support | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 fail Make fate failed
andriy/make_x86 success Make finished
andriy/make_fate_x86 fail Make fate failed

Commit Message

James Almer Aug. 7, 2022, 12:34 a.m. UTC
There's no need to keep using a custom decoder for this pixel format.

Signed-off-by: James Almer <jamrial@gmail.com>
---
What's the process to remove a decoder? Deprecation period, or just git rm?

 libavformat/riff.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

James Almer Aug. 7, 2022, 12:37 a.m. UTC | #1
On 8/6/2022 9:34 PM, James Almer wrote:
> There's no need to keep using a custom decoder for this pixel format.
> 
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
> What's the process to remove a decoder? Deprecation period, or just git rm?

Actually, it's a decoder, encoder, *and* the AVCodecID, the latter which 
definitely needs a deprecation period, so might as well do it with all 
three.

> 
>   libavformat/riff.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/riff.c b/libavformat/riff.c
> index df7e9df31b..7a97cf1ccf 100644
> --- a/libavformat/riff.c
> +++ b/libavformat/riff.c
> @@ -237,6 +237,7 @@ const AVCodecTag ff_codec_bmp_tags[] = {
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('U', 'Y', 'V', 'Y') },
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('V', 'Y', 'U', 'Y') },
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('I', 'Y', 'U', 'V') },
> +    { AV_CODEC_ID_RAWVIDEO,     MKTAG('A', 'Y', 'U', 'V') },
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('Y', '8', '0', '0') },
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('Y', '8', ' ', ' ') },
>       { AV_CODEC_ID_RAWVIDEO,     MKTAG('H', 'D', 'Y', 'C') },
> @@ -302,7 +303,6 @@ const AVCodecTag ff_codec_bmp_tags[] = {
>       { AV_CODEC_ID_V210,         MKTAG('C', '2', '1', '0') },
>       { AV_CODEC_ID_V308,         MKTAG('v', '3', '0', '8') },
>       { AV_CODEC_ID_V408,         MKTAG('v', '4', '0', '8') },
> -    { AV_CODEC_ID_AYUV,         MKTAG('A', 'Y', 'U', 'V') },
>       { AV_CODEC_ID_V410,         MKTAG('v', '4', '1', '0') },
>       { AV_CODEC_ID_YUV4,         MKTAG('y', 'u', 'v', '4') },
>       { AV_CODEC_ID_INDEO3,       MKTAG('I', 'V', '3', '1') },
diff mbox series

Patch

diff --git a/libavformat/riff.c b/libavformat/riff.c
index df7e9df31b..7a97cf1ccf 100644
--- a/libavformat/riff.c
+++ b/libavformat/riff.c
@@ -237,6 +237,7 @@  const AVCodecTag ff_codec_bmp_tags[] = {
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('U', 'Y', 'V', 'Y') },
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('V', 'Y', 'U', 'Y') },
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('I', 'Y', 'U', 'V') },
+    { AV_CODEC_ID_RAWVIDEO,     MKTAG('A', 'Y', 'U', 'V') },
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('Y', '8', '0', '0') },
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('Y', '8', ' ', ' ') },
     { AV_CODEC_ID_RAWVIDEO,     MKTAG('H', 'D', 'Y', 'C') },
@@ -302,7 +303,6 @@  const AVCodecTag ff_codec_bmp_tags[] = {
     { AV_CODEC_ID_V210,         MKTAG('C', '2', '1', '0') },
     { AV_CODEC_ID_V308,         MKTAG('v', '3', '0', '8') },
     { AV_CODEC_ID_V408,         MKTAG('v', '4', '0', '8') },
-    { AV_CODEC_ID_AYUV,         MKTAG('A', 'Y', 'U', 'V') },
     { AV_CODEC_ID_V410,         MKTAG('v', '4', '1', '0') },
     { AV_CODEC_ID_YUV4,         MKTAG('y', 'u', 'v', '4') },
     { AV_CODEC_ID_INDEO3,       MKTAG('I', 'V', '3', '1') },