[FFmpeg-devel] m1v support in ffmpeg libavformat/mov.c

[Reimar Doeffinger]

On Tue, Aug 19, 2008 at 04:56:06PM -0700, Gordon McCreight wrote:
> I have an example file, and have made some changes to the
> libavformat/mov.c source to allow for the processing of the file.  The
> file now processes, however, the stream codec framerate differs
> significantly from the container framerate, so the output video
> appears very short.

Which frame rate is the right one? The container or the stream one?
So far the policy is to assume that the container is always right
(though I do not know if it alway followed properly) -
for the other case it is still possible to just dump the raw video
stream and remux it again or some such.

> Index: libavformat/mov.c
> ===================================================================
> --- libavformat/mov.c   (revision 14856)
> +++ libavformat/mov.c   (working copy)
> @@ -323,6 +323,8 @@
>         st->codec->codec_type = CODEC_TYPE_AUDIO;
>     else if(type == MKTAG('m','1','a',' '))
>         st->codec->codec_id = CODEC_ID_MP2;
> +    else if(type == MKTAG('m','1','v',' '))
> +        st->codec->codec_id = CODEC_ID_MPEG1VIDEO;

Hm, it is better to attach patches, it looks to me like you mailer messed it up a bit
(a space gone missing from the unchanged parts?)...