[FFmpeg-devel] [PATCH] Add CRC check to ALS decoder

Michael Niedermayer michaelni
Sun Jun 13 03:07:17 CEST 2010


On Sat, Jun 12, 2010 at 07:47:14PM +0200, Thilo Borgmann wrote:
> Am 09.06.10 11:53, schrieb Thilo Borgmann:
> > 
> >>> 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
> > files.
> 
> If nobody complains about the code, I'm going to attach after the weekend.

qubit decoherrence detected in /dev/brain
hint: you meant apply not attach i think ;)

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100613/7fcfb0e0/attachment.pgp>



More information about the ffmpeg-devel mailing list