[FFmpeg-devel] [PATCH] Add CRC check to ALS decoder
Sun Jun 13 18:55:22 CEST 2010
Thilo Borgmann wrote:
> Am 12.06.10 21:08, schrieb Justin Ruggles:
>> Thilo Borgmann wrote:
>>>>> CRC check for the ALS decoder.
>>>>> Works fine on my Intel-driven architecture. Any tests on other archs are
>>>>> of course appreciated.
>>>> I'm pretty sure the CRC for 24-bit source files are calculated based on
>>>> 3-byte samples, not 4-byte samples. I don't think your patch would work
>>>> in this case. Have you tested this with 24-bit source files?
>>> Sorry, indeed not. Revision 1 is tested against the 24 bit conformance
>>> Still, I would appreciate if someone can test it on a non-intel arch
>>> (before FATE does...).
>> Maybe the patch could be simplified and some macros avoided by doing
>> something like:
>> int swap = HAVE_BIGENDIAN != sconf->msb_first;
>> if (swap)
> Good idea.
You can use the same logic in the allocation.
More information about the ffmpeg-devel