[FFmpeg-soc] [soc]: r5570 - in amr: amrnbdata.h amrnbdec.c

[Ronald S. Bultje]

Hi,

On Fri, Jan 8, 2010 at 10:19 PM, Vitor Sessak <vitor1001 at gmail.com> wrote:
> I think that this is only guaranteed to work if you declare the AMRNBFrame
> struct as packed, since the spec allows the compiler to add some padding
> space inside structs otherwise.

I'm not sure that's true (at least in practice), since practically
AMRNBFrame is a uint16_t[]. I'd say this is OK in principle (although
I have to go through it a little to see if this is actually easier).

Ronald