Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#5511 - Fix for codec failure on simulator with voice enabled

Attached to Project: Rockbox
Opened by Steve Bavin (pondlife) - Wednesday, 07 June 2006, 13:59 GMT+2
Task Type Patches
Category Simulator
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

I don't know if it's just me, but there seems to be a general problem running the (SDL) simulator under Windows with file/dir voicing enabled, along with crossfade. This combination results in "Codec Failure" errors because the simulator attempts to create/overwrite _temp_codec.dll while it is still in use. As a workaround, this patch will attempt to use a different filename (_temp_codecn, where n is in range 1 to 999) so that the DLL hackery always succeeds while multiple codecs are loaded.
   sim_multi_codec.patch (1.4 KiB) 
 uisimulator/common/io.c |   18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

This task depends upon

Closed by  Peter D'Hoye (petur)
Saturday, 17 June 2006, 12:57 GMT+2
Reason for closing:  Fixed
Comment by Steve Bavin (pondlife) - Thursday, 08 June 2006, 09:57 GMT+2
Code policed myself - got to get used to the Rockbox bracket standard...
   sim_multi_codec.patch (1.4 KiB) 
 uisimulator/common/io.c |   17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

Comment by Linus Nielsen Feltzing (linusnielsen) - Thursday, 08 June 2006, 11:19 GMT+2
FYI, there is no "bracket standard" in Rockbox. All you need to do is to maintain the style in the file you're editing.
Comment by Steve Bavin (pondlife) - Thursday, 15 June 2006, 13:14 GMT+2
Thanks for the clarification Linus. Don't suppose you'd consider commiting this patch as it's pretty safe and needed by anyone who wants to attempt hacking at the playback/voice code under the sim?
Comment by Steve Bavin (pondlife) - Saturday, 17 June 2006, 12:50 GMT+2
This (or Hardeep's version of it) is now in CVS and can be closed.

Loading...