[FFmpeg-devel] [PATCH] RealMedia muxer: support audio codecs other than AC-3
Sat May 8 09:15:30 CEST 2010
On Mon, 2010-05-03 at 18:39 +0200, Michael Niedermayer wrote:
> On Mon, May 03, 2010 at 03:47:12PM +0100, M?ns Rullg?rd wrote:
> > "Ronald S. Bultje" <rsbultje at gmail.com> writes:
> > > I'm OK with doing it if codec_id is unset (CODEC_ID_NONE). In any
> > > other case, codec_tag should be ignored (imo) and if codec_id is not
> > > in our table, we should error out. This way, we force ourselves to add
> > > the *correct* tag for each codec_id, rather than taking the AVI fourcc
> > > and probably ending up creating files only mplayer can playback.
> > +1
> this breaks stream copy of old xvid and some mpeg4 variants like XVIX
> and ump4
> iam thus not in favor of it
> what you 2 can do if you like to improve the situation is to implement the
> FIXME below from libavformat/utils.c
> //check that tag + id is in the table
> //if neither is in the table -> OK
> //if tag is in the table with another id -> FAIL
> //if id is in the table with another tag -> FAIL unless strict < ?
> st->codec->codec_tag= av_codec_get_tag(s->oformat->codec_tag, st->codec->codec_id);
> I understand you want to fail in a broader set of cases and iam not
> even saying iam neccessarily against that but i think we first should
> implement the cases we agree about. Once thats done we can discuss
> the remaining cases
I'm not Ronald nor Mans, but I think the attached does what you ask for.
I chose strict < normal as the last condition, hope that's fine.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1644 bytes
Desc: not available
More information about the ffmpeg-devel