[ANNOUNCE] VDR Version 2.7.3 freigegeben

kls · October 12, 2024 at 3:43 PM

VDR version 2.7.3 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=s…/latest;sf=tbz2

The changes since version 2.7.2:

- Removed defining DEPRECATED_* macros with value 0, because this is the preprocessor's

default (suggested by Winfried Köhler).

- Fixed error checking in case of large PTS discontinuities (reported by Matthias Senzel).

- Fixed handling negative values in cSource::Position() on systems where 'int' is 64 bit

(reported by Markus Ehrnsperger, fix suggested by Winfried Köhler).

- Fixed expiring of one-time VPS timers in case there is more than one event with the

same VPS time (suggested by Markus Ehrnsperger).

- The Channel+/- keys can now be used to jump between errors while replaying a recording

(suggested by Stefan Hofmann).

- Added vdrrootdir and incdir to vdr.pc (thanks to Stefan Hofmann).

- Plugins that have been built with API version 5 do not need to be rebuilt.

Homepage: http://www.tvdr.de

Facebook: https://www.facebook.com/VideoDiskRecorder

Have fun!

Klaus

SHofmann · October 12, 2024 at 4:19 PM

Quote from kls

Removed defining DEPRECATED_* macros with value 0, because this is the preprocessor's
default

Das führt aber leider zu jeder Menge Warnungen, weil der Compiler bei "empfindlichen Einstellungen" solch offene Enden – aus Gründen der nicht-funktionalen Code-Qualität zurecht – nicht mag. Hier ein Beispiel:

Quote

In file included from /usr/local/vdr/include/vdr/eit.h:13,

from /usr/local/vdr/include/vdr/device.h:16,

from /usr/local/vdr/include/vdr/dvbdevice.h:15,

from /usr/local/vdr/include/vdr/menuitems.h:14,

from /usr/local/vdr/include/vdr/plugin.h:14,

from live.h:17,

from live.cpp:7:

/usr/local/vdr/include/vdr/filter.h:50:5: warning: "DEPRECATED_SECTIONSYNCER_SYNC_REPEAT" is not defined, evaluates to 0 [-Wundef]

Display More

Das war wohl eher nicht beabsichtigt. Das würde meines Erachtens nur funktionieren, wenn #ifdef bzw. #if defined(…) anstelle von #if verwendet würde.

PS: Und wenn man das im VDR in den entsprechenden Files patcht, kompiliert Live auch wieder ohne Warnings. Es sind leider mehrere Dateien betroffen, hier der Patch:

Fixed warnings of undefined DEPRECATED macros.patch.txt

Viele Grüße

Stefan

wirbel · October 12, 2024 at 4:32 PM

Das macht jede Menge Sinn. So wird das entsprechende Plugin endlich umgestellt.

SHofmann · October 12, 2024 at 4:41 PM

Quote from wirbel

Das macht jede Menge Sinn. So wird das entsprechende Plugin endlich umgestellt.

Das Argument kam schon einmal, aber hier ist der Code des VDR das Problem…

kls · October 12, 2024 at 4:43 PM

Solche Sachen wurden ganz früher mal mit #ifdef gemacht, wurde dann aber auf #if umgestellt, um es von aussen überschreiben zu können. Dass das jetzt solche Meldungen verursacht, wusste ich nicht (bei mir kommen die nicht). In der nächsten Version werde ich diese Codeteile eh ganz entfernen und dann nur noch [[deprecated]] verwenden.

MarkusE · October 12, 2024 at 4:45 PM

Quote from wirbel

Das macht jede Menge Sinn. So wird das entsprechende Plugin endlich umgestellt.

Immer auf die armen, unschuldigen Plugin Entwickler.

live ist schon umgestellt.

Fehler ist in VDR, und müsste beim Compilieren von VDR auch angezeigt werden, wenn der Warning level des Compilers entsprechend eingestelt wird.

kls · October 12, 2024 at 4:48 PM

Quote from wirbel

Nein, nicht VDR ist das Problem. live ist das Problem

Da bin ich mir nicht so sicher. Live wäre nur dann das Problem, wenn es DEPRECATED_SECTIONSYNCER_SYNC_REPEAT=1 definieren würde, was es aber offensichtlich nicht tut.

wirbel · October 12, 2024 at 4:49 PM

Quote

Immer auf die armen, unschuldigen Plugin Entwickler.

Die durchaus geschätzt sind..

wirbel · October 12, 2024 at 4:49 PM

Quote from kls

Da bin ich mir nicht so sicher. Live wäre nur dann das Problem, wenn es DEPRECATED_SECTIONSYNCER_SYNC_REPEAT=1 definieren würde, was es aber offensichtlich nicht tut.

Da hast du Recht.

MarkusE · October 12, 2024 at 4:49 PM

Anbei ein Patch gegen VDR, der das Problem behebt.

wirbel , kannst Du das bitte mal testen

