Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category Games
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Version 3.0
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by drspoon - 2006-04-09
Last edited by linuxstb - 2006-05-20

FS#5077 - wormlet runs out of food

When playing the wormlet game sometimes eating food does not force creation of new food, thus food runs out. The game also starts with varying amounts of food. The non-creation of food might be something to do with eating food when already growing, though I couldn’t find a way of reproducing the bug every time.

Closed by  midkay
2006-05-22 07:00
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Just committed this. Thanks for the report+patch. :)

drspoon commented on 2006-04-09 20:20

happens on h300s on target and sim, not on ipod-video

drspoon commented on 2006-04-09 22:11

it's a strange bug - but i've noticed 2 quirks in the code.

foodx, foody, arghx and arghy are chars - and in init_wormlet the food and arghs are not always created

make_food and make_argh return ints, but aren't used in assignment anywhere.

I've included a so far as I can see working version but the debug code is a quick hack and still in. I'll clean it up for tomorrow

   wormlet.c (62.2 KiB)
drspoon commented on 2006-04-10 08:40

A guess, but it might be something to do with a char's bit length. On the ipod-video there are no problems, but in the ipod-video sim there are problems. My fixed version works in the sims, untested on target.

Project Manager
bagder commented on 2006-04-10 08:43

chars are the same size, but are you possible assuming them to be unsigned without specificly saying so?

drspoon commented on 2006-04-10 09:04

Wormlet isnt my code, I know nothing about chars. My int change works on the actual h300 and in the h300 and ipod-video sims. Here's the cleaned version.

drspoon commented on 2006-04-10 09:04

ok here's the file…

   wormlet.c (61.8 KiB)
drspoon commented on 2006-04-13 20:00

http://www.rockbox.org/tracker/task/1510 also contains very old wormlet changes. I've tried to bring it up to date and include it in my changes. Some of the code is strange though i'd appreciate a 2nd opinion.

   wormlet.c (71.8 KiB)
(application/octet-stream)    wormletchanges.patch (38 KiB)
midkay commented on 2006-04-22 09:46

You're right, the chars should be a problem for the larger LCDs. I had a look at your most recent patch and I'm glad to see MT's update with the adjustables menu (haven't used it in years!). :) After I finish a few touch-ups with JdGordon's viewer update and commit that, I'll test and check this out more thoroughly and probably commit it (definitely at least the fixes).

Thanks. :)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing