[ANNOUNCE] VDR Version 2.7.3 freigegeben

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

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

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…

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.

Quote

Immer auf die armen, unschuldigen Plugin Entwickler.

Die durchaus geschätzt sind..

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.

Anbei ein Patch gegen VDR, der das Problem behebt.

wirbel , kannst Du das bitte mal testen

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

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.

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.

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.

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?

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

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

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