Ändern eines Menüeintrages - Generelles Plugin Problem?

Saxman2k · 30. November 2003

Hallo zusammen!
Ich wollte eigentlich mal das lcdproc Plugin überarbeiten, weil da so manche Sachen nicht korrekt angezeigt werden. Das Plugin ist halt schon etwas in die Monate (Jahre?) gekommen.
Ein Anzeigeproblem tritt auf, wenn man einen Menüparameter ändert (z. B. Daten eines Timers). Die Anzeige des LCDs wird dabei nicht aktualisiert. Also hab ich flugs mal in dem control Plugin nachgeschaut, aber hier ist das selbe Problem. Das graphlcd Plugin kann ich leider mangels Hardware nicht testen, aber es würde mich wunder, wenn das da klappt.
Irgendwie scheint da ein Aufruf von VDR selber zu fehlen. Ich verwende derzeit VDR 1.2.6 mit dem Komplettpatch. Mit "vanilla" hab ich das noch nicht getestet.
Kann jemand die Beobachtung bestätigen? Wie sieht es mit den Entwicklern des graphlcd Plugins aus? Habt Ihr das auch bemerkt?
Bin gerade etwas verwirrt.

anbr · 30. November 2003

Hi,

ich glaube unter graphlcd-0.0.7/patches gibt es schon etwas passende dazu.
Sprich damit Änderungen im Menu auch im Display auftauchen.

vdr-1.2.1-setup_menu_patch.diff

Diff

--- vdr-1.2.1-org/menuitems.c   Sat Apr 12 11:21:33 2003
+++ vdr-1.2.1/menuitems.c       Thu Jun 12 20:58:27 2003
@@ -31,12 +31,18 @@
 
 void cMenuEditItem::SetValue(const char *Value)
 {
-  free(value);
-  value = strdup(Value);
-  char *buffer = NULL;
-  asprintf(&buffer, "%s:\t%s", name, value);
-  SetText(buffer, false);
-  Display();
+  int changed = 0;
+
+  if((value == NULL) ||
+     (Value && value && (changed = strcmp(Value, value)))) {
+    free(value);
+    value = strdup(Value);
+    char *buffer = NULL;
+    asprintf(&buffer, "%s:\t%s", name, value);
+    SetText(buffer, false);
+    if(changed) cStatus::MsgOsdCurrentItem(buffer);
+    Display();
+  }
 }
 
 // --- cMenuEditIntItem ------------------------------------------------------

Alles anzeigen

Andreas

Tobias · 30. November 2003

genau das Phänomen habe ich auch schon bemerkt und bemängelt
Graphlcd 0.0.7
Das ein Patch existiert hatte ich wohl übersehen. Eigentlich Könnte der ja auch mit in den KomplettPatch mit rein. Wer halt kein GLCD dran hat merkts halt nicht ...

tobias

Saxman2k · 1. Dezember 2003

Ihr seid spitze!
Das werde ich dann heute Abend gleich mal ausprobieren. Müßte ja eigentlich mit allen Plugins funktionieren, die sich von cStatus ableiten.

Saxman2k · 2. Dezember 2003

Hat alles soweit funktioniert!
Jetzt frage ich mich aber, warum das im VDR Code vergessen wurde? Das ist doch eine essentielle Funktion für alle cStatus Klassen. Weiß Klaus davon? Sonst würde ich ihn mal kontaktieren.

Ändern eines Menüeintrages - Generelles Plugin Problem?

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema