[Announce] VDR developer version 2.1.2 (updated)

Zitat von kls

Den DrawMenuFrame() Aufruf halte ich an der Stelle nicht für sinnvoll/notwendig, da in cSkinLCARSDisplayMenu::SetMenuCategory() ja entweder die eine oder die andere Form gezeichnet wird,

Ich dachte eigentlich, der Aufruf von SetMenuCategory ist optional, nur wenn man eine andere Menükategorie setzen will. Die anderen Skins brauchen keinen zusätzlichen Aufruf von SetMenuCategory.

Wenn SetMenuCategory notwendig wäre, bräuchte man die ganze Initialisierung in diesem Patch nicht, da diese Variablen ja auch in SetMenuCategory gesetzt werden.

Hallo,

ich habe am Wochenende testweise auf die Version 2.1.2 umgestellt.
Ohne jeden Patch und ohne jedes weitere Plugin außer dvbhddevice und dvbsddevice kann ich keine Aufnahmen mehr abspielen, die ich mit Versionen <= 2.1.1 gemacht habe.
Die Wiedergabe bricht nach spätestens 20 Sekunden ab, Springen und Spulen geht ebenfalls nicht.

Ich habe eine TT Premium S2-6400 als Ausgabedevice.

Ich habe bisher noch nichts von diesem Problem gelesen - hat da jemand ne Idee, was das sein könnte?

Thx!

Ciao.
Michael.

Zitat von nobanzai

Hallo,

ich habe am Wochenende testweise auf die Version 2.1.2 umgestellt.
Ohne jeden Patch und ohne jedes weitere Plugin außer dvbhddevice und dvbsddevice kann ich keine Aufnahmen mehr abspielen, die ich mit Versionen <= 2.1.1 gemacht habe.
Die Wiedergabe bricht nach spätestens 20 Sekunden ab, Springen und Spulen geht ebenfalls nicht.

Ich habe eine TT Premium S2-6400 als Ausgabedevice.

Ich habe bisher noch nichts von diesem Problem gelesen - hat da jemand ne Idee, was das sein könnte?

Thx!

Ciao.
Michael.

Alles anzeigen

Mach doch bitte einen Thread dafür auf.

Hi,

Zitat von TheChief

Mach doch bitte einen Thread dafür auf.

jo, danke für den Hinweis.
In welchen Bereich würde das denn am besten passen?
Bisher habe ich bei Developer-Versionen eigentlich immer nur direkt in den Ankündigungs-Thread gepostet, aber ja, ich hab eigentlich schon mitbekommen, dass das eher unerwünscht ist.
Nur weiß ich dann eben nicht so genau, wo dazu gehörige Fehler abgehandelt werden sollen.

Ciao.
Michael.

http://www.vdr-portal.de/board17-developer/board97-vdr-core/

Zitat von Copperhead

http://www.vdr-portal.de/board17-developer/board97-vdr-core/

thx!

kämpft ausser mir noch jemand mit vdr --genindex?

Index regenerieren im UI läuft wie geplant und baut den Index, danach die Index Datei wieder löschen und dasselbe mittels --genindex auf der Konsole ausgeführt liefert mir reproduzierbar einen Segfault in libc:

/usr/local/src/vdr/vdr-2.1.2/vdr --genindex=/var/lib/video/True_blood/At_last/2013-11-29.00.30.203-0.rec/
Speicherzugriffsfehler (Speicherabzug geschrieben)
gdb /usr/local/src/vdr/vdr-2.1.2/vdr ./core
Reading symbols from /usr/local/src/vdr/vdr-2.1.2/vdr...done.
Loaded symbols for /lib/i686/cmov/libc.so.6                                                                                                                                                               
<<snip>>
Core was generated by `/usr/local/src/vdr/vdr-2.1.2/vdr --genindex=/var/lib/video/True_blood/At_l'.
Program terminated with signal 11, Segmentation fault.
#0  0xb7496df5 in ?? () from /lib/i686/cmov/libc.so.6
(gdb) bt
#0  0xb7496df5 in ?? () from /lib/i686/cmov/libc.so.6
#1  0x0813b644 in strstr (this=0xbfb16560, FileName=0x82600a8 "/var/lib/video/True_blood/At_last/2013-11-29.00.30.203-0.rec") at /usr/include/string.h:335
#2  cRecording (this=0xbfb16560, FileName=0x82600a8 "/var/lib/video/True_blood/At_last/2013-11-29.00.30.203-0.rec") at recording.c:814
#3  0x0813be54 in GenerateIndex (FileName=0xbfb177ee "/var/lib/video/True_blood/At_last/2013-11-29.00.30.203-0.rec/") at recording.c:2679
#4  0x0818c589 in main (argc=2, argv=0xbfb16904) at vdr.c:345

Was genau passt ihm da nicht in der recording.c?
Der vdr isr ungepatcht, gebaut mit "make LCLBLD=1"

Zitat von Ulrich Eckhardt

Ich dachte eigentlich, der Aufruf von SetMenuCategory ist optional, nur wenn man eine andere Menükategorie setzen will. Die anderen Skins brauchen keinen zusätzlichen Aufruf von SetMenuCategory.

Wenn SetMenuCategory notwendig wäre, bräuchte man die ganze Initialisierung in diesem Patch nicht, da diese Variablen ja auch in SetMenuCategory gesetzt werden.

Die Initialisierung der Variablen kostet nicht viel und vermeidet mögliche Abstürze, was sicher sinnvoll ist.

Es wäre sehr schade, wenn die LCARS-Skin im Konstruktor DrawMenuFrame() aufrufen müsste, denn das wäre im Normalfall immer unnötiger Aufwand (VDR selber ruft ja *immer* SetMenuCategory() auf).
Wenn ich es mir recht überlege, dann müsste die MenuCategory eigentlich bereits im Konstruktor bekannt sein und somit im Aufruf von

cSkin::DisplayMenu()

und von dort an die konkrete cSkinDisplayMenu-Klasse durchgereicht werden.
Das wäre vermutlich die sauberste Lösung, würde aber bedeuten, daß alle Skin-Plugins entsprechend angepasst werden müssten, was sicherlich wieder einigen Aufruhr verursachen dürfte ;-).

Ich tendiere daher dazu, einfach den Aufruf von SetMenuCategory() verpflichtend zu machen.

Meinungen dazu?

Klaus

Moin,

Zitat von kls

Ich tendiere daher dazu, einfach den Aufruf von SetMenuCategory() verpflichtend zu machen.

Meinungen dazu?

finde ich gut...wie würde denn "verpflichtend machen" genauer ausschauen?

Ciao Louis

Zitat von louis

Moin,

finde ich gut...wie würde denn "verpflichtend machen" genauer ausschauen?

Eigentlich nur durch einen entsprechenden Kommentar im Header-File.
Für alles andere müssten ja die Plugins angepasst werden...

Klaus

Zitat von kls

Ich tendiere daher dazu, einfach den Aufruf von SetMenuCategory() verpflichtend zu machen.

OK, dann übernehme ich die entsprechenden Patche in mein cd-player Plugin.