[FFmpeg-soc] [soc]: r2856 - aac/aac.c

[Kostya]

On Sun, Jul 27, 2008 at 03:09:10PM +0100, Robert Swain wrote:
> 2008/7/27 Michael Niedermayer <michaelni at gmx.at>:
> > On Sun, Jul 27, 2008 at 11:53:32AM +0200, superdump wrote:
> >> Author: superdump
> >> Date: Sun Jul 27 11:53:32 2008
> >> New Revision: 2856
> >>
> >> Log:
> >> Add doxygen comments for band types
> >>
> >>
> >> Modified:
> >>    aac/aac.c
> >>
> >> Modified: aac/aac.c
> >> ==============================================================================
> >> --- aac/aac.c (original)
> >> +++ aac/aac.c Sun Jul 27 11:53:32 2008
> >> @@ -174,12 +174,12 @@ enum WindowSequence {
> >>  };
> >>
> >>  enum BandType {
> >> -    ZERO_BT        = 0,
> >> +    ZERO_BT        = 0,     ///< Scalefactors and spectral data are all zero.
> >
> >>      FIRST_PAIR_BT  = 5,
> >
> > and this one?
> 
> I couldn't find any specific description from the spec only code
> depending on it. And I couldn't find anything about the order of what
> they call the huffman codebooks to be able to disseminate what the
> 'pair' is to be able to assign it some meaning.

That's easy - first codebooks encode 4 values with one codeword, next encode
only two values with one codeword, so FIRST_PAIR_BT is the first codebook number
which encodes pairs instead of quads.
 
[...]
> >>      ESC_FLAG       = 16,
> >
> > and this?
> 
> Unused, can be removed.

Indeed.
 
> Rob