VDR version 2.7.5 freigegeben

kls · April 12, 2025 at 12:46 PM New

Code

VDR version 2.7.5 is now available at the official VDR GIT archive

      git://git.tvdr.de

You can also get the latest stable version with

      git clone --branch stable/latest git://git.tvdr.de/vdr.git

or as a tar archive with

      http://git.tvdr.de/?p=vdr.git;a=snapshot;h=stable/latest;sf=tbz2

The changes since version 2.7.4:

- Added the "override" keyword to virtual functions reimplemented in derived classes.
  Plugins may want to do the same, but don't have to.
- Removed -Werror=overloaded-virtual from Makefile and Make.config(.template).
  Plugins may want to do the same, but don't have to.
- Renamed cStatus::Osd*2() to cStatus::Osd*().
  Plugins that use these recently introduced functions need to remove the '2' from the name.
- The new virtual function cSkinDisplayMenu::SetItemEvent(..., const cTimer *Timer) can be
  used to get full access to the timer (if any) defined for this event.
- Added mutex locks to protect creating/deleting cStatus and cEpgHandler objects (suggested
  by Markus Ehrnsperger).
- Making absolutely sure cEvent::Title() never returns NULL.
  APIVERSNUM is now 30007.
- Improved subtitle handling:
  + Subtitles now disappear as signaled in the data stream.
  + Subtitles are now kept in memory for at least 120 seconds, to have them readily
    available after a short rewind during replay.
- Subtitles can now be temporarily displayed after a fast rewind:
  + The setup option "DVB/Display subtitles" now has three settings: "no" and "always"
    behave like before, the new "after rewind" turns on subtitles after a fast rewind
    during replay, and off again when the point where the rewind was started is reached.
- Removed an unnecessary call to cDevice::GetVideoSize().
- Moved the call to Empty() back into the pmSlow case (thanks to Andreas Baierl).
- Fixed spurious times shown in the progress display when switching from "play" to "fast
  forward".
- Now deleting old recording info before reading modified info file (suggested by
  Stefan Hofmann).
- Plugins need to be rebuilt.

Homepage: http://www.tvdr.de
Facebook: https://www.facebook.com/VideoDiskRecorder

Have fun!

Klaus

Display More

seahawk1986 · April 12, 2025 at 1:25 PM New

kls Kann es sein, dass da noch Anpassungen an den Plugins nötig sind?

Bei einem ungepatchten VDR 2.7.5 meckert er über die Plugins hello und skincurses:

Code

*** Plugin hello:
CC hello.o
hello.c: In member function ‘virtual const char* cPluginHello::Version()’:
hello.c:27:56: error: ‘override’ was not declared in this scope
   27 |   virtual const char *Version(void) { return VERSION;  override}
      |                                                        ^~~~~~~~
hello.c: In member function ‘virtual const char* cPluginHello::Description()’:
hello.c:28:68: error: ‘override’ was not declared in this scope
   28 |   virtual const char *Description(void) { return tr(DESCRIPTION);  override}
      |                                                                    ^~~~~~~~
hello.c: In member function ‘virtual const char* cPluginHello::MainMenuEntry()’:
hello.c:33:72: error: ‘override’ was not declared in this scope
   33 |   virtual const char *MainMenuEntry(void) { return tr(MAINMENUENTRY);  override}
      |                                                                        ^~~~~~~~
make[1]: *** [Makefile:67: hello.o] Error 1

[...]
*** Plugin skincurses:
CC skincurses.o
skincurses.c: In member function ‘virtual void cSkinCursesDisplayTracks::SetAudioChannel(int)’:
skincurses.c:650:52: error: ‘override’ was not declared in this scope; did you mean ‘overwrite’?
  650 |   virtual void SetAudioChannel(int AudioChannel) { override}
      |                                                    ^~~~~~~~
      |                                                    overwrite
