[FFmpeg-soc] EAC3 review

Justin Ruggles justinruggles at bellsouth.net
Tue Sep 11 04:57:14 CEST 2007


Justin Ruggles wrote:
> Michael Niedermayer wrote:
>>>         ff_ac3_bit_alloc_calc_psd((int8_t *)s->dexps[ch], start, end,
>>>                 s->psd[ch], s->bndpsd[ch]);
>> why the int8_t cast?
> 
> That's leftover from Fabrice's original AC3 encoder, which uses int8_t 
> for exponents.  That function is shared between the encoder and decoder. 
>   I see no reason why it can't be changed, but it would have to be 
> across-the-board.

Oh wait, nevermind.  I saw this in my last review.  The variable should 
be changed in the context to be int8_t and the cast would not be needed. 
  Or again we could just change it everywhere to plain int.

-Justin



More information about the FFmpeg-soc mailing list