[gcompris/GCOMPRIS_8_3] When shuffling the items for the left bar, the last item was skipped. As the



commit 83c5c90c69bdf53973f098592116512d9aefe510
Author: Wolfram Sang <wolfram iswi org>
Date:   Fri Jun 19 00:15:32 2009 +0200

    When shuffling the items for the left bar, the last item was skipped. As the
    upper limit for random-functions is exclusive, we have to pass the whole
    list_size, not list_size - 1. Also, simplify the code.

 src/boards/shapegame.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
---
diff --git a/src/boards/shapegame.c b/src/boards/shapegame.c
index 41f1b5c..bb492b9 100644
--- a/src/boards/shapegame.c
+++ b/src/boards/shapegame.c
@@ -1702,7 +1702,7 @@ parse_doc(xmlDocPtr doc)
     {
       Shape *shape;
 
-      i = list_length == 1 ? 0 : g_random_int_range(0, g_list_length(shape_list_init)-1);
+      i = g_random_int_range(0, list_length);
       shape = g_list_nth_data(shape_list_init, i);
       add_shape_to_canvas(shape);
 



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]