[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