[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