[FFmpeg-devel] [PATCH] AAC Decoder round 4

Kostya kostya.shishkov
Wed Jul 30 16:06:52 CEST 2008


On Wed, Jul 30, 2008 at 03:02:49PM +0200, Michael Niedermayer wrote:
> On Wed, Jul 30, 2008 at 01:54:16PM +0100, Robert Swain wrote:
[...]
> > Kostya has said that he would like these changes reverted as he uses
> > these elements of these data structures in his code.
> 
> He can just put these 2 fields in the 2 structs back with his encoder patch.
> But as long as they are not used i will not approve unused variables, its
> far too hard to keep track of what might be used one day and then check back
> if that actually happened or if we ended up with some permamently unused
> fields in structs.
 
fine with me
I'd like to submit my encoder for review ASAP and it's hard to do without
stable (i.e. fully committed) AAC decoder in SVN.
 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> I know you won't believe me, but the highest form of Human Excellence is
> to question oneself and others. -- Socrates




More information about the ffmpeg-devel mailing list