Rockbox

Tasklist

FS#2973 - PCM AIFF codec for 8-24bit audio

Attached to Project: Rockbox
Opened by Jvo Studer (vinylivo) - Wednesday, 01 February 2006, 14:04 GMT
Last edited by Dave Chapman (linuxstb) - Wednesday, 01 February 2006, 16:52 GMT
Task Type Patches
Category
Status Closed
Assigned To Dave Chapman (linuxstb)
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

This is the codec (actually decoder only) for AIFF
(Audio Interchange File Format) sounds.
At the moment it supports mono or stereo, 8 to 24 bit
resolution.
This is taken from the WAV codec wav.c and adapted for
bigendian readings. No compressed formats supported
(like "AIFC" uLaw, ALaw, ADPCM or others).
Unknown chunks like 'INST' or 'MARK' are skipped.
Accepts *.aif or *.aiff file extensions.

Problems:
- format ('COMM') and data ('SSND') chunks have to be
in this order
- long chunks before 'SSND' prevent the metadata parser
from reaching the actual sound data and returns an error
This task depends upon

Closed by  Dave Chapman (linuxstb)
Wednesday, 01 February 2006, 16:43 GMT
Reason for closing:  Accepted
Comment by Dave Chapman (linuxstb) - Wednesday, 01 February 2006, 16:43 GMT

Nice work on the patch.

There was an endian problem with the 16-bit audio case - it
only worked for big-endian CPUs.

But apart from that, I've committed it to CVS "as is" and
added your name (as it appears in aiff.c) to the Rockbox
CREDITS file.

Thanks.
Comment by Dave Chapman (linuxstb) - Wednesday, 01 February 2006, 16:52 GMT

Re-reading my comment, I realised that I didn't make it
clear that I fixed the endian issue before committing to
cvs. The version in CVS works fine on both big and little
endian targets.

Loading...