Playlists?

Message ID 42E80679.9030008@syphir.sytes.net
State New
Headers

Commit Message

C.Y.M July 27, 2005, 10:11 p.m. UTC
  Hardy Flor wrote:
> Hallo Aron,
> 
> 
>>is there any way to define playlists for the video recordings?
> 
> 
> with the playlist-plugin
> http://www.fast-info.de/vdr/playlist/index.htm
> 

If you want to use this plugin with vdr-1.3.27, you will need this too.. (BTW,
the #ifdefs are not exactly right but it works). :)

Regards,
C.
  

Comments

C.Y.M July 28, 2005, 12:29 a.m. UTC | #1
C.Y.M wrote:
> Hardy Flor wrote:
> 
>>Hallo Aron,
>>
>>
>>
>>>is there any way to define playlists for the video recordings?
>>
>>
>>with the playlist-plugin
>>http://www.fast-info.de/vdr/playlist/index.htm
>>
> 
> 
> If you want to use this plugin with vdr-1.3.27, you will need this too.. (BTW,
> the #ifdefs are not exactly right but it works). :)
> 

Seems to be a bug in this playlist plugin where if I try to change the playlist
options more than one time, it will crash VDR.  The first time I try to change
the options after I create a playlist it seems fine, but the second time I hit
the yellow button to change the options, it goes boom.

Regards,
C.
  
Johannes Schoeller July 28, 2005, 8:47 a.m. UTC | #2
C.Y.M wrote:
> Hardy Flor wrote:
> 
>>Hallo Aron,
>>
>>
>>
>>>is there any way to define playlists for the video recordings?
>>
>>
>>with the playlist-plugin
>>http://www.fast-info.de/vdr/playlist/index.htm
>>
> 
> 
> If you want to use this plugin with vdr-1.3.27, you will need this too.. (BTW,
> the #ifdefs are not exactly right but it works). :)
> 
> Regards,
> C.

hi c.

thanks for the patch. now i can use the plugin again!

regards j.schoeller
  

Patch

--- playlist/dataplaylist.c.orig	2005-07-27 09:45:35.000000000 -0700
+++ playlist/dataplaylist.c	2005-07-27 09:47:29.000000000 -0700
@@ -145,7 +145,7 @@ 
     char *temp = strrchr(title, '\t');
     if (temp)
       *temp = 0;
-    summary = recording->Summary() ? strdup(recording->Summary()) : NULL;
+    summary = recording->Info()->Description() ? strdup(recording->Info()->Description()) : NULL;
     return true;
   }
   isdel = true;
--- playlist/menuplaylist.c.orig	2005-07-27 09:50:50.000000000 -0700
+++ playlist/menuplaylist.c	2005-07-27 09:55:06.000000000 -0700
@@ -154,7 +154,7 @@ 
     else
     {
       cRecording *recording = GetRecording(ri);
-      SetHelp(singleselect ? tr("Select") : tr("Add"), singleselect ? NULL : ri->IsMark() ? tr("UnMark") : tr("Mark"), deleterecords.u ? tr("Delete") : NULL, (recording && recording->Summary() && *recording->Summary()) ? tr("Summary") : NULL);
+      SetHelp(singleselect ? tr("Select") : tr("Add"), singleselect ? NULL : ri->IsMark() ? tr("UnMark") : tr("Mark"), deleterecords.u ? tr("Delete") : NULL, (recording && recording->Info()->Description() && *recording->Info()->Description()) ? tr("Info") : NULL);
     }
   } else
   {
@@ -256,9 +256,9 @@ 
   if (ri && !ri->IsDirectory())
   {
     cRecording *recording = GetRecording(ri);
-    if (recording && recording->Summary() && *recording->Summary())
+    if (recording && recording->Info()->Description() && *recording->Info()->Description())
 #if VDRVERSNUM >= 10307
-      return AddSubMenu(new cMenuText(tr("Summary"), recording->Summary()));
+      return AddSubMenu(new cMenuText(tr("Info"), recording->Info()->Description()));
 #else
       return AddSubMenu(new cMenuItemText(tr("Summary"), recording->Summary()));
 #endif
@@ -514,8 +514,8 @@ 
   }
 #endif
 
-  char *buffer = NULL;
-  eOptions display = (Current() >= 0) && (Current() < MAXOptionLines) ? displayoption[Current()] : Option_max;
+//  char *buffer = NULL;
+//  eOptions display = (Current() >= 0) && (Current() < MAXOptionLines) ? displayoption[Current()] : Option_max;
 /*  if (display < Option_max && data.OptionInherited(display))
     asprintf(&buffer, "%s %s", tr("OptionPL$Effective setting is:"), NoYesDefault[data.Option(display)]);
   if (buffer)