[FFmpeg-devel] [PATCH] riff: set bits_per_coded_sample foratrac3

[Paul B Mahol]

On 12/27/12, Piotr Bandurski <ami_stuff at o2.pl> wrote:
> Hi,
>
>> Does atrac3 really use 16 bps? It outputs float, this gives 50%
>> compression ratio.
>>
>> Size of atrac3 frames are fixed and each frame have fixed number of
>> samples,
>> but there is catch, frame can have subframes.
>>
>> Atrac3 decoder use block_align as size of singe frame.
>
> Sorry, I did 2 changes at once in the source and misleaded myself.
> Attached seems to fix the real duration for atrac3 in wav, but it's probably
> not enough.

Atrac3 have subframes. So code that use code you changed thinks that
packet feed to decoder is always producing 1024 samples.
Your patch likely breaks duration for other containters that use atrac3.