FS#5839 - Use Menu API in Plugins

Attached to Project: Rockbox
Opened by Tom Ross (midgey34) - Friday, 18 August 2006, 07:57 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


I went through and adapted some of the plugins to use the standard Rockbox Menu API instead of their own implementation. The plugins changed were Dice, Disktidy, Minesweeper, Star, and Xobox. Some of the plugins also received slight cleanups and/or removed dead code and defines. One thing that could be improved is adding titles in these new menus since I couldn't figure out the correct way to implement one.
This task depends upon

Closed by  Tom Ross (midgey34)
Wednesday, 28 March 2007, 07:41 GMT
Reason for closing:  Accepted
Comment by Tom Ross (midgey34) - Wednesday, 25 October 2006, 04:59 GMT
Updated to CVS and broken into separate patch file
Comment by Chris (decayed.cell) - Tuesday, 02 January 2007, 07:18 GMT
dice.patch, disktidy.patch, minesweeper.patch, needs updating for 20070201 source. Others seem to work fine
Comment by Tom Ross (midgey34) - Friday, 05 January 2007, 19:09 GMT
I didn't know there was much of an interest in these patches. Here are updated versions, enjoy.
Comment by Chris (decayed.cell) - Monday, 15 January 2007, 23:36 GMT
They need updating for latest SVN again :D coz the Sansa builds have the games now
Comment by Tom Ross (midgey34) - Thursday, 18 January 2007, 01:44 GMT
I committed a modified version of the disktidy patch to SVN. Expect more to come soon.
Comment by Chris (decayed.cell) - Tuesday, 06 March 2007, 11:16 GMT
Well its now March, have you had time for any updating?
Comment by Tom Ross (midgey34) - Friday, 16 March 2007, 06:29 GMT
Here's the newest patches. These versions fix issues with leaking menus and get rid of unnecessary changes. I'll commit these assuming there are no issues that creep up.
(application/octet-stream)    dice.diff (11.4 KiB)
Comment by Tom Ross (midgey34) - Friday, 16 March 2007, 06:31 GMT
Err, hit the 'Add Comment' button a bit too early...
Comment by Tom Ross (midgey34) - Friday, 16 March 2007, 21:22 GMT
Updated snake to fix a warning.
Comment by Tom Ross (midgey34) - Friday, 23 March 2007, 19:50 GMT
This should be the most up to date version of the patch. It currently uses the old menu API but I haven't been successful on learning the new version with all the macros. Anyway, this also eliminates a nasty global in Star and provides a work-around so that Star won't crash on the Gigabeat. The code that was crashing works fine in the sim, so I'm guessing there's an issue with the LCD driver.
Comment by Jonathan Gordon (jdgordon) - Sunday, 25 March 2007, 14:33 GMT
After much arm twisting in IRC I have commited a converted xobox menu whic is hopefully enough example for you to do the rest "properly" :)
Comment by Tom Ross (midgey34) - Monday, 26 March 2007, 01:34 GMT
Here's an updated version that uses as much of the new API as possible. It also converts Disktidy to this new API. Let me know if I broke anything or did something incorrectly.