From 1722e6dcdc845d3204c1077f6990acaae436e107 Mon Sep 17 00:00:00 2001
From: kugel <kugel@a1c6a512-1295-4272-9138-f99709370657>
Date: Mon, 24 Aug 2009 01:11:24 +0000
Subject: [PATCH] Bubbles: Have one quit item that doesn't save anything ("Quit
 without Saving") and one that saves everything again, partly reverting r22143
 and picking up Hilton Shumway's suggestion about naming it "Quit without
 Saving".

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@22488 a1c6a512-1295-4272-9138-f99709370657
---
 apps/plugins/bubbles.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/apps/plugins/bubbles.c b/apps/plugins/bubbles.c
index 697f69045..344b73296 100644
--- a/apps/plugins/bubbles.c
+++ b/apps/plugins/bubbles.c
@@ -41,8 +41,8 @@ PLUGIN_HEADER
 /* final game return status */
 enum {
     BB_LOSE,
+    BB_QUIT_WITHOUT_SAVING,
     BB_QUIT,
-    BB_SAVE_AND_QUIT,
     BB_USB,
     BB_END,
     BB_WIN,
@@ -2369,7 +2369,7 @@ static int bubbles_menu(struct game_context* bb) {
     MENUITEM_STRINGLIST(menu,"Bubbles Menu",NULL,
                         "Resume Game", "Start New Game",
                         "Level", "High Scores", "Playback Control",
-                        "Quit", "Save and Quit");
+                        "Quit without Saving", "Quit");
 
     while(!startgame){
         switch (rb->do_menu(&menu, &selected, NULL, false))
@@ -2400,10 +2400,10 @@ static int bubbles_menu(struct game_context* bb) {
             case 4: /* Playback Control */
                 playback_control(NULL);
                 break;
-            case 5: /* quit */
-                return BB_QUIT;
+            case 5: /* quit but don't save */
+                return BB_QUIT_WITHOUT_SAVING;
             case 6: /* save and quit  */
-                return BB_SAVE_AND_QUIT;
+                return BB_QUIT;
             case MENU_ATTACHED_USB:
                 bubbles_callback(bb);
                 return BB_USB;
@@ -2518,14 +2518,17 @@ enum plugin_status plugin_start(const void* parameter) {
                 exit = true;
                 break;
 
-            case BB_SAVE_AND_QUIT:
-                rb->splash(HZ/2, "Saving Game ...");
-                bubbles_savegame(&bb);
-                /* fall through */
-
             case BB_QUIT:
+                /* the first splash is to make sure it's read, but don't make it
+                 * too long to not delay the saving further */
+                rb->splash(HZ/5, "Saving Game ...");
+                rb->splash(0, "Saving Game ...");
+                bubbles_savegame(&bb);
                 bubbles_savedata();
                 highscore_save(SCORE_FILE, highscores, NUM_SCORES);
+                /* fall through */
+
+            case BB_QUIT_WITHOUT_SAVING:
                 exit = true;
                 break;
 
-- 
2.11.4.GIT