Ticket #7939: tv_jumppoints_v9.patch

mythtv/libs/libmythtv/tv_play.cpp

@@ -3530 +3530 @@
 
 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")
@@ -3542 +3558 @@
         idleTimerId = StartTimer(db_idle_timeout, __LINE__);
     }
 
-    QStringList actions;
-    bool handled = false;
+    handled = false;
 
     if (ignoreKeys)
     {
@@ -3551 +3566 @@
                   "TV Playback", e, actions);
 
         if (handled || actions.isEmpty())
-            return;
+            continue;
 
         bool esc   = has_action("ESCAPE", actions) ||
                      has_action("BACK", actions);
@@ -3559 +3574 @@
         bool play  = has_action("PLAY",   actions);
 
         if ((!esc || browsehelper->IsBrowsing()) && !pause && !play)
-            return;
+            continue;
     }
 
     OSD *osd = GetOSDLock(actx);
     if (osd && osd->DialogVisible())
     {
-        osd->DialogHandleKeypress(e);
+        if (isJumpingToMain)
+            osd->DialogQuit();
+        else
+            osd->DialogHandleKeypress(e);
         handled = true;
     }
     ReturnOSDLock(actx, osd);
@@ -3584 +3602 @@
             }
             if (has_action("ESCAPE", actions))
             {
+                if (isJumpingToMain)
+                    actx->player->DisableEdit(false);
+                else
                 if (!actx->player->IsCutListSaved(actx))
                     ShowOSDCutpoint(actx, "EXIT_EDIT_MODE");
                 else
@@ -3617 +3638 @@
     }
 
     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.
@@ -3629 +3650 @@
         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))
@@ -3643 +3665 @@
 
         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;
         }
     }
 
@@ -3663 +3689 @@
 
         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__);
@@ -3679 +3709 @@
               "TV Playback", e, actions);
 
     if (handled || actions.isEmpty())
-        return;
+        continue;
 
     handled = false;
 
@@ -3706 +3736 @@
 #endif // DEBUG_ACTIONS
 
     if (handled)
-        return;
+        continue;
 
     if (!handled)
     {
@@ -3723 +3753 @@
             }
         }
     }
+
+    } while (isJumpingToMain &&
+             exitPlayerTimerId == 0 &&
+             iterations <= kMaxIterations);
 }
 
 bool TV::BrowseHandleAction(PlayerContext *ctx, const QStringList &actions)

mythtv/programs/mythfrontend/playbackbox.cpp

@@ -2337 +2337 @@
 
     m_playingSomething = false;
 
-    if (inPlaylist && !m_playListPlay.empty())
+    if (inPlaylist && !m_playListPlay.empty() && !GetMythMainWindow()->IsExitingToMain())
     {
         QCoreApplication::postEvent(
             this, new MythEvent("PLAY_PLAYLIST"));