From: Greg Haerr <greg_at_censoft.com>
Date: Wed, 12 Jun 2002 02:24:59 -0700

I just pulled down the latest 6-11 tarball, and tried
making the win32 version. After fixing "intptr" -> "int"
in firmware/common/dir.h, and adding uisimulator/win32/uisw32.rc,
the make failed with undefined routines yield() and
create_thread() in firmware/drivers/lcd.c.

This is because the current win32 implementation uses
_beginthread rather than the rockbox create_thread to
start threads, and thread support is now required in
the lcd firmware driver. I can work on this to fix it all
up, but I suggest the low level driver code remain
just that - low level. That would mean that the scrolling
code, etc would move up a level, and be shared for
the uisimulator version. Then, the win32 version could
be rewritten to just output dots at a low level, and all
the other code would be shared...

This would also remove a number of low-level
driver #if SIMULATOR defines. Perhaps a
hierarchical API chart, showing top-level,
app-level, firmware level, and driver level routine
names is in order. This would also help with the
API documentation.

Great project!

I better go to bed now...

Regards,

Greg
Received on 2002-06-12