[Ffmpeg-devel] [PATCH] aiff/aiff-c format encoder and decoder

Patrick Guimond patg
Mon Jan 30 23:52:21 CET 2006


Michael Niedermayer wrote:
> 
> the uint8_t buffer is ok, but i think the code should use ldexp/frexp like the
> others
[...]
> 
> segfault due to 64bit alignment on some architecture
[...]
> 
> IIRC NAN/INFINITY are not defined everywere, 0.0/0.0 is used by the others
> so thats probably safer
[...]
> breaks if the system doesnt use IEEE doubles or if the int and double byte 
> ordering differs


Considered those issues in this patch, any comments welcome

patg

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-aiff.patch
Type: text/x-patch
Size: 18527 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060130/8e428799/attachment.bin>



More information about the ffmpeg-devel mailing list