[vdr] Re: MP3/MPlayer plugin 0.9.4 (development)

[s.huelswitt@gmx.de (Stefan Huelswitt)]

On 04 Sep 2004 Sebastian Kemper <sebastian_ml@gmx.net> wrote:

> the green button (play from beginning) in the mplayer menu doesn't work 
> anymore. It shows up but it has no function. I can press the green 
> button until forever without anything happening at all. If I deselect 
> the resume funtion in mplayer options it dissapears. So I guess that's 
> working :)

Actualy the green button is linked to "Parent" functionality. The
sources button fix broke this.

Try this fix:

--- mplayer.c	2004-09-03 18:17:53.000000000 +0200
+++ mplayer.c	2004-09-05 10:42:06.000000000 +0200
@@ -525,33 +525,20 @@
 
 eOSState cMenuMPlayBrowse::ProcessKey(eKeys Key)
 {
-  eOSState state=cMenuBrowse::ProcessKey(Key);
-
-  if(state==osContinue && sourcing && !HasSubMenu()) // eval the return value from submenus
-    return Source(true);
-
+  eOSState state=osUnknown;
   bool rew=false;
-  if(state==osUnknown) {
-    switch(Key) {
-      case kGreen:
-        {
-        cFileObj *item=CurrentItem();
-        if(item && item->Type()==otFile) {
-          lastselect=new cFileObj(item);
-          state=osBack;
-          rew=true;
-          } 
-        else state=osContinue;
-        break;
-        }
-      case kYellow:
-        state=Source(false);
-        break;
-      default:
-        break;
-      }
+  if(Key==kGreen) {
+    cFileObj *item=CurrentItem();
+    if(item && item->Type()==otFile) {
+      lastselect=new cFileObj(item);
+      state=osBack;
+      rew=true;
+      } 
+    else state=osContinue;
     }
-
+  if(state==osUnknown) state=cMenuBrowse::ProcessKey(Key);
+  if(state==osContinue && sourcing && !HasSubMenu()) return Source(true);
+  if(state==osUnknown && Key==kYellow) state=Source(false);
   if(state==osBack && lastselect) {
     cMPlayerControl::SetFile(lastselect,rew);
     cControl::Launch(new cMPlayerControl);


Regards.

-- 
Stefan Huelswitt
s.huelswitt@gmx.de  | http://www.muempf.de/