[FFmpeg-devel] [PATCH] add dca and vc1 objects to Makefile when parsers are enabled
Thu May 3 13:24:14 CEST 2007
On Thu, May 03, 2007 at 12:14:40PM +0200, Benoit Fouet wrote:
> Michael Niedermayer wrote:
> > Hi
> > On Thu, May 03, 2007 at 11:54:56AM +0200, Diego Biurrun wrote:
> >> On Thu, May 03, 2007 at 09:28:57AM +0200, Benoit Fouet wrote:
> >>> seen on ffmpeg user mailing list
> >>> when disabling en/decoders, those two objects are not built even though
> >>> they are needed to parse
> >> The patch is OK, but I think it would be better to put the dca and vc1
> >> parser code in parser.c along with the rest of the parsers. The parser
> >> code is a very small part of both decoders.
> > _all_ parsers should be put in seperate files. putting them in parser.c
> > is an extreemly ugly hack and while i agreed for ac3 as noone was willing
> > to solve that properly i will NOT agree to move the other parsers to the
> > wrong file too!
> so the correct solution is to create, for instance:
> dca_parser.c and vc1_parser.c ?
putting them in parser.c is like putting decoders in utils.c/avcodec.h
or like putting (small) video filters into the files which are supposed
to only contain the core video filter code ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the ffmpeg-devel