Fix "bookmark on stop: ask" when idle poweroff triggers.

Previously, the prompt would come up during poweroff, and then if the user did not respond, the shutdown timeout would be hit and a hard poweroff would happen, which is bad. Now it just assumes you don't want a bookmark.


git-svn-id: svn://svn.rockbox.org/rockbox/trunk@25376 a1c6a512-1295-4272-9138-f99709370657

4 files changed, 7 insertions, 7 deletions

diff --git a/apps/bookmark.c b/apps/bookmark.c
index 85570839be..dd800f75c0 100644
--- a/apps/bookmark.c
+++ b/apps/bookmark.c
@@ -159,7 +159,7 @@ bool bookmark_mrb_load()
 /* This function handles an autobookmark creation.  This is an interface   */
 /* function.                                                               */
 /* ----------------------------------------------------------------------- */
-bool bookmark_autobookmark(void)
+bool bookmark_autobookmark(bool prompt_ok)
 {
     char*  bookmark;
     if (!system_check())
@@ -192,7 +192,7 @@ bool bookmark_autobookmark(void)
     const struct text_message message={lines, 2};
 #endif
 
-    if(gui_syncyesno_run(&message, NULL, NULL)==YESNO_YES)
+    if(prompt_ok && gui_syncyesno_run(&message, NULL, NULL)==YESNO_YES)
     {
         if (global_settings.autocreatebookmark == BOOKMARK_RECENT_ONLY_ASK)
             return write_bookmark(false, bookmark);
diff --git a/apps/bookmark.h b/apps/bookmark.h
index e8770f5c4e..bde8e84697 100644
--- a/apps/bookmark.h
+++ b/apps/bookmark.h
@@ -24,7 +24,7 @@
 #include <stdbool.h>
 
 bool bookmark_load_menu(void);
-bool bookmark_autobookmark(void);
+bool bookmark_autobookmark(bool prompt_ok);
 bool bookmark_create_menu(void);
 bool bookmark_mrb_load(void);
 bool bookmark_autoload(const char* file);
diff --git a/apps/gui/wps.c b/apps/gui/wps.c
index e2a7dcb961..b6a42769fe 100644
--- a/apps/gui/wps.c
+++ b/apps/gui/wps.c
@@ -1258,7 +1258,7 @@ long gui_wps_show(void)
                 fade(false, true);
 
             if (bookmark)
-                bookmark_autobookmark();
+                bookmark_autobookmark(true);
             audio_stop();
 #ifdef AB_REPEAT_ENABLE
             ab_reset_markers();
diff --git a/apps/misc.c b/apps/misc.c
index 554d29e630..f9c6116205 100644
--- a/apps/misc.c
+++ b/apps/misc.c
@@ -291,7 +291,7 @@ static bool clean_shutdown(void (*callback)(void *), void *parameter)
 #ifdef SIMULATOR
     (void)callback;
     (void)parameter;
-    bookmark_autobookmark();
+    bookmark_autobookmark(false);
     call_storage_idle_notifys(true);
     exit(0);
 #else
@@ -356,7 +356,7 @@ static bool clean_shutdown(void (*callback)(void *), void *parameter)
                     sleep(1);
             }
 #endif
-            bookmark_autobookmark();
+            bookmark_autobookmark(false);
 
             /* audio_stop_recording == audio_stop for HWCODEC */
             audio_stop();
@@ -420,7 +420,7 @@ bool list_stop_handler(void)
         {
             if (global_settings.fade_on_stop)
                 fade(false, false);
-            bookmark_autobookmark();
+            bookmark_autobookmark(true);
             audio_stop();
             ret = true;  /* bookmarking can make a refresh necessary */
         }