Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12056 - Xobox: Xonix starts moving on its own (without any keypress) at beginning of a level

Attached to Project: Rockbox
Opened by Ophir Lojkine (lovasoa) - Saturday, 09 April 2011, 16:44 GMT+2
Last edited by MichaelGiacomelli (saratoga) - Monday, 25 April 2011, 01:52 GMT+2
Task Type Patches
Category Games
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.8.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Steps to reproduce the bug:
1. Start playing xobox
2. When you are about to finish the first level, keep the down key pressed to cut a part of the infected area.
3. Keep the key pressed even when it tells you «first level finished».
4. On the second level, your key press will still be in the buffer, and xonix will start moving even if you are not pressing a key anymore.


My little patch just adds a call to rb->button_clear_queue(); at the beginning of every new level.
   patchXobox.patch (0.4 KiB) 
 ../apps/plugins/xobox.c |    1 +
 1 file changed, 1 insertion(+)

This task depends upon

Closed by  MichaelGiacomelli (saratoga)
Monday, 25 April 2011, 01:52 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  Accepted in r29781.
Comment by Ophir Lojkine (lovasoa) - Saturday, 09 April 2011, 17:05 GMT+2
The fix is trivial, and all credits go to the one who discovered the bug.
So I want to thank my little brother, Noah, a 7 year old xobox addict.
Comment by Hayden Pearce (St.) - Sunday, 10 April 2011, 10:56 GMT+2
Thankyou Lovasoa (and Noah) for finding this,

Months ago my girlfriend noticed this, told me about it, and I forgot about it almost immediately ;)

I have tested that this occurs on my available targets, Nano 1/2G, and 4G/Color.
Also confirmed this your little patch does indeed fix the issue.


[St.]
Comment by Thomas Martitz (kugel.) - Tuesday, 12 April 2011, 18:20 GMT+2
But it has tabs!
Comment by Ophir Lojkine (lovasoa) - Tuesday, 19 April 2011, 23:12 GMT+2
Can someone commit this patch to the svn?
That would make my little brother very happy...

Loading...