[Scummvm-cvs-logs] SF.net SVN: scummvm:[54335] scummvm/trunk/backends/platform/ds/arm9/source/ cdaudio.cpp

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Thu Nov 18 18:45:07 CET 2010


Revision: 54335
          http://scummvm.svn.sourceforge.net/scummvm/?rev=54335&view=rev
Author:   fingolfin
Date:     2010-11-18 17:45:07 +0000 (Thu, 18 Nov 2010)

Log Message:
-----------
DS: Add note that ARM_ADPCM is unused (and fix a typo)

Modified Paths:
--------------
    scummvm/trunk/backends/platform/ds/arm9/source/cdaudio.cpp

Modified: scummvm/trunk/backends/platform/ds/arm9/source/cdaudio.cpp
===================================================================
--- scummvm/trunk/backends/platform/ds/arm9/source/cdaudio.cpp	2010-11-18 17:42:28 UTC (rev 54334)
+++ scummvm/trunk/backends/platform/ds/arm9/source/cdaudio.cpp	2010-11-18 17:45:07 UTC (rev 54335)
@@ -254,7 +254,12 @@
 }
 
 #ifdef ARM_ADPCM
-extern "C" void ARM_adcpm(int *block, int len, int stepTableIndex,
+// FIXME: This code, as well as the source file adpcm_arm.s, are
+// apparently unused. Maybe that is a mistake? Or maybe there is a bug
+// in ARM_adpcm (then this should be reported and fixed). Or maybe there
+// are other good reasons to prefer the C code, but then this as well as
+// the assembler source file should be removed.
+extern "C" void ARM_adpcm(int *block, int len, int stepTableIndex,
                           int firstSample, s16 *decompressionBuffer);
 #endif
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list