SHofmann · October 12, 2024 at 4:50 PM

Und wenn Live das so definieren würde, wäre der Compiler – zumindest bezüglich der Makros – ja eh glücklich…

SHofmann · October 12, 2024 at 4:59 PM

Quote from MarkusE

Anbei ein Patch gegen VDR, der das Problem behebt.
wirbel , kannst Du das bitte mal testen

Dein Patch entspricht genau meinem weiter oben geposteten, und der kompiliert ohne Warnungen.

PS: Und der VDR wie auch Live funktionieren erwartungsgemäß ohne Probleme.

wirbel · October 12, 2024 at 5:36 PM

Ich vote dafür, diese Warnungen jetzt noch bestehen zu lassen.

Ich denke, dass diese Warnungen durchaus ihre Berechtigung haben und bestehen bleiben dürften und sogar sollten, damit VDR möglichst bald ohne jeden veralteten 'deprecated' Code auskommt. Das gibt weniger Zeilen Code und besser menschlich lesbaren Code in VDR, weniger '#if(def) .. #else .. #endif.

Viele von uns Plugin-Entwicklern sind doch mit dem letzten VDR auf die Nase gefallen, nachdem nun teilweise viel(!) später endlich mal alter/toter Code in VDR entfernt wurde.

Soweit es geht, sollten alle Plugin Entwickler, die noch veralteten Code verwenden, animiert werden, diesen Code mal anzufassen && upzugraden. Und dazu muss irgendwo eine störende (ja, ich weiß..) Info kommen. Die zu Issues auf github, Foren Beiträgen oder emails an den Entwickler führt, und final Aktionen triggert. Ich kenne das selbst von altem Code, man vergisst ab && zu alte Baustellen.

Ich bin immer dafür jede Warnung auszuschalten, die von To-Do Lists ablenkt. Aber Warnungen, die wirklich nützlich sind und noch nicht abgearbeitet sind, sollten durchaus bleiben als Erinnerung bzw. Reminder. Deswegen sind es ja auch keine 'fatalen Fehler', eben nur eine Warnung. Jeder kann immer noch das Plugin oder VDR bauen.

SHofmann · October 12, 2024 at 6:00 PM

Quote from wirbel

Ich denke, dass diese Warnungen durchaus ihre Berechtigung haben und bestehen bleiben dürften und sogar sollten, damit VDR möglichst bald ohne jeden veralteten 'deprecated' Code auskommt.

Das Problem ist aber doch, dass selbst Plugins wie Live, die vollständig auf die veralteten Funktionen verzichten, auch die Warnungen bringen. Weil das Problem in diesem Fall eben nicht beim Plugin, sondern beim VDR liegt.

Ob die Warnungen kommen, hängt ausschließlich von der im Makefile vorgegebenen Warnstufe des Compilers ab. Wenn man ganz "krätzig" ist und mit -Werror übersetzt, wie in der Industrie üblich also jede Warnung gleich als Kompilierfehler wertet, scheitert das Bauen dann zwangsläufig.

wirbel · October 12, 2024 at 7:09 PM

Dann führt das im besten Falle zu einem Patch für den VDR, der bald ohnehin kommt.

Und den man doch gerne an den Autor zur Hilfestellung bereitstellen kann..

Wo ist das Problem?

SHofmann · October 12, 2024 at 7:11 PM

Ich denke, dass deine Argumentation in die falsche Richtung geht… und Klaus habe ich einen Patch hochgeladen.

wirbel · October 12, 2024 at 7:18 PM

Ich denke, dass der Autor sicher seinen eigenen Weg gehen wird, nach seinen Rules.

kls · October 12, 2024 at 9:07 PM

wirbel Ich gehe mal davon aus, dass diese Warnungen bei dir nicht aufgetreten sind, sonst hättest du diese Änderung ja nicht vorgeschlagen, oder? Ich habe mich da wohl zu sehr darauf verlassen, dass es damit kein Problem gibt, sonst hätte ich sie auf jeden Fall nicht gemacht.

@alle Ich werde das jetzt nicht nach #ifdef ändern, sondern in der nächsten Version diesen Code ganz entfernen.

wirbel · October 12, 2024 at 9:45 PM

Sorry...! Hätte nie gedacht, dass neueste compiler da auch nur warnen.

kls · October 12, 2024 at 9:59 PM

Wer mit den Warnungen wegen der DEPRECATED-Macros Probleme hat, könnte mal beiliegenden Patch probieren. Damit werden diese Codeteile komplett entfernt, und das kommt auch in die nächste Version so rein.

Bei der Stelle in cControl::Launch() steht zwar ein "TODO obsolete once DEPRECATED_CCONTROL is gone", aber das fasse ich lieber nicht an, daher habe ich hier nur das TODO entfernt. Falls jemand mehr dazu sagen kann, bitte melden.

[ANNOUNCE] VDR Version 2.7.3 freigegeben

Participate now!

Users Viewing This Thread