On 13/03/15 12:24 PM, Luca Barbato wrote:
> On 13/03/15 16:17, Diego Biurrun wrote:
>> From: Niels Möller <ni...@lysator.liu.se>
>>
>> ---
>>
>> Changes since last round:
>>
>> - XLL disabled by default.
>> - Return error on too many downmix coefficients
>>
>> This has survived Oracle, so it's good to go IMO and will hit the
>> tree very soon, barring last minute comments/objections.
>>
>
> Fine for me.
>
> lu
Valgrind is complaining about this code ("Conditional jump or move depends on
uninitialised
value" error), as seen here
https://fate.libav.org/x86_64-linux-gcc-valgrind/20150316044429
Zero initializing the param_state[16] struct from ff_dca_xll_decode_audio()
with { { 0 } }
"fixes" it, but it's possible it may instead be hiding the real bug in the code.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