skincurses.c: In member function ‘virtual const char* cPluginSkinCurses::MainMenuEntry()’:
skincurses.c:802:72: error: ‘override’ was not declared in this scope; did you mean ‘overwrite’?
  802 |   virtual const char *MainMenuEntry(void) { return tr(MAINMENUENTRY);  override}
      |                                                                        ^~~~~~~~
      |                                                                        overwrite
make[1]: *** [Makefile:67: skincurses.o] Error 1

Display More

MarkusE · April 12, 2025 at 1:44 PM New

Ich werde meine Plugins anpassen, es wird aber ein paar Tage dauern.

seahawk1986 · April 12, 2025 at 1:49 PM New

MarkusE Was ich nicht verstehe ist das override innerhalb des Körpers einer Funktion - https://en.cppreference.com/w/cpp/language/override zeigt das nur außerhalb.

Statt

Code

virtual const char *Version(void) { return VERSION;  override}

hätte ich da

Code

virtual const char *Version(void) override { return VERSION; }

erwartet.

seahawk1986 · April 12, 2025 at 2:43 PM New

Damit baut es zumindest:

Diff

Index: vdr-stable.latest-add3d1c/PLUGINS/src/hello/hello.c
===================================================================
--- vdr-stable.latest-add3d1c.orig/PLUGINS/src/hello/hello.c    2025-04-12 08:32:04.000000000 +0200
+++ vdr-stable.latest-add3d1c/PLUGINS/src/hello/hello.c 2025-04-12 14:35:28.961672113 +0200
@@ -24,13 +24,13 @@
 public:
   cPluginHello(void);
   virtual ~cPluginHello() override;
-  virtual const char *Version(void) { return VERSION;  override}
-  virtual const char *Description(void) { return tr(DESCRIPTION);  override}
+  virtual const char *Version(void) override { return VERSION;}
+  virtual const char *Description(void) override { return tr(DESCRIPTION); }
   virtual const char *CommandLineHelp(void) override;
   virtual bool ProcessArgs(int argc, char *argv[]) override;
   virtual bool Start(void) override;
   virtual void Housekeeping(void) override;
-  virtual const char *MainMenuEntry(void) { return tr(MAINMENUENTRY);  override}
+  virtual const char *MainMenuEntry(void) override { return tr(MAINMENUENTRY); }
   virtual cOsdObject *MainMenuAction(void) override;
   virtual cMenuSetupPage *SetupMenu(void) override;
   virtual bool SetupParse(const char *Name, const char *Value) override;
Index: vdr-stable.latest-add3d1c/PLUGINS/src/skincurses/skincurses.c
===================================================================
--- vdr-stable.latest-add3d1c.orig/PLUGINS/src/skincurses/skincurses.c  2025-04-12 08:32:04.000000000 +0200
+++ vdr-stable.latest-add3d1c/PLUGINS/src/skincurses/skincurses.c       2025-04-12 14:39:36.164117836 +0200
@@ -647,7 +647,7 @@
   cSkinCursesDisplayTracks(const char *Title, int NumTracks, const char * const *Tracks);
   virtual ~cSkinCursesDisplayTracks() override;
   virtual void SetTrack(int Index, const char * const *Tracks) override;
-  virtual void SetAudioChannel(int AudioChannel) { override}
+  virtual void SetAudioChannel(int AudioChannel) override;
   virtual void Flush(void) override;
   };

@@ -799,7 +799,7 @@
   virtual bool Initialize(void) override;
   virtual bool Start(void) override;
   virtual void Housekeeping(void) override;
-  virtual const char *MainMenuEntry(void) { return tr(MAINMENUENTRY);  override}
+  virtual const char *MainMenuEntry(void) override { return tr(MAINMENUENTRY); }
   virtual cOsdObject *MainMenuAction(void) override;
   virtual cMenuSetupPage *SetupMenu(void) override;
   virtual bool SetupParse(const char *Name, const char *Value) override;

Display More

