[Ffmpeg-cvslog] r5886 - trunk/libavformat/gxf.c

Reimar Döffinger Reimar.Doeffinger
Wed Aug 2 11:57:48 CEST 2006


Hello,
On Wed, Aug 02, 2006 at 10:45:10AM +0200, bcoudurier wrote:
> Log:
> do not set unneeded fourcc
[...]
>          case 22:
>          case 23:
>              st->codec->codec_type = CODEC_TYPE_VIDEO;
>              st->codec->codec_id = CODEC_ID_MPEG1VIDEO;
> -            st->codec->codec_tag = MKTAG('M', 'P', 'G', '1');
>              break;
>          case 9:
>              st->codec->codec_type = CODEC_TYPE_AUDIO;

Hmm, didn't see before, but any reason why you didn't remove codec_tag
for the audio codecs? I only have PCM samples to test, but I think there
is no reason to keep them, and I like being consistent.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-cvslog mailing list