[FFmpeg-devel] [PATCH] seperate parser from codec selection

Paul Kendall paul
Tue May 26 08:56:14 CEST 2009


On Tuesday 26 May 2009 02:02:09 pm Michael Niedermayer wrote:
> On Sun, May 17, 2009 at 03:25:22PM +1200, Paul Kendall wrote:
> > This patch allows an AVStream to specify a parser_id separately to the
> > codec_id. This will be useful for streams such as DVB-T which has AAC
> > audio which may be contained in an LATM multiplexed stream.
> >
> > The patch will fall-back to using the codec_id for parser selection if
> > the parser_id is set to CODEC_ID_NONE, which is the default.
>
> I do not like having 2 codec ids.
>
> [...]
>
> >      enum CodecID codec_id; /* see CODEC_ID_xxx */
> > +    enum CodecID parser_id; /* see CODEC_ID_xxx */
>
> even less so without any documentation
>
> [...]
Micheal, would it be preferable to make a new enum for parsers such as 
PARSER_ID_xxx?

I can look at this, but I'm not really familiar with how the configure script
and Makefiles all work together for this, but I guess I can only try :-)

Cheers,
Paul




More information about the ffmpeg-devel mailing list