Skindesigner 0.8.8

louis · 12. Februar 2016

Moin,

bzgl. des Crashes mit dem gedöns Patch: kann mal jemand bitte den folgenden Patch testen (coreengine/viewlist.c ab Zeile 405):

Code

void cViewListMain::Set(const char *text, int index, bool current, bool selectable) {
+   if (i < 0 || i  >= numElements)
+       return;
    if (!current)
        listMain[index]->StopScrolling();
...

Ciao Louis

louis · 12. Februar 2016

Moin,

Zitat von tomas

Das habe ich auch schon beobachtet. Als Endzeit wird hier, unabhängig von der eingestellten Dauer für Direktaufnahmen, die Endzeit des Events benutzt, das zum Zeitpunkt des Starts der Sofortaufnahme lief. Wenn man jetzt die Aufnahme anschaut, stimmt die Anzeige des Fortschrittsbalkens genau bis zum Ende dieses Events. Danach ist dann 'Ende Gelände'.
Es wird wohl schwierig sein, das abzufangen. Selbst wenn man hier den eingestellten Wert für die Dauer einer Direktaufzeichnung heranzieht, passt der, wenn die Direktaufzeichnung schon vorher abgebrochen wird, ja auch nicht mehr.

Schaut man die Direktaufnahme erst an, wenn sie abgeschlossen ist, passt ja alles, weil die tatsächliche Länge dann bekannt ist.

Jo, irgendwas muss ich benutzen. Ich habe ja eine "Timeshift" Funktionalität im displayreplay eingebaut, damit man beim Anschauen einer laufenden Aufnahme den aktuellen "Zustand" der Aufnahme erkennen kann. Da brauche ich halt irgendeine Endezeit. Wenn die Aufnahme über einen normalen Timer passiert, funktioniert ja alles. Ich sollte wohl besser versuchen, eine "Sofortaufnahme" zu erkennen und dann diesen Mechanismus einfach völlig abschalten. Wäre das sinnvoller, oder passt es bei den meisten Direktaufnahmen und nur bei manchen nicht?

Ohne jetzt im VDR Code nachgesehen zu haben...weiss jemand zufällig, ob eine Direktaufnahme erkennbar ist?

Ciao Louis

mini73 · 12. Februar 2016

i oder index?

Lars.

tomas · 12. Februar 2016

Mit 'i' kompiliert es garnicht:

Code

coreengine/viewlist.c: In member function ‘void cViewListMain::Set(const char*, int, bool, bool)’:
coreengine/viewlist.c:406:6: error: ‘i’ was not declared in this scope
  if (i < 0 || i  >= numElements)
      ^

aber mit index ..... und es funktioniert!

Kein Crash mehr

Gruß,
Tomas

tomas · 12. Februar 2016

Zitat von louis

Ich sollte wohl besser versuchen, eine "Sofortaufnahme" zu erkennen und dann diesen Mechanismus einfach völlig abschalten.

Abschalten ist wohl das einfachste.

Zitat von louis

oder passt es bei den meisten Direktaufnahmen und nur bei manchen nicht?

Ist immer so. Ich hatte das beim Implementieren der Timeshiftanzeige in meinen Skins beim Testen schon bemerkt.
Wie geschrieben, der Balken läuft bis zu dem Zeitpunkt, an dem das Event endet, das zum Zeitpunkt des Starts der Direktaufnahme aktiv war. Beispielsweise starte ich um 20:10 eine Direktaufzeichnung auf dem Ersten, es läuft die Tagesschau bis 20:15. Wenn ich jetzt die Direktaufzeichnung anschaue, läuft der Balken korrekt, bis die Tagesschau zu Ende ist. Dann ist er am rechten Rand.

Zitat von louis

Ohne jetzt im VDR Code nachgesehen zu haben...weiss jemand zufällig, ob eine Direktaufnahme erkennbar ist?

Ohne Ahnung davon zu haben, aber der vdr setzt ja bei einer Direktaufzeichnung auf jeden Fall ein '@' vor den Titel, also @Tagesschau z.B.

Gruß,
Tomas

Hollywood · 12. Februar 2016

Hi,

hi die Crashes sind nun bei mir mit dem Patch auch weg.

Ich würde das Abschalten der Timeshiftanzeige bei Direktaufnahmen auch bevorzugen. Ich hatte diese Timeshiftgeschichte zwischenzeitlich schon komplett rausgenommen um immer eine braucbare Anzeige zu haben.

PS: Hinsichtlich der Customtokens im displayreplay fehlt in viewdisplayreplay.c unter void cViewReplay::Flush(void) { nur ein Render((int)eVeDisplayReplay::customtokens); danach funktionnieren die Custumtokens auch wieder während der Wiedergabe von Aufnahmen.

rookie1 · 12. Februar 2016

Beim navigieren im epgsearch geht er mit der 0.8 in die Knie.

Hab log mit angehängt

rookie

Hollywood · 13. Februar 2016

Im mailbox-plugin wird bei mir der Inhalt der Nachrichten nicht angezeigt.

louis · 13. Februar 2016

Moin,

im Git ist eine Version 0.8.4 mit den folgenden Bugfixes:

Code

- disabled timeshift display for instant recordings
- fixed scaling bug when closing tvguideng
- display customtokens in displayreplay
- removed % from cutted recordings recname
- fixed direct display of pluginmenu with keymacro
- fixed fading of currentelement
- fixed crash if not visible menuitem is called

Frodo: Bezüglich des Problems, dass das TV Bild beim Schließen mit "menu" (ich habe das auch so eingestellt) nicht auf Vollbild skaliert wird: ich konnte das bei mir nicht so wirklich nachstellen. Auch mit dem blackholefrodo Skin nicht. Da war es genau ein mal so, ansonsten wurde das Bild immer brav skaliert...das ist irgendwie ein bisschen schräg. Ich habe jetzt mal auf Verdacht was geändert, ich hoffe das bringt was Ansonsten müsstest du mir vielleicht mal deine genauen Setup Werte von deinem benutzen Skin geben, ggf. müsste ich noch was einstellen.

Ciao Louis

marco · 13. Februar 2016

Beim Kompilieren von vdr-plugin-skindesigner-0.8.4 mit dem neuen gcc6 Kompiler bekomme ich die folgende Fehlermeldung:

Code

g++ -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -fPIC -c -DLIBAPIVERSION=0 -DLIBVERSION="0.1.0"  -o pluginstructure.o pluginstructure.c
In file included from coreengine/area.h:10:0,
                 from coreengine/viewelement.h:12,
                 from coreengine/viewelementplugin.h:4,
                 from coreengine/view.h:13,
                 from coreengine/viewdisplaychannel.h:4,
                 from designer.h:7,
                 from config.h:10,
                 from skindesigner.c:12:
coreengine/definitions.h:66:30: error: could not convert 'count' from 'eVeDisplayMenu' to 'int'
     timers = eVeDisplayMenu::count,
                              ^~~~~
coreengine/definitions.h:81:36: error: could not convert 'count' from 'eVeDisplayMenu' to 'int'
     detailheader = eVeDisplayMenu::count,
                                    ^~~~~
coreengine/definitions.h:510:40: error: could not convert 'count' from 'eScraperPosterBannerIT' to 'int'
     islivetv = eScraperPosterBannerIT::count,
                                        ^~~~~
coreengine/definitions.h:520:37: error: could not convert 'count' from 'eScraperPosterBannerST' to 'int'
     title = eScraperPosterBannerST::count,
                                     ^~~~~
coreengine/definitions.h:1041:31: error: could not convert 'count' from 'eScraperHeaderST' to 'int'
     title = eScraperHeaderST::count,
                               ^~~~~
coreengine/definitions.h:1056:36: error: could not convert 'count' from 'eScraperHeaderIT' to 'int'
     daynumeric = eScraperHeaderIT::count,
                                    ^~~~~
coreengine/definitions.h:1070:30: error: could not convert 'count' from 'eScraperHeaderST' to 'int'
     name = eScraperHeaderST::count,
                              ^~~~~
coreengine/definitions.h:1084:36: error: could not convert 'count' from 'eScraperHeaderIT' to 'int'
     daynumeric = eScraperHeaderIT::count,
                                    ^~~~~
coreengine/definitions.h:1209:25: error: could not convert 'count' from 'eScraperST' to 'int'
     title = eScraperST::count,
                         ^~~~~
coreengine/definitions.h:1227:30: error: could not convert 'count' from 'eScraperIT' to 'int'
     daynumeric = eScraperIT::count,
                              ^~~~~
coreengine/definitions.h:1258:24: error: could not convert 'count' from 'eScraperST' to 'int'
     name = eScraperST::count,
                        ^~~~~
coreengine/definitions.h:1283:30: error: could not convert 'count' from 'eScraperIT' to 'int'
     daynumeric = eScraperIT::count,
                              ^~~~~
coreengine/definitions.h:1427:24: error: could not convert 'count' from 'eScraperST' to 'int'
     name = eScraperST::count,
                        ^~~~~
coreengine/definitions.h:1438:30: error: could not convert 'count' from 'eScraperIT' to 'int'
     daynumeric = eScraperIT::count,
                              ^~~~~

Alles anzeigen

louis · 13. Februar 2016

marco: das kam jetzt aber nicht erst mit er 0.8.4 oder?

Offensichtlich ist bei dieser gcc Version der cast auf int für enums nicht mehr erlaubt...das ist unschön.

Ciao Louis

mamomoz · 13. Februar 2016

Hallo Louis,
was mir beim Testen aufgefallen ist beim faden und shiften mit openglosd:

Bei Deinen skins laufen die Animationen super smoothy.
Bei meinem angepassten blackhole immer etwas ruckelig.
Auch beim backholefrodo ist es so.
Channelinfo von unten nach oben, bleibt beim ausfaden sogar hängen bei meinem Skin.

Hast Du ne Idee warum das so ist ?

Vielen Dank

Gruß
moz

maz · 13. Februar 2016

Ist bei mir genauso, beim shifting der channelinfo, und im Main Menu. Mich wundert es auch dass es mit der Hardwarebeschleunigung mit blackholefrodo oder bluestyle Skin noch so "unsanft" läuft.

marco · 13. Februar 2016

Zitat von louis

marco: das kam jetzt aber nicht erst mit er 0.8.4 oder?

Offensichtlich ist bei dieser gcc Version der cast auf int für enums nicht mehr erlaubt...das ist unschön.

Ciao Louis

Nein, die Meldung kam auch bereits in der 0.8.0, habs gerade nochmals mit den verschiedenen Versionen getestet.

vdr-plugin-skindesigner-0.6.3:
vdr-plugin-skindesigner-0.7.2:

Code

/usr/include/c++/6.0.0/bits/stl_deque.h:577:8: error: call of overloaded 'swap(std::_Deque_base<_xmlNode*, std::allocator<_xmlNode*> >::iterator&, std::_Deque_base<_xmlNode*, std::allocator<_xmlNode*> >::iterator&)' is ambiguous
    swap(this->_M_start, __x._M_start);
    ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vdr-plugin-skindesigner-0.8.0:
vdr-plugin-skindesigner-0.8.4:

Code

coreengine/definitions.h:66:30: error: could not convert 'count' from 'eVeDisplayMenu' to 'int'
     timers = eVeDisplayMenu::count,
                              ^~~~~

Frodo · 13. Februar 2016

Zitat von louis

Frodo: Bezüglich des Problems, dass das TV Bild beim Schließen mit "menu" (ich habe das auch so eingestellt) nicht auf Vollbild skaliert wird: ich konnte das bei mir nicht so wirklich nachstellen. Auch mit dem blackholefrodo Skin nicht. Da war es genau ein mal so, ansonsten wurde das Bild immer brav skaliert...das ist irgendwie ein bisschen schräg. Ich habe jetzt mal auf Verdacht was geändert, ich hoffe das bringt was Ansonsten müsstest du mir vielleicht mal deine genauen Setup Werte von deinem benutzen Skin geben, ggf. müsste ich noch was einstellen.

Danke für die neue Version, der Bug ist verschwunden. Dein Verdacht war genau richtig

3PO · 13. Februar 2016

Zitat von marco

[...] mit dem neuen gcc6 Kompiler bekomme ich die folgende Fehlermeldung: ...

Wenn man halt meint, dass man immer unbedingt jede pre-alpha installieren muss, dann muss man halt auch damit rechnen, dass manche Dinge vor den Baum gehen.

marco · 13. Februar 2016

Zitat von 3PO

Wenn man halt meint, dass man immer unbedingt jede pre-alpha installieren muss, dann muss man halt auch damit rechnen, dass manche Dinge vor den Baum gehen.

Das hat hier nichts mit Einsatz von alpha Versionen zu tun. Ich bin Fedora Paket Maintainer für ein paar vdr Pakete die bereits für eine zukünftige Version testen, und es ist sicherlich von Vorteil im Vorfeld Fehler zu beseitigen.

3PO · 13. Februar 2016

Zitat von marco

[...] Ich bin Fedora Paket Maintainer ...

Nun, ich gehe doch mal davon aus, dass der "Maintainer" in der Lage ist, das Problem entsprechend zu fixen?

maz · 13. Februar 2016

Zitat von Frodo

Danke für die neue Version, der Bug ist verschwunden. Dein Verdacht war genau richtig

Kann ich bestätigen, klappt jetzt

marco · 13. Februar 2016

Zitat von 3PO

Nun, ich gehe doch mal davon aus, dass der "Maintainer" in der Lage ist, das Problem entsprechend zu fixen?

Nein, muss er nicht ! Ich erstelle ja nur die rpm Pakete entsprechend.

Skindesigner 0.8.8

Jetzt mitmachen!

Teilen