class cStatus->OsdStatusMessage(const char *Message) wird manchmal nicht gerufen

MarkusE · January 18, 2025 at 7:31 PM New

Hi kls ,

class cStatus->OsdStatusMessage(const char *Message) wird manchmal nicht gerufen.

Z.b. wenn ich im Kanal Menü einen Kanal markiere, wird

Code: osdbase.c

void cOsdMenu::Mark(void)                                                                                                                                     
{                                                                                                                                                             
  if (Count() && marked < 0) {                                                                                                                                
     marked = current;                                                                                                                                        
     SetStatus(tr("Up/Dn for new location - OK to move"));                                                                                                    
     }                                                                                                                                                        
}

ausgeführt. SetStatus ruft direkt displayMenu->SetMessage(mtStatus, s); . Damit wird cStatus::MsgOsdStatusMessage(s); nicht gerufen.

Was wäre hier eine geeignete Lösung?

Wir könnten in SetStatus direkt cStatus::MsgOsdStatusMessage(s); aufrufen.

Alternativ könnten wir auch in SetStatus cSkins::Message(...) aufrufen, anstelle von displayMenu->SetMessage(mtStatus, s); .

Was meinst Du?

~ Markus

MarkusE · January 18, 2025 at 7:44 PM New

Ergänzung:

Wir sollten auch eine weitere Methode

Code

class cStatus {
...  
  virtual void OsdStatusMessage2(eMessageType Type, const char *Message) {}
...
}

einführen.

MarkusE · January 21, 2025 at 1:41 PM New

Hi,

Anbei ein Patch, der die hier genannten Punkte umsetzt.

~ Markus

kls · February 10, 2025 at 10:34 PM New

Ich habe den Patch mit folgenden Änderungen übernommen:
1.) In den Interfaces erst Type, dann Message (passt besser zu entsprechenden anderen Stellen).
2.) Kein Defaultwert für Type in MsgOsdStatusMessage() (wegen 1.), daher eine zusätzliche Funktion und die bisherige "deprecated".

MarkusE Passt das so für dich?

MarkusE · February 11, 2025 at 7:42 AM New

Passt

Participate now!