seahawk1986 · April 12, 2025 at 3:20 PM New

Nächstes Problem ist das status-Plugin, wenn man es mit den Build-Flags -Wall -Werror=overloaded-virtual kompiliert:

Code

*** Plugin status:
CC status.o
In file included from status.c:10:
/home/alexander/src/vdr-stable.latest-add3d1c/include/vdr/status.h:94:16: error: ‘virtual void cStatus::OsdCurrentItem(const char*)’ was hidden [-Werror=overloaded-virtual=]
   94 |   virtual void OsdCurrentItem(const char *Text) {}
      |                ^~~~~~~~~~~~~~
status.c:33:16: note:   by ‘virtual void cStatusTest::OsdCurrentItem(const char*, int)’
   33 |   virtual void OsdCurrentItem(const char *Text, int Index) override;
      |                ^~~~~~~~~~~~~~
/home/alexander/src/vdr-stable.latest-add3d1c/include/vdr/status.h:90:16: error: ‘virtual void cStatus::OsdItem(const char*, int)’ was hidden [-Werror=overloaded-virtual=]
   90 |   virtual void OsdItem(const char *Text, int Index) {}
      |                ^~~~~~~
status.c:32:16: note:   by ‘virtual void cStatusTest::OsdItem(const char*, int, bool)’
   32 |   virtual void OsdItem(const char *Text, int Index, bool Selectable) override;
      |                ^~~~~~~
/home/alexander/src/vdr-stable.latest-add3d1c/include/vdr/status.h:84:16: error: ‘virtual void cStatus::OsdStatusMessage(const char*)’ was hidden [-Werror=overloaded-virtual=]
   84 |   virtual void OsdStatusMessage(const char *Message) {}
      |                ^~~~~~~~~~~~~~~~
status.c:30:16: note:   by ‘virtual void cStatusTest::OsdStatusMessage(eMessageType, const char*)’
   30 |   virtual void OsdStatusMessage(eMessageType Type, const char *Message) override;
      |                ^~~~~~~~~~~~~~~~
cc1plus: some warnings being treated as errors
make[1]: *** [Makefile:66: status.o] Error 1

Display More

kamel5 · April 12, 2025 at 3:34 PM New

Quote from seahawk1986

Nächstes Problem ist das status-Plugin, wenn man es mit den Build-Flags -Wall -Werror=overloaded-virtual kompiliert:

Du musst das "-Werror=overloaded-virtual" aus der Make.config im VDR-Verzeichnis entfernen.

Das hatte kls schon mal mal irgendwo geschrieben, das es sonst eine Fehlermeldung gibt.
Bei mir hat es danach kompiliert.

Grüße
kamel5

SHofmann · April 12, 2025 at 4:07 PM New

Quote from MarkusE

Ich werde meine Plugins anpassen, es wird aber ein paar Tage dauern.

Mit den Patches von seahawk1986 haben die aktuellen Master-Branches von Live und TvScraper ohne Probleme gebaut. Diesbezüglich ist also keine Aktion erforderlich,

Zabrimus · April 12, 2025 at 5:11 PM New

Ich musste noch 2 Patches anpassen (zapcockpit und undelete).

vdr-2.7.5-undelete.patch.txt

vdr-2.7.5-zapcockpit.patch.txt

SHofmann · April 12, 2025 at 5:48 PM New

Bei Live erscheint mit VDR 2.7.5 bei "OSD" in der Fernbedienung kein Menü mehr:

Wohl aber reagiert der VDR auf Eingaben, etwa "4" zum Aufruf der Aufzeichnungen – die dann aber ebenfalls nicht angezeigt werden. Interessanterweise werden aber bspw. die Status-Informationen beim Programmwechsel ausgegeben:

Hat sich denn im Umfeld von cOsdMenu bzw. cOsdItem dahingehend etwas Wesentliches geändert, das diesen Effekt erklären könnte?

Beim A/B-Vergleich verhält sich der VDR 2.7.4 wie erwartet.

FireFly · April 12, 2025 at 6:02 PM New

Quote from kls

- Renamed cStatus::Osd*2() to cStatus::Osd*().
Plugins that use these recently introduced functions need to remove the '2' from the name.

SHofmann Vermutlich deshalb

SHofmann · April 12, 2025 at 6:29 PM New

Wenn die umbenannten Funktionen nicht rückwärtskompatibel sind und auch keinen Kompilierfehler liefern (läuft durch), finde ich das keine schöne Lösung… hat aber geholfen. Danke!

seahawk1986 · April 12, 2025 at 9:58 PM New

Quote from kamel5

Du musst das "-Werror=overloaded-virtual" aus der Make.config im VDR-Verzeichnis entfernen.
Das hatte kls schon mal mal irgendwo geschrieben, das es sonst eine Fehlermeldung gibt.
Bei mir hat es danach kompiliert.

-Werror=overloaded-virtual=1 scheint mit gcc zu genügen, man muss das nicht komplett rauswerfen.

kls · April 12, 2025 at 10:37 PM New

Quote from seahawk1986

Kann es sein, dass da noch Anpassungen an den Plugins nötig sind?

Sorry, ich hatte die Änderungen per sed automatisch durchgeführt, aber leider übersehen, dass hello und skincurses nicht neu compiliert wurden. Werde das in meinen Testablauf integrieren.

MarkusE · April 13, 2025 at 8:04 PM New

Im live git ist ein Update. Damit funktioniert die OSD Fernbedienung wieder mit VDR 2.7.5

wakko · April 14, 2025 at 9:14 AM New

MarkusE

Kannst du bitte schaun wegen dynamite?

Ich bekomme mit 2.7.5 rejects

MarkusE · April 14, 2025 at 10:36 AM New

Quote from wakko

MarkusE
Kannst du bitte schaun wegen dynamite?
Ich bekomme mit 2.7.5 rejects

Ja, werde ich mir anschauen

FireFly · April 14, 2025 at 6:44 PM New

Ich bin gerade auf ein Problem gestoßen, dass bei einigen frischen Aufnahmen die doppelte Spielzeit der tatsächlichen Dauer angezeigt wird.
- Das Problem tritt auch mit den Standard-Skins (LCARS, Klass. VDR) auf.
- in der info-Datei steht die korrekte Anzahl FramesPerSecond = 50 drin
- über NFS bei einem anderen VDR eingelesen wird überall die korrekte Dauer 1:01 angezeigt (nach einem Neustart des VDR sicher auch, aber da laufen derzeit noch Aufnahmen)
- beim Abspielen zeigt der Balken die korrekte Dauer 1:00:58 an

Ich denke, das kommt durch das doppelte Vorhalten der Variablen framesPerSecond in cRecording und cRecordingInfo wenn die Frame-Info etwas später kommt und nur in cRecordingInfo gesetzt wird. Die Skins nutzen cRecording->LengthInSeconds() die mit der falschen cRecording::framesPerSecond rechnet, ebenso LengthInSecondsAfterEdit()
kls: Jedes cRecording-Objekt hat ja auch ein cRecordingInfo-Objekt, sollte man dann nicht einfach immer auf cRecordingInfo::FramesPerSecond() zugreifen anstatt die Variable nochmals mitzuführen? Dann würde ich mal einen Patch bauen

kfb77 · April 14, 2025 at 7:58 PM New

Live zeigt auch die doppelte Länge an, "touch .update" bringt nichts, nach VDR restart stimmt die Länge.

kls · April 14, 2025 at 10:26 PM New

FireFly Das hab ich in letzter Zeit auch ein paarmal beobachtet. Danke für den Hinweis, schau ich mir an.

VDR version 2.7.5 freigegeben

Participate now!

Similar Threads

25 Jahre VDR

Users Viewing This Thread