From 402b8ebeae49b498d725f85262a02e718c8c2f11 Mon Sep 17 00:00:00 2001
From: Teruaki Kawashima <teru@rockbox.org>
Date: Thu, 9 Jul 2009 13:36:22 +0000
Subject: fix FS#9383 (chessclock freezes on deleting a player), patch by
 Yoshihisa Uchida.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@21733 a1c6a512-1295-4272-9138-f99709370657
---
 apps/plugins/chessclock.c | 29 ++++++++++++++++++-----------
 1 file changed, 18 insertions(+), 11 deletions(-)

diff --git a/apps/plugins/chessclock.c b/apps/plugins/chessclock.c
index a69b3bb582..514cac174e 100644
--- a/apps/plugins/chessclock.c
+++ b/apps/plugins/chessclock.c
@@ -367,17 +367,22 @@ enum plugin_status plugin_start(const void* parameter)
     do {
         int ret=0;
         done=true;
-        for (i=0; done && i<settings.nr_timers; i++) {
+        i = nr;
+        do {
             if (!timer_holder[i].hidden)
-                done=false;
-        }
+            {
+                nr = i;
+                done = false;
+                break;
+            }
+            if (++i == settings.nr_timers)
+                i = 0;
+        } while (i != nr);
+
         if (done) {
             return PLUGIN_OK;
         }
-        if (!timer_holder[nr].hidden) {
-            done=false;
-            ret=run_timer(nr);
-        }
+        ret = run_timer(nr);
         switch (ret) {
             case -1: /* exit */
                 done=true;
@@ -390,9 +395,11 @@ enum plugin_status plugin_start(const void* parameter)
                     nr=0;
                 break;
             case 2:
-                nr--;
-                if (nr<0)
-                    nr=settings.nr_timers-1;
+                do {
+                    nr--;
+                    if (nr<0)
+                        nr=settings.nr_timers-1;
+                } while (timer_holder[nr].hidden);
                 break;
         }
     } while (!done);
@@ -521,7 +528,7 @@ static int run_timer(int nr)
                 switch(rb->do_menu(&menu, NULL, NULL, false))
                 {
                     case 0:
-                         /* delete timer */
+                         /* delete player */
                         timer_holder[nr].hidden=true;
                         retval=1;
                         done=true;
-- 
cgit v1.2.3