Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by midgey34 - 2006-08-18

FS#5839 - Use Menu API in Plugins

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.

Closed by  midgey34
2007-03-28 07:41
Reason for closing:  Accepted

Updated to CVS and broken into separate patch file

dice.patch, disktidy.patch, minesweeper.patch, needs updating for 20070201 source. Others seem to work fine

I didn’t know there was much of an interest in these patches. Here are updated versions, enjoy.

They need updating for latest SVN again :D coz the Sansa builds have the games now

I committed a modified version of the disktidy patch to SVN. Expect more to come soon.

Well its now March, have you had time for any updating?

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)

Err, hit the ‘Add Comment’ button a bit too early…

Updated snake to fix a warning.

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.

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” :)

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.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing