Rockbox

  • Status Closed
  • Percent Complete
    0%
  • Task Type Patches
  • Category
  • Assigned To
    linuxstb
  • 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
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

   aiffcodec.patch (16.4 KiB)
Closed by  linuxstb
2006-02-01 16:43
Reason for closing:  Accepted
linuxstb commented on 2006-02-01 16:43

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.

linuxstb commented on 2006-02-01 16:52

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

Available keyboard shortcuts

Tasklist

Task Details

Task Editing