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:  

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

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

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)

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

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

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.

ok here's the file…

   wormlet.c (61.8 KiB)

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.

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