Version 1.2.2 und 1.2.3

louis · 28. Oktober 2016

Moin Wolfi,

man könnte das Problem zwar im Estuary Skin fixen, das wäre aber dann irgendwie ein Hack. Man müsste dann auf das Feature verzichten, dass im estuary4vdr die erste Spalte in den Standardmenüs scrollt, falls der Text zu lange ist.

Korrekterweise sollte man das im music Plugn fixen. Ich würde darauf wetten, dass es dort für das problematische Menü einen SetCol() Aufruf gibt, der nicht ganz korrekt ist. Die meisten Skins zeigen das dann zwar toleranterweise korrekt an, aber nach meiner Implementierung (und die funktioniert zumindest bei allen Menüs, die Klaus erstellt hat) ist es ein Bug im entsprechenden Plugin. Welche Sourcen benutzt ihr denn? Ich kann bei Zeit mal schauen, was da zu patchen wäre. Sollte nur eine minimale Änderung sein...

Ciao Louis

wolfi.m · 28. Oktober 2016

Hallo Louis,

Zitat von louis

Ich kann bei Zeit mal schauen, was da zu patchen wäre. Sollte nur eine minimale Änderung sein...

hier der Link zum Sources -> http://easy-vdr.de/git/?p=trus…458b0c1be5d926330;hb=HEAD

Merci & Gruss
Wolfgang

louis · 28. Oktober 2016

Moin,

Zitat von wolfi.m

hier der Link zum Sources -> http://easy-vdr.de/git/?p=trusty-ng/.git…d926330;hb=HEAD

ok, ich hab mal geschaut...du musst die cOsdMenu() Aufrufe in der commands.c anpassen. Bei deinem Beispiel (Untermenü "Aussehen") wäre das hier. Entweder, du entfernst die 4 als zweiten Parameter ganz, oder du setzt einen vernünftigen Wert. Die "4" setzt die erste Spalte auf eine Breite von 4 Zeichen. Das musst du dann halt für alle Menüs entsprechend machen, bei denen es nicht passt.

Ciao Louis

wolfi.m · 28. Oktober 2016

Hi,

Zitat von louis

Das musst du dann halt für alle Menüs entsprechend machen, bei denen es nicht passt.

Merci, Merci passt jetzt!

Patch:

Diff

diff -Nru vdr-plugin-music-0.9.9/commands.c vdr-plugin-music-0.9.9/commands.c
--- vdr-plugin-music-0.9.9/commands.c	2013-10-27 22:51:38.000000000 +0000
+++ vdr-plugin-music-0.9.9/commands.c	2016-10-28 17:06:51.000000000 +0000
@@ -296,7 +296,7 @@
 
 
 cMP3Themes::cMP3Themes(void)
-:cOsdMenu(tr("Music: Appearance"),4)
+:cOsdMenu(tr("Music: Appearance"),32)
 {
 
   LoadSkinFiles();
@@ -462,7 +462,7 @@
 
 
 cMP3Visual::cMP3Visual(void)
-:cOsdMenu(tr("Music: Visualizations"),4)
+:cOsdMenu(tr("Music: Visualizations"),32)
 {
   Add(new cOsdItem(hk(tr("Disable visualization"))));
   LoadVisFiles();
@@ -614,7 +614,7 @@
 
 
 cMP3User::cMP3User(void)
-:cOsdMenu(tr("Music: User"),4)
+:cOsdMenu(tr("Music: User"),32)
 {
   LoadUsername();
   SetHelp();

Alles anzeigen

Gruss
Wolfgang

iNOB · 31. Oktober 2016

Läßt sich das irgendwie einrichten, dass das eingeblendete Hauptmenü nach einer einstellbaren Zeit wieder ohne weiteres Zutun des Nutzers verschwindet? Quasi so wie die Kanaleinblendung. Ich frag aus dem Grund, weil ich jetzt schon desöfteren den Fall hatte, dass bei eingeblendetem Hauptmenü die Kiste nicht mehr runterfährt.

louis · 31. Oktober 2016

Moin,

das ginge nur mit einem VDR Patch.

Ciao Louis

3PO · 31. Oktober 2016

Zitat von iNOB

Läßt sich das irgendwie einrichten, dass das eingeblendete Hauptmenü nach einer einstellbaren Zeit wieder ohne weiteres Zutun des Nutzers verschwindet? Quasi so wie die Kanaleinblendung. Ich frag aus dem Grund, weil ich jetzt schon desöfteren den Fall hatte, dass bei eingeblendetem Hauptmenü die Kiste nicht mehr runterfährt.

Wenn es nur um das Herunterfahren geht, brauchst Du doch blos, z.B. das,

Code

svdrpsend HITK Back

in Dein "Exitscript" einzubauen.

iNOB · 31. Oktober 2016

Jo... so einfach kann das sein Hätt ich Dösbaddel auch selber drauf kommen können.

thx
iNOB

kaminkehrer · 4. November 2016

Hallo,

beim starten des VDRs kommt es zu folgender Fehlermeldung:

Code

messages:Nov  4 08:36:55 vdrmb5 vdr[3241]: [3241] skindesigner: ERROR Unknown Setup Parameter BlockFlush

Ich benutze das aktuelle git von vorgestern, sprich 1.2.3.
Was macht der Parameter und warum kennt skindesigner diesen nicht?

Kann es sein, dass hier ein Memory whole vorliegt wenn er den Speicher nicht frei gibt?

Grüße
kaminkehrer

TheChief · 4. November 2016

Das heisst wohl nur, dass es den Parameter nicht mehr gibt und aus der setup.conf des VDR entfernt werden kann. Wie man da auf ein Speicherleck kommt, ist mir nicht ganz klar.

kaminkehrer · 4. November 2016

Hallo,

ich denke ja auch das es so simpel ist.

Aber der Name "BlockFlush" suggeriert bei mir das "flushen" von genutzten Speicher Blöcken.
Wenn nun der Befehl nicht mehr benutzt wird kann es sehr gut sein, dass im Code Speicher nicht frei gegeben wird.

Währe schön wenn das kurz geprüft werden würde.

Grüße
kaminkehrer

louis · 4. November 2016

Moin,

das ist ein alter Setup Parameter, den es nicht mehr gibt. Wie schon geschrieben, einfach den Wert aus der setup.conf manuell rausschmeissen.

Mit Speicher hat das nix zu tun gehabt, das "flush" bezog sich auf das "flushen" der OSD Ausgabe bzw. das blockieren dieser Ausgabe

Ciao Louis

kaminkehrer · 4. November 2016

DANKE !!!

P3f · 1. März 2017

Hallo Louis,

wir sind dabei nun auch die MLD für die VDR 2.3.2 vorzubereiten, damit darf natürlich auch der Skindesigner nicht fehlen.

Nur leider ist es so, das beim Skin Estuary4vdr ja eigentlich die Funktionen mit den Rechts/Links Tasten die jeweiligen Kanalübersicht einfliegen lässt. Für die VDR <=2.2.0 war ja ein Patch notwendig, ist aber obsolete für die VDR 2.3.2 geworden.

Diese Funktion ist aktuell bei uns nicht möglich, es kommt auch keine Fehlermeldung im /var/log/messages. Wie kann man das nun analysieren?
Es ist das aktuellste GIT checkout vom Skindesigner genommen worden.

Hast Du eine Idee zur Analyse?

Danke,

Pit

P.S: Im Anhang ist die fehlerhafte (nicht erwartete) OSD-Anzeige...

louis · 1. März 2017

Moin,

Zitat von P3F

Für die VDR <=2.2.0 war ja ein Patch notwendig, ist aber obsolete für die VDR 2.3.2 geworden.

der "Zapcockpit Patch" ist auch beim VDR 2.3.2 nocht notwendig...ich habe nicht gesehen, dass Klaus ihn übernommen hätte. Ob der Patch allerdings gegen VDR 2.3.x baut weiß ich nicht. Falls nein sollten die Anpassungen aber überschaubar sein. Müsste sich mal der "Herr Jemand" anschauen

Ciao Louis

P3f · 1. März 2017

Hallo Louis,

vielen Dank für den Hinweis. Leider lässt er sich wirklich nicht 1:1 übernehmen und ich kapiere es irgendwie nicht mit der notwendigen Anpassungen für die "channels"-Klassen.

folgende Fehler kommen beim compile:

Code

menu.c: In constructor ‘cDisplayChannel::cDisplayChannel(int, bool)’:
menu.c:4538:34: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
   channel = Channels->GetByNumber(Number);
             ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
menu.c: In constructor ‘cDisplayChannel::cDisplayChannel(eKeys, bool)’:
menu.c:4562:34: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
   channel = Channels->GetByNumber(cDevice::CurrentChannel());
             ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
menu.c: In member function ‘virtual eOSState cDisplayChannel::ProcessKey(eKeys)’:
menu.c:4643:44: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channels->GetByNumber(number);
                       ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
menu.c:4694:36: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channels->Get(group);
                       ~~~~~~~~~~~~~^~~~~~~
menu.c:4713:23: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channel;
                       ^~~~~~~
menu.c:4734:44: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channels->GetByNumber(number);
                       ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
menu.c:4749:36: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channels->Get(Channels->GetNextNormal(group));
                       ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
menu.c:4757:44: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
             channel = Channels->GetByNumber(number);
                       ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
menu.c:4779:40: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
         channel = Channels->GetByNumber(cDevice::CurrentChannel());
                   ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
menu.c:4788:19: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
         channel = NewChannel;
                   ^~~~~~~~~~
menu.c: In member function ‘void cDisplayChannelExtended::StateNumberKey(int, cSkinDisplayChannelExtended*)’:
menu.c:4908:34: error: ‘Channels’ was not declared in this scope
   int selectedChannel = number > Channels.MaxNumber() ? key - k0 : number * 10 + key - k0;
                                  ^~~~~~~~
menu.c: In member function ‘void cDisplayChannelExtended::SetChannelList()’:
menu.c:5296:22: error: ‘Channels’ was not declared in this scope
   for (cChannel *c = Channels.First(); c; c = Channels.Next(c)) {
                      ^~~~~~~~
menu.c: In member function ‘void cDisplayChannelExtended::SetGroupList()’:
menu.c:5341:25: error: ‘Channels’ was not declared in this scope
      for (cChannel *c = Channels.First(); c; c = Channels.Next(c)) {
                         ^~~~~~~~
menu.c:5359:22: error: ‘Channels’ was not declared in this scope
   for (cChannel *c = Channels.First(); c; c = Channels.Next(c)) {
                      ^~~~~~~~
menu.c: In member function ‘int cDisplayChannelExtended::GetIndexGroup(cChannel*)’:
menu.c:5379:34: error: ‘Channels’ was not declared in this scope
   for (cChannel *c = cur; c; c = Channels.Prev(c)) {
                                  ^~~~~~~~
menu.c: In member function ‘void cDisplayChannelExtended::SetGroupChannelList(cSkinDisplayChannelExtended*)’:
menu.c:5420:74: error: ‘Channels’ was not declared in this scope
   for (cChannel *c = dynamic_cast<cChannel*>(curChannel->Next()); c; c = Channels.Next(c)) {
                                                                          ^~~~~~~~
menu.c: In member function ‘bool cDisplayChannelExtended::SwitchChannel()’:
menu.c:5562:40: error: ‘Channels’ was not declared in this scope
      for (cChannel *c = cGroup; c; c = Channels.Next(c))
                                        ^~~~~~~~
menu.c:5571:3: error: ‘Channels’ was not declared in this scope
   Channels.SwitchTo(newChannel->Number());
   ^~~~~~~~
menu.c: In member function ‘cChannel* cDisplayChannelExtended::LastChannelSep()’:
menu.c:5579:22: error: ‘Channels’ was not declared in this scope
   for (cChannel *c = Channels.Last(); c; c = Channels.Prev(c))
                      ^~~~~~~~

Alles anzeigen

Weißt Du warum Klaus nur den einen Teil vom Patch in den VDR-Core einfließen lassen hat?

Hat schon jemand den Skindesigner für den VDR 2.3.2 gebaut und vollumfänglich im Einsatz. Vielleicht ist ja auch nur vergessen worden den überarbeiten Patch Zapcockpit bereit zustellen.

Gruß,

Pit

louis · 1. März 2017

Hi,

wie man den Patch auf VDR 2.3.x anpasst, kannst du aus den vielen bereits verfügbaren Beispielen entnehmen. Im Prinzip hat sich nur die Art geändert, wie man auf die globalen Strukturen zugreift. "Früher" konnte man einfach direkt auf Channels zugreifen, ab VDR2.3.1 muss man das über ein Macro machen, das ein entsprechendes Lock setzt.

Zitat von P3F

Weißt Du warum Klaus nur den einen Teil vom Patch in den VDR-Core einfließen lassen hat?

Ich glaube du verwechselst da was. Es gibt zwei Patches: der erste erlaubt, Menüs horizontal auszurichten und dann auch in den Menüs mit der Fernbedienung passend zu navigieren. Dieser Patch ist in VDR 2.3.1 aufgenommen worden. Der Zapcockpit Patch ist notwendig, um das neue "Kanalgruppenhandling" in DisplayChannel (also beim Umschalten) über den Skindesigner zu ermöglichen. Zu diesem Patch hat sich Klaus nie geäußert, ich denke, das ist ihm viel zu viel Bling Bling und viel zu umständlich, ich glaube nicht, dass der jemals im VDR landen wird.

Ciao Louis

P3f · 1. März 2017

Hallo Louis,

ok, ich kann nachvollziehen warum der Bling,Bling-Patch nicht mit in den VDR Core wandern wird. Schade, aber wenn es nun mal so ist...

Ich habe mir mal die Stellen in der menu.c angeschaut. Da sieht es aber doch schon so aus, als wenn die "neue" Struktur einzig gefunden hat.

Auszug aus der menu.c (Zeile 4538

Code

void cDisplayChannel::DisplayInfo(void)
{
  if (withInfo && channel) {
     LOCK_SCHEDULES_READ;
     if (const cSchedule *Schedule = Schedules->GetSchedule(channel)) {
        const cEvent *Present = Schedule->GetPresentEvent();
        const cEvent *Following = Schedule->GetFollowingEvent();
        if (Present != lastPresent || Following != lastFollowing) {

Compilerfehler lautet:

Code

menu.c: In constructor ‘cDisplayChannel::cDisplayChannel(int, bool)’:
menu.c:4538:34: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
   channel = Channels->GetByNumber(Number);
             ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~

Oder könntest Du anhand des Beispiel mir auf den richtigen Weg helfen?

Danke,

Pit

louis · 2. März 2017

Moin P3F,

also irgendwie passt der von dir gepostete Fehler nicht zu dem Codeauszug

Schau dir doch am besten mal die Änderungen für die VDR2.3.1 Kompatibilitätfür den Skindesigner selbst an. In den Änderungen der Datei coreengine/viewdetail.c (das kann man irgendwie nicht verlinken, ca. 1/3 nach unten scrollen) siehst du z.B. die notwendigen Maßnahmen, um auf die Channels zugreifen zu können.

Ciao Louis

P3f · 2. März 2017

Ciao Louis,

ich will ja deine Geduld nicht unnötig strapazieren, aber irgendwie finde ich es nicht. habe nun mal im Anhang die gepaschten menu.c/.h Dateien gehängt. Aber ich finde einfach nicht die Stelle wo die Änderungen einfließen müssen. Das von Dir verlinkte Beispiel ist sicherlich erklärbar, aber ich finde die Stelle nicht wo ich das anpassen muß.

Darf ich doch noch um die Unterstützung bitten? Ich denke es ist auch nicht nur ein einzelnes Interesse, schließlich ist das Plugin Skindesigner mittlerweile zu einem "must-have" geworden (inkl. Bling-Bling).

Was ich gefunden habe, und ändern würde wäre die Zeile 4912 (menu.c)

Code

cChannel *candidatesStart = Channels.GetByNumber(candidateStartNumber);

in

Code

cChannel *candidatesStart = channels->GetByNumber(candidateStartNumber);

==> Dann kommen aber andere Comilierfehler....

Wo wird das denn mit Compilerfehler (als Beispiel):

Code

menu.c: In constructor ‘cDisplayChannel::cDisplayChannel(int, bool)’:
menu.c:4538:34: error: invalid conversion from ‘const cChannel*’ to ‘cChannel*’ [-fpermissive]
   channel = Channels->GetByNumber(Number);
             ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~

gefixt?

Danke für deine Geduld.

Pit

Version 1.2.2 und 1.2.3

Jetzt mitmachen!

Teilen