[Ffmpeg-devel] [PATCH] avctx->frame_size for FAAD

[Måns Rullgård]

Andrew Voznytsa said:
> Guillaume Poirier wrote:
>>>
>>> +    avctx->frame_size = 1024; // faad does not support other frame size
>>>
>>
>> Very well... but shouldn't this be considered as a bug in FAAC that
>> should be fixed upstream rather than worked-around here?
>>
>> ... or is it a feature?
>>
>
> There are a few points:
> a) it is better than 0 which was previously there, so it improves
> situation at all
> b) taking into account that FAAD does not support other frame size it
> won't broke 3rd party software which uses FAAD through lavc

Last I checked, faad supported 960 samples per frame.  I've never seen such
a file though.

-- 
M?ns Rullg?rd
mru at inprovide.com