Dokumentation von virtual void cStatus::OsdCurrentItem(const char *Text)

Quote from MarkusE

if (Key != kNone) DisplayCurrent(true);

Scheint zu funktionieren.

ProcessKey() wird einmal pro Sekunde aufgerufen, auch wenn keine Taste gedrückt wurde, um dem OSD die Möglichkeit zu geben, dynamisch zu sein (z.B. Anzeige der Uhrzeit). In dem Fall wird dann kNone übergeben.

Wir wollen jetzt besser nicht in operative Hektik verfallen ;-).
Die eOSState Werte dienen dazu, nach "oben" zu melden, wie es weitergehen soll, nicht, ob sich das Item verändert hat. Hier auf kNone abzufragen erscheint mir sinnvoll und funktioniert in meinem Test auch. Weitergehende Änderungen möchte ich an der Stelle aber nicht machen.

Danke, das wäre auch mein nächster Schritt gewesen ;-).

Anbei zum Abgleich der komplette Stand der Änderungen bzgl. Statusmeldungen.
Bei den Aufrufen von MsgOsdCurrentItem() habe ich anstatt Item->Index() den jeweils an dieser Stelle bereits vorhandenen Wert genommen, da Index() die Liste durchläuft und "zählt", was hier aber unnötig ist, da der Wert bereits bekannt ist.

Bei den Untersuchungen zum Thema cStatus ist mir noch aufgefallen, dass beim Scrollen jedes Mal für alle Items OsdItem2() aufgerufen wird, was an dieser Stelle völlig unnötig ist. Beiliegender Patch (nach vdr-2.7.3-status-01.diff.txt anzuwenden) behebt das.
MarkusE Magst du dir das bitte auch anschauen? Ich versuche jetzt noch herauszubekommen, warum das Menü bei ersten Mal manchmal zweimal aufgebaut wird (habe ich erst durch die doppelten Aufrufe von OsdItem2() gesehen). Das ist auch unnötig und vermutlich auch ohne Verwendung von cStatus schlecht für die Performance.

MarkusE Danke fürs Testen. Den Link hab ich auf dem Schirm.

Auf Anhieb kann ich das nicht nachvollziehen.
Kannst du mir eine channels.conf schicken, mit der das auftritt?

Ist das ein neuer Fehler (durch vdr-2.7.3-status-02.diff.txt verursacht), oder tritt das auch mit Version 2.7.3 auf?

<edit>
Kann es sein, dass du das nicht im OSD beobachtest, sondern nur über die cStatus-Meldungen?
Da fehlt andscheinend ein Neuaufbau über OsdItem2(). Schau ich mir an.
</edit>

Das war ja ein saudummer Fehler von mir

Ich hatte ursprünglich "inhibitStatus" geschrieben, wollte dann aber bei den Abfragen keine doppelte Verneinung haben und habe es in "conveyStatus" umbenannt - und (ganz gemäß Murphy) übersehen, dass das natürlich auch in DisplayNoStatus() entsprechend geändert werden muss.
Richtig muss es also heißen:

void cOsdMenu::DisplayNoStatus(void)
{
  conveyStatus = false;
  Display();
  conveyStatus = true;
}

Das ist seltsam: Ich hatte in #32 geschrieben "...Verneinung haben und habe...", aber es erscheint nur "...Verneinung habe...". Wenn ich den Post editiere, steht "...Verneinung haben und habe..." aber drin. Lösche ich "haben" raus dann erscheint "...Verneinung und habe...".

Eigenwilliger Editor...

Mir ist noch aufgefallen, dass teilweise Menüs mehrfach hintereinander Display() machen, obwohl dies nicht nötig ist (kann man mit dem status-Plugin gut beobachten). Beliegender Patch sollte das fixen.

MarkusE Magst du dir das bitte auch noch anschauen? Nicht dass ich da was kaputtgemacht habe.

Eines hab ich inzwischen selber gefunden: das 'Force' bei cMenuMain::Update() ist doch notwendig, weil sonst die Button-Texte beim Öffnen nicht gesetzt werden.

Anbei der korrigierte Patch.