Rockbox

Tasklist

FS#9383 - chessclock freezes on deleting a player

Attached to Project: Rockbox
Opened by Frank Gevaerts (fg) - Monday, 08 September 2008, 21:10 GMT
Last edited by Teruaki Kawashima (teru) - Thursday, 09 July 2009, 13:42 GMT
Task Type Bugs
Category Applications
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

To reproduce :
start chessclock
choose two players
choose any time value you like
once everything is setup, get the menu
select Delete player
press any button
The player is now frozen

Reproduced on ipod video, ipod mini, H120, gigabeat F, X5, and Archos player
This task depends upon

Closed by  Teruaki Kawashima (teru)
Thursday, 09 July 2009, 13:42 GMT
Reason for closing:  Accepted
Additional comments about closing:  committed as r21733. thanks.
Comment by Yoshihisa Uchida (Uchida) - Thursday, 08 January 2009, 02:04 GMT
I confirmed this bug.

I send the patch file.
Please confirm it.
Comment by Yoshihisa Uchida (Uchida) - Tuesday, 17 March 2009, 09:07 GMT
sync r20338
Comment by Teruaki Kawashima (teru) - Wednesday, 03 June 2009, 13:16 GMT
I noticed one issue in Uchida's patch.
if a player is deleted, switching to previous player is not work properly.
e.g.
1. start with 3 players.
2. delete one of them. e.g. 2nd player.
3. now, 3rd player is displayed.
4. press button to show previous player. player is not changed.
i think 1st player should be displayed.
here is a patch that enables to show previous player.
Could someone confirm this?
Comment by Yoshihisa Uchida (Uchida) - Friday, 05 June 2009, 10:54 GMT
Hi, Kawashima, thank you for the report about another problem.
I confirmed this problem.
Because your patch file is baesed on  FS#9707 , I corrected it to this task's patch (chessclock_2.patch).
I corrected the patch of  FS#9707  too. Please look at  FS#9707 .





Comment by Yoshihisa Uchida (Uchida) - Wednesday, 17 June 2009, 09:20 GMT
sync r21316

Loading...