• 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 learman - 2003-03-23
Last edited by hohensoh - 2003-05-21

FS#1136 - UI simulator updates for Win32

These patches contains several updates to the Win32
version of the UI simulator. It now works quite well
for me, when building with Cygwin.

First out is lcd-sim.diff, which adds the bare
neccessary for it to compile and work - sort of. It
adds the required functionality to lcd-win32.c.
However, at least the player version doesn't work too
well then, as the "zoom" of the LCD display is quite
different then. See the uisw32.diff below for details.
Also, the button handling isn't impressive… :)

Second patch is button.diff, which revamps the keyboard
handling. It is largely based on the "real" button.c,
with two major changes: 1) reboot handling is removed
and 2) rather than reading the hardware, the button
input queue is fed through events from the Windows
message pump.

I've tested it briefly on player and recorder
simulators and seems to work quite well. At least I can
enter menus now. :)

Third patch is uisw32.diff, which removes the UI256.bmp
from the main window, leaving only the actual LCD
display. The new player simulation code requires some
changes to make it work decently, so I chose this
solution, as it makes the Win32 simulator more like the
X11 one, AFAIK.

The location of the volume icon is tweaked a little, so
as to be fully visible. I guess there are more things
to tweak (e.g., so the window title shows e.g. 120%
zoom rather than 119%), but now it works rather good.

I split up the patches into separate files manually, so
I might have made an error somewhere. Hopefully you'll
be able to pick your poision this way. :)

Closed by  hohensoh
2003-05-21 09:38
Reason for closing:  Accepted

(Now why can't I attach a file with the initial patch submit?)

Checked in, except for the bitmap removal.
As Markus suggests, a mutex or critical section would be
appropriate to protect the button queue from different thread
ends. Not done by now, maybe better to generally do this for
Win32 simulated queues on kernel level.


Available keyboard shortcuts


Task Details

Task Editing