• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category
  • Assigned To
  • Operating System
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by vinylivo - 2006-02-01
Last edited by linuxstb - 2006-02-01

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

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
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.

- 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

Closed by  linuxstb
2006-02-01 16:43
Reason for closing:  Accepted

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


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.


Available keyboard shortcuts


Task Details

Task Editing