Ticket #7939: tv_jumppoints_v4.patch

libs/libmythtv/tv_play.cpp

@@ -3469 +3469 @@
 
 void TV::ProcessKeypress(PlayerContext *actx, QKeyEvent *e)
 {
+    QStringList actions;
+    bool handled = GetMythMainWindow()->
+        TranslateKeyPress("TV Playback", e, actions, false);
+    if (handled)
+        VERBOSE(VB_IMPORTANT,
+                "ProcessKeypress(): "
+                "TranslateKeyPress() unexpectedly returned true");
+    bool isJumpingToMain = has_action("ESCAPE", actions) &&
+        GetMythMainWindow()->IsExitingToMain();
+    unsigned iterations = 0;
+    // Set a limit on the number of iterations in case pressing escape
+    // repeatedly somehow gets into a loop.
+    const unsigned kMaxIterations = 10;
+    do {
+        ++iterations;
+
     bool ignoreKeys = actx->IsPlayerChangingBuffers();
 #if DEBUG_ACTIONS
     VERBOSE(VB_IMPORTANT, LOC + QString("ProcessKeypress() ignoreKeys: %1")
@@ -3481 +3497 @@
         idleTimerId = StartTimer(db_idle_timeout, __LINE__);
     }
 
-    QStringList actions;
-    bool handled = false;
+    handled = false;
 
     if (ignoreKeys)
     {
@@ -3490 +3505 @@
                   "TV Playback", e, actions);
 
         if (handled || actions.isEmpty())
-            return;
+            continue;
 
         bool esc   = has_action("ESCAPE", actions);
         bool pause = has_action("PAUSE",  actions);
         bool play  = has_action("PLAY",   actions);
 
         if ((!esc || browsemode) && !pause && !play)
-            return;
+            continue;
     }
 
     OSD *osd = GetOSDLock(actx);
     if (osd && osd->DialogVisible())
     {
-        osd->DialogHandleKeypress(e);
+        if (GetMythMainWindow()->IsExitingToMain())
+            osd->DialogQuit();
+        else
+            osd->DialogHandleKeypress(e);
         handled = true;
     }
 
@@ -3530 +3548 @@
     ReturnOSDLock(actx, osd);
 
     if (handled)
-        return;
+        continue;
 
     // If text is already queued up, be more lax on what is ok.
     // This allows hex teletext entry and minor channel entry.
@@ -3542 +3560 @@
         if (ok || txt=="_" || txt=="-" || txt=="#" || txt==".")
         {
             AddKeyToInputQueue(actx, txt.at(0).toLatin1());
-            return;
+            continue;
         }
     }
 
+    bool should_continue;
     // Teletext menu
     actx->LockDeletePlayer(__FILE__, __LINE__);
     if (actx->player && (actx->player->GetCaptionMode() == kDisplayTeletextMenu))
@@ -3556 +3575 @@
 
         if (!handled && !tt_actions.isEmpty())
         {
+            should_continue = false;
             for (int i = 0; i < tt_actions.size(); i++)
             {
                 if (actx->player->HandleTeletextAction(tt_actions[i]))
                 {
                     actx->UnlockDeletePlayer(__FILE__, __LINE__);
-                    return;
+                    should_continue = true;
+                    break;
                 }
             }
+            if (should_continue)
+                continue;
         }
     }
 
@@ -3576 +3599 @@
 
         if (!handled && !itv_actions.isEmpty())
         {
+            should_continue = false;
             for (int i = 0; i < itv_actions.size(); i++)
             {
                 if (actx->player->ITVHandleAction(itv_actions[i]))
                 {
                     actx->UnlockDeletePlayer(__FILE__, __LINE__);
-                    return;
+                    should_continue = true;
+                    break;
                 }
             }
+            if (should_continue)
+                continue;
         }
     }
     actx->UnlockDeletePlayer(__FILE__, __LINE__);
@@ -3592 +3619 @@
               "TV Playback", e, actions);
 
     if (handled || actions.isEmpty())
-        return;
+        continue;
 
     handled = false;
 
@@ -3618 +3645 @@
 #endif // DEBUG_ACTIONS
 
     if (handled)
-        return;
+        continue;
 
     if (!handled)
     {
@@ -3635 +3662 @@
             }
         }
     }
+
+    } while (isJumpingToMain &&
+             exitPlayerTimerId == 0 &&
+             iterations <= kMaxIterations);
 }
 
 bool TV::BrowseHandleAction(PlayerContext *ctx, const QStringList &actions)