[FFmpeg-devel] [PATCH] lavfi/af_atempo: use av_malloc for rDFT buffers.
Pavel Koshevoy
pkoshevoy at gmail.com
Wed Aug 29 02:56:07 CEST 2012
On 08/28/2012 10:58 AM, Nicolas George wrote:
> Le duodi 12 fructidor, an CCXX, Reimar Döffinger a écrit :
>> I can't see why the data for the other cases would need to be preserved?
>> So you could just make all use this code.
> The other fields seem to be filled progressively by samples from the input.
> Of course, currently lavfi does not support reconfiguration, but it seems
> that the design is specifically made to support it.
>
> I would like Pavel's advice on that, though.
You can switch everything to RE_MALLOC...
If you look at the second-to-last line of yae_reset(..) it calls yae_clear,
which resets both fragments to initial state. So, all prior data is dumped.
The only reason I used realloc was to help reduce memory fragmentation.
Although, that's probably not an issue for this filter.
>> Btw. the double () around field_size looks a bit ugly.
> It is indeed not necessary for a single argument. I will remove it before
> pushing anything.
Thank you,
Pavel.
More information about the ffmpeg-devel
mailing list