Rockbox

Tasklist

FS#2860 - simulator sdl patch

Attached to Project: Rockbox
Opened by Nick Lanham (mikerackhabit) - Thursday, 22 December 2005, 01:15 GMT
Last edited by Daniel Stenberg (bagder) - Monday, 09 January 2006, 11:24 GMT
Task Type Patches
Category Simulator
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

sdl patch for the simulator, shouldn't hurt any of the
other sims. should work for all players and supports
sound. uses sdl threads.

probably only compiles on linux as the makefiles are
right now, to get it to work on windows you'll need to
change all the #include <SDL/SDL.h> lines to just read
#include <SDL.h>

on linux you will also need to:

> cd uisimulator/sdl
> cp ../win32/*.bmp .

otherwise your make will fail looking for the bmp for
your player.
This task depends upon

Closed by  Daniel Stenberg (bagder)
Monday, 09 January 2006, 11:24 GMT
Reason for closing:  Accepted
Additional comments about closing:  Logged In: YES
user_id=1110

I applied and committed Nathan's version that used parts of
yours - you were both added to the CREDITS file due to this.
I hope that we can continue improving the simulator from the
now committed code. Thanks.
Comment by Daniel Stenberg (bagder) - Thursday, 22 December 2005, 07:56 GMT

Thanks!

Uh, did you by any chance check/compare with patch #1385137
"sdl uisimulator" by Nathan Hand - nathanh?

It would be nice to get a unified single SDL approach...
Comment by Nick Lanham (mikerackhabit) - Thursday, 22 December 2005, 19:58 GMT

yeah, me and nathan chatted a bit about this. we're going
to look at each others' patches and try and get the best
bits of each one into a unified patch.
Comment by Daniel Stenberg (bagder) - Monday, 09 January 2006, 11:24 GMT

I applied and committed Nathan's version that used parts of
yours - you were both added to the CREDITS file due to this.
I hope that we can continue improving the simulator from the
now committed code. Thanks.

Loading...