[FFmpeg-devel] [PATCH]lavf/isom: Recognize fourcc HEVC

[James Almer]

On 5/2/2018 3:44 PM, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes ticket #7110 for me.

hevc in AVI is not a thing, and adding a non official fourcc to
ff_codec_movvideo_tags to support weird files created by faulty software
that are not even mov/mp4 is not a good idea.

Why the hell is the avi demuxer even looking at mov tags to begin with?

> 
> Please comment, Carl Eugen
> 
> 
> 0001-lavf-isom-Recognize-fourcc-HEVC.patch
> 
> 
> From 281899c95c5b94242fa2bdbb4e830eecaef78f8d Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg at gmail.com>
> Date: Wed, 2 May 2018 20:43:09 +0200
> Subject: [PATCH] lavf/isom: Recognize fourcc HEVC.
> 
> Fixes ticket #7110.
> ---
>  libavformat/isom.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/isom.c b/libavformat/isom.c
> index 2792371..261fc6f 100644
> --- a/libavformat/isom.c
> +++ b/libavformat/isom.c
> @@ -163,6 +163,7 @@ const AVCodecTag ff_codec_movvideo_tags[] = {
>  
>      { AV_CODEC_ID_HEVC, MKTAG('h', 'e', 'v', '1') }, /* HEVC/H.265 which indicates parameter sets may be in ES */
>      { AV_CODEC_ID_HEVC, MKTAG('h', 'v', 'c', '1') }, /* HEVC/H.265 which indicates parameter sets shall not be in ES */
> +    { AV_CODEC_ID_HEVC, MKTAG('H', 'E', 'V', 'C') }, /* VirtualDub */
>  
>      { AV_CODEC_ID_H264, MKTAG('a', 'v', 'c', '1') }, /* AVC-1/H.264 */
>      { AV_CODEC_ID_H264, MKTAG('a', 'v', 'c', '2') },
> -- 1.7.10.4
> 
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>