skindesigner plugin lässt sich mit vdr-2.5.4 nicht kompilieren

Jarod · 6. Juni 2021

Mit VDR größer Version 2.5.2 bekomme ich beim Kompilieren des skindesigner Plugins folgende Fehlermeldung:

Code

coreengine/viewdetail.c: In member function 'virtual bool cViewDetailRec::Parse(bool)':
coreengine/viewdetail.c:679:31: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
         errors = info->Errors();
                               ^
In file included from /usr/local/src/vdr-2.5.4/include/vdr/skins.h:18:0,
                 from /usr/local/src/vdr-2.5.4/include/vdr/osdbase.h:15,
                 from coreengine/../services/epgsearch.h:31,
                 from coreengine/viewdetail.h:4,
                 from coreengine/viewdetail.c:1:
/usr/local/src/vdr-2.5.4/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   int Errors(void) { return errors; }
       ^~~~~~
coreengine/viewelementsdisplaymenu.c: In member function 'virtual bool cVeDmDetailheaderRec::Parse(bool)':
coreengine/viewelementsdisplaymenu.c:1234:31: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
         errors = info->Errors();
                               ^
In file included from /usr/local/src/vdr-2.5.4/include/vdr/skins.h:18:0,
                 from /usr/local/src/vdr-2.5.4/include/vdr/osdbase.h:15,
                 from /usr/local/src/vdr-2.5.4/include/vdr/menuitems.h:15,
                 from /usr/local/src/vdr-2.5.4/include/vdr/plugin.h:14,
                 from coreengine/globals.h:14,
                 from coreengine/viewelement.h:11,
                 from coreengine/viewelementsdisplaymenu.h:4,
                 from coreengine/viewelementsdisplaymenu.c:1:
/usr/local/src/vdr-2.5.4/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   int Errors(void) { return errors; }
       ^~~~~~
coreengine/listelements.c: In member function 'virtual bool cLeMenuRecordings::Parse(bool)':
coreengine/listelements.c:2062:27: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
     errors = info->Errors();
                           ^
In file included from /usr/local/src/vdr-2.5.4/include/vdr/skins.h:18:0,
                 from /usr/local/src/vdr-2.5.4/include/vdr/osdbase.h:15,
                 from /usr/local/src/vdr-2.5.4/include/vdr/menuitems.h:15,
                 from /usr/local/src/vdr-2.5.4/include/vdr/plugin.h:14,
                 from coreengine/globals.h:14,
                 from coreengine/viewelement.h:11,
                 from coreengine/listelements.h:4,
                 from coreengine/listelements.c:1:
/usr/local/src/vdr-2.5.4/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   int Errors(void) { return errors; }

Alles anzeigen

Es sind noch weitere Stellen, aber immer der gleiche Fehler.

Mit vdr-2.4.6 kommt noch kein Fehler.

Vermutlich liegt es an der Änderung hier in der recording.h vom vdr in Version 2.5.2:

Diff

@@ -150,7 +153,7 @@ public:
        ///< Returns the full path name to the recording directory, including the
        ///< video directory and the actual '*.rec'. For disk file access use.
   const char *Title(char Delimiter = ' ', bool NewIndicator = false, int Level = -1) const;
-  const cRecordingInfo *Info(void) const { return info; }
+  cRecordingInfo *Info(void) const { return info; }
   const char *PrefixFileName(char Prefix);
   int HierarchyLevels(void) const;
   void ResetResume(void) const;:q

Ich hoffe, dass der Patch hier das Ganze korrigiert und auch ein paar falsche "if defined" entfernt:

Diff

diff --git a/coreengine/listelements.c b/coreengine/listelements.c
index 10d4837..fcfde42 100644
--- a/coreengine/listelements.c
+++ b/coreengine/listelements.c
@@ -2017,7 +2017,11 @@ bool cLeMenuRecordings::Parse(bool forced) {
     ADD_TOKEN_MR1(TOKEN_LMR_IT);
 
     const cEvent *event = NULL;
-    const cRecordingInfo *info = usedRecording->Info();
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
+    const cRecordingInfo *info = recording->Info();
+#endif
 
     if (!info) {
         delete[] recName;
@@ -2058,9 +2062,7 @@ bool cLeMenuRecordings::Parse(bool forced) {
     }
 
     int errors = -1;
-#if defined (APIVERSNUM) && (APIVERSNUM >= 20504)
     errors = info->Errors();
-#endif
 
     // do the same stuff as in cCeMenuRecordings::Parse() (part 2)
     ADD_TOKEN_MR2(TOKEN_LMR_IT, TOKEN_LMR_ST);
@@ -2244,7 +2246,11 @@ bool cCeMenuRecordings::Parse(bool forced) {
     ADD_TOKEN_MR1(TOKEN_CMR_IT);
 
     const cEvent *event = NULL;
-    const cRecordingInfo *info = usedRecording->Info();
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
+    const cRecordingInfo *info = recording->Info();
+#endif
 
     if (!info) return true;
     event = info->GetEvent();
@@ -2278,9 +2284,7 @@ bool cCeMenuRecordings::Parse(bool forced) {
     }
 
     int errors = -1;
-#if defined (APIVERSNUM) && (APIVERSNUM >= 20504)
     errors = info->Errors();
-#endif
 
     // do the same stuff as in cLeMenuRecordings::Parse() (part 2)
     ADD_TOKEN_MR2(TOKEN_CMR_IT, TOKEN_CMR_ST);
diff --git a/coreengine/viewdetail.c b/coreengine/viewdetail.c
index 56910d3..4786856 100644
--- a/coreengine/viewdetail.c
+++ b/coreengine/viewdetail.c
@@ -668,16 +668,18 @@ bool cViewDetailRec::Parse(bool forced) {
     tokenContainer->AddStringToken((int)eDmDetailedRecST::name, recording->Name());
     tokenContainer->AddIntToken((int)eDmDetailedRecIT::cutted, recording->IsEdited());
 
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
     const cRecordingInfo *info = recording->Info();
+#endif
     if (info) {
         tokenContainer->AddStringToken((int)eDmDetailedRecST::epgname, info->Title());
         tokenContainer->AddStringToken((int)eDmDetailedRecST::shorttext, info->ShortText());
         tokenContainer->AddStringToken((int)eDmDetailedRecST::description, info->Description());
         tokenContainer->AddIntToken((int)eDmDetailedRecIT::framesPerSecond, info->FramesPerSecond());
-    int errors = -1;
-#if defined (APIVERSNUM) && (APIVERSNUM >= 20504)
+        int errors = -1;
         errors = info->Errors();
-#endif
         tokenContainer->AddIntToken((int)eDmDetailedRecIT::errors, errors);
         const cEvent *event = info->GetEvent();
         if (event) {
diff --git a/coreengine/viewelementsdisplaymenu.c b/coreengine/viewelementsdisplaymenu.c
index 24240a7..d7c0c11 100644
--- a/coreengine/viewelementsdisplaymenu.c
+++ b/coreengine/viewelementsdisplaymenu.c
@@ -557,7 +557,11 @@ void cVeDmCurrentschedule::ParseFromRecording(const cRecording *recording) {
     string recFolder = "";
     RecName(recFullName, recName, recFolder);
     tokenContainer->AddStringToken((int)eDMCurrentscheduleST::title, recName.c_str());
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
     const cRecordingInfo *info = recording->Info();
+#endif
     if (info)
         tokenContainer->AddStringToken((int)eDMCurrentscheduleST::subtitle, info->ShortText());
     tokenContainer->AddIntToken((int)eDMCurrentscheduleIT::duration, recording->LengthInSeconds() / 60);
@@ -988,7 +992,12 @@ bool cVeDmLastrecordings::Parse(bool forced) {
 #endif
         string recName = "";
         string recSeriesName = "";
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+        cRecordingInfo *recInfo = recording->Info();
+#else
         const cRecordingInfo *recInfo = recording->Info();
+#endif
+
         if (recInfo) {
             recName = recInfo->Title() ? recInfo->Title() : "";
             if (recInfo->Title() && recInfo->ShortText()) {
@@ -1224,15 +1233,17 @@ bool cVeDmDetailheaderRec::Parse(bool forced) {
     tokenContainer->Clear();
     tokenContainer->AddStringToken((int)eDmDetailedHeaderRecST::name, recording->Name());
 
+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
     const cRecordingInfo *info = recording->Info();
+#endif
     if (info) {
         tokenContainer->AddStringToken((int)eDmDetailedHeaderRecST::epgname, info->Title());
         tokenContainer->AddStringToken((int)eDmDetailedHeaderRecST::shorttext, info->ShortText());
         tokenContainer->AddIntToken((int)eDmDetailedHeaderRecIT::framesPerSecond, info->FramesPerSecond());
         int errors = -1;
-#if defined (APIVERSNUM) && (APIVERSNUM >= 20504)
         errors = info->Errors();
-#endif
         tokenContainer->AddIntToken((int)eDmDetailedHeaderRecIT::errors, errors);
         const cEvent *event = info->GetEvent();
         if (event) {

Alles anzeigen

wirbel · 6. Juni 2021

Warum nicht anstelle von

Code

+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
+    const cRecordingInfo *info = recording->Info();
+#endif

ohne #if (..) #endif

Code

const cRecordingInfo *info = dynamic_cast<const cRecordingInfo*>(recording->Info());

Jarod · 6. Juni 2021

Zitat von wirbel

Warum nicht anstelle von

Code

+#if defined (APIVERSNUM) && (APIVERSNUM >= 20502)
+    cRecordingInfo *info = recording->Info();
+#else
+    const cRecordingInfo *info = recording->Info();
+#endif

ohne #if (..) #endif

Code

const cRecordingInfo *info = dynamic_cast<const cRecordingInfo*>(recording->Info());

...weil meine C++ Kenntnisse eher spärlich gesät sind und ich genau das wusste/konnte. Dein Vorschlag kompiliert und scheint zu funktionieren. Danke, hab wieder was gelernt.
In meiner Version sieht man allerdings aufgrund welcher VDR Version hier etwas angepasst werden musste.

Wie es am Ende in den Code kommt und für alle nutzbar wird, ist mir ziemlich Schnuppe....

wirbel · 6. Juni 2021

Zitat

weil meine C++ Kenntnisse eher spärlich gesät sind

Das ist bei fast allen von uns so.

kamel5 · 6. Juni 2021

Zitat von Jarod

Es sind noch weitere Stellen, aber immer der gleiche Fehler.

Das ist eine Funktion, die mit VDR-2.5.4 neu eingeführt wurde.

Bis VDR-2.5.3 sollte es keine Probleme beim Übersetzen geben.

Die Lösung für dieses Problem gibt es hier:

[skindesigner] Aktuelle Version

Grüße

kamel5

Jarod · 7. Juni 2021

Zitat von kamel5

Das ist eine Funktion, die mit VDR-2.5.4 neu eingeführt wurde.

Bis VDR-2.5.3 sollte es keine Probleme beim Übersetzen geben.

Die Lösung für dieses Problem gibt es hier:

[skindesigner] Aktuelle Version

Ich hab mir das diff aus obigem Thread angesehen und verstehe nicht, warum ihr das so gelöst habt.
Wenn ich das richtig sehe, dann wird "cRecordingInfo" nur nicht mehr als "const" deklariert seit Version 2.5.2.

siehe "git diff 2.5.1 2.5.2 recording.h":

Diff

diff --git a/recording.h b/recording.h
index 6f26f2f6..217adea9 100644
--- a/recording.h
+++ b/recording.h
@@ -4,7 +4,7 @@
  * See the main source file 'vdr.c' for copyright information and
  * how to reach the author.
  *
- * $Id: recording.h 5.1 2020/12/26 15:49:01 kls Exp $
+ * $Id: recording.h 5.3 2021/01/19 20:38:28 kls Exp $
  */

 #ifndef __RECORDING_H
@@ -74,8 +74,6 @@ private:
   char *fileName;
   cRecordingInfo(const cChannel *Channel = NULL, const cEvent *Event = NULL);
   bool Read(FILE *f);
-  void SetData(const char *Title, const char *ShortText, const char *Description);
-  void SetAux(const char *Aux);
 public:
   cRecordingInfo(const char *FileName);
   ~cRecordingInfo();
@@ -93,6 +91,8 @@ public:
   bool Write(FILE *f, const char *Prefix = "") const;
   bool Read(void);
   bool Write(void) const;
+  void SetData(const char *Title, const char *ShortText, const char *Description);
+  void SetAux(const char *Aux);
   };

 class cRecording : public cListObject {
@@ -150,7 +150,7 @@ public:
        ///< Returns the full path name to the recording directory, including the
        ///< video directory and the actual '*.rec'. For disk file access use.
   const char *Title(char Delimiter = ' ', bool NewIndicator = false, int Level = -1) const;
-  const cRecordingInfo *Info(void) const { return info; }
+  cRecordingInfo *Info(void) const { return info; }
   const char *PrefixFileName(char Prefix);
   int HierarchyLevels(void) const;
   void ResetResume(void) const;
@@ -421,6 +421,10 @@ public:
 #define RUC_EDITINGRECORDING "editing"
 #define RUC_EDITEDRECORDING  "edited"
 #define RUC_DELETERECORDING  "deleted"
+#define RUC_RENAMEDRECORDING "renamed" // same directory, only the base name is changed
+#define RUC_MOVEDRECORDING   "moved"   // different directory (and maybe base name), or "copy" to other filesystem + delete original (triggers copying->copied->deleted)
+#define RUC_COPYINGRECORDING "copying"
+#define RUC_COPIEDRECORDING  "copied"

 class cRecordingUserCommand {
 private:

Alles anzeigen

Wie schon gesagt, meine C++ Kenntnissen sind nicht so prall. Deswegen möchte ich gern verstehen, warum ihr die "int Error" declaration in recordings.h patched habt, anstatt der eigentlichen Stelle mit "const cRecordingInfo *info = recording->Info();" im Plugin? Obendrein habt ihr noch den Aufruf "errors = info->Errors();" mit nem precompiler "#if" versehen?. Warum das?

Grüße

Jarod

kamel5 · 7. Juni 2021

Zitat von Jarod

Wenn ich das richtig sehe, dann wird "cRecordingInfo" nur nicht mehr als "const" deklariert seit Version 2.5.2.

Das ist aber nicht das Problem, um das es hier geht. Deshalb scheitert das Übersetzen nicht.

Das Problem, um das es hier geht befindet sich in dieser Zeile:

int Errors(void) const { return errors; }

Es fehlt einfach das "const" in der Mitte.

Warum also nicht einfach ein "cRecordingInfo" ohne const? Weil dann in allen Skins, die das "Errors()" nutzen wollen, das "const cRecordingInfo" geändert werden müsste.

Hier [skindesigner] Aktuelle Version habe ich dazu den letzten Stand geschrieben.

Wenn Du also den letzten Git-Stand vom Plugin benutzt, sollte es sowohl mit als auch ohne Patch funktionieren.

Grüße

kamel5

Jarod · 7. Juni 2021

Zitat von kamel5

Warum also nicht einfach ein "cRecordingInfo" ohne const? Weil dann in allen Skins, die das "Errors()" nutzen wollen, das "const cRecordingInfo" geändert werden müsste.

Jetzt hassu mich abgehängt... Bestehen die Skins nicht ausschließlich aus XML und SVG/PNG Files? Gibt es auch welche, die ihren eigenen C++-Code mitbringen? Hast du mal ein Beispiel, an dem ich sehen kann, wo etwas anzupassen wäre?

Jarod · 7. Juni 2021

Vergiss meine Frage... Wer lesen kann ist klar im Vorteil. Hab die Doku hier gefunden.

kls · 7. Juni 2021

cRecordingInfo::Errors() wird in der nächsten Version 'const' sein.

Ich bin gerade noch am Debuggen einiger Probleme mit PAT/PMT, dann spiele ich den aktuellen Stand ins GIT.

kamel5 · 7. Juni 2021

Zitat von Jarod

Bestehen die Skins nicht ausschließlich aus XML und SVG/PNG Files?

Im Prinzip ja, aber nur die Skins vom skindesigner. Es gibt aber auch noch andere Skins...

Grüße

kamel5

skindesigner plugin lässt sich mit vdr-2.5.4 nicht kompilieren

Jetzt mitmachen!

Teilen

Tags