[softhddevice] OSD Demo flackert teilweise stark

Es gibt einen Eintrag im Bug Tracker: http://projects.vdr-developer.org/issues/1468

Wird wohl am Clear für die Untertitel liegen, ohne diesen verschwinden aber die Untertitel nicht mehr.

Johns

diff --git a/softhddevice.cpp b/softhddevice.cpp
index 126cd2d..f649e6c 100644
--- a/softhddevice.cpp
+++ b/softhddevice.cpp
@@ -399,7 +399,7 @@ void cSoftOsd::Flush(void)
     //
     // VDR draws subtitle without clearing the old
     //
-    if (OsdLevel >= OSD_LEVEL_SUBTITLES) {
+    if (0 && OsdLevel >= OSD_LEVEL_SUBTITLES) {
        VideoOsdClear();
        cSoftOsd::Dirty = 1;
 #ifdef OSD_DEBUG

Ansonsten mal OSD_DEBUG aktivieren und gucken was so falsch läuft.

Johns

Moin,

johns: sehe ich das richtig, dass du bei einem OSD Flush wegen den Subtitles immer zuerst das komplette OSD löscht?

Copperhead: wenn du eh gerade am testen bist, könntest du auch mal testen, wie sich nOpacity / tvguide mit diesem Patch verhalten? Ggf. läuft das ganze dann etwas flüssiger, z.B. beim scrollen im EPG oder so?!

Wenn dem so wäre, würde ich vorschlagen, im Setup vom softhddevice einen Parameter dafür einzuführen und das damit konfigurierbar zu machen...Leute, die keine Untertitel verwenden (wie ich z.B. und wohl die meisten anderen auch?!) könnten das dann bequem abschalten.

Ciao Louis

Warum nicht einfach das eigentliche Problem beheben?

--- ./PLUGINS/src/osddemo/osddemo.c     2014/01/05 10:56:18     3.1
+++ ./PLUGINS/src/osddemo/osddemo.c     2014/02/04 10:35:13
@@ -119,7 +119,7 @@


 void cLineGame::Show(void)
 {
-  osd = cOsdProvider::NewOsd(cOsd::OsdLeft(), cOsd::OsdTop(), 50);
+  osd = cOsdProvider::NewOsd(cOsd::OsdLeft(), cOsd::OsdTop());
   if (osd) {
      int x1 = cOsd::OsdWidth() - 1;
      int y1 = cOsd::OsdHeight() - 1;
@@ -496,7 +496,7 @@


 void cTrueColorDemo::Show(void)
 {
-  osd = cOsdProvider::NewOsd(cOsd::OsdLeft(), cOsd::OsdTop(), 50);
+  osd = cOsdProvider::NewOsd(cOsd::OsdLeft(), cOsd::OsdTop());
   if (osd) {
      if (SetArea()) {
         osd->DrawRectangle(0, 0, osd->Width() - 1, osd->Height() - 1, clrGray50);

Klaus

Ah ok, jetzt hab ichs verstanden Mein obiges Post ist damit hinfällig...

Copperhead: den Test für nOpacity und tvguide kannst du dir sparen, da wird sich nix ändern...

Ciao Louis

Zitat von kls

Warum nicht einfach das eigentliche Problem beheben?

Wenn man weiss, daß dies nicht richtig ist, dann ist es einfach.
Da dies aber die VDR OSD Demo ist, ist man davon ausgegangen, daß sie als Referenz gilt.

Und mein Code ist ja mehr und weniger ein Hack.

Problem war, daß die Untertitebereiche nicht gelöscht werden, wenn der Nächste kommt,
bei den Beschreibungen und Debuggen ist mir nichts aufgefallen wie man dies erkennt.

louis

Dies passiert nur für Untertitelebene bzw. alle Ebenen die über den Untertitel liegen,
wobei man es vielleicht nur auf die Untertitelebene begrenzen könnte.

Johns

Zitat von johns

Wenn man weiss, daß dies nicht richtig ist, dann ist es einfach.
Da dies aber die VDR OSD Demo ist, ist man davon ausgegangen, daß sie als Referenz gilt.

Fehler können überall passieren ;-).

Zitat

Und mein Code ist ja mehr und weniger ein Hack.

Problem war, daß die Untertitebereiche nicht gelöscht werden, wenn der Nächste kommt,
bei den Beschreibungen und Debuggen ist mir nichts aufgefallen wie man dies erkennt.

Kannst du das etwas genauer beschreiben, wo das Problem liegt?
Bei mir hier werden Untertitel einwandfrei angezeigt (TT S2-6400).
Wie sieht denn die SetAreas()-Funktion des verwendeten OSD-Devices aus? Löscht die die alte OSD-Anzeige?

Klaus

Zitat von johns

louis

Dies passiert nur für Untertitelebene bzw. alle Ebenen die über den Untertitel liegen,
wobei man es vielleicht nur auf die Untertitelebene begrenzen könnte.

Jo, anhand des Patches von Klaus habe ich gesehen, wie "OsdLevel >= OSD_LEVEL_SUBTITLES" zu deuten ist. Ich hole mir das OSD immer mit dem Default OSD Level, und der ist ja 0. Für SUbtitles ist der 10...deshalb bin ich da nicht betroffen.

Ciao Louis

Der Effekt ist ganz einfach, wenn ein neuer Untertitel angezeigt werden soll, dann überschreibt er nur den alten. Es erfolgt kein Löschen,
Ich gehe mal von TrueColor OSD aus, da dies normalerweise immer verwendet wird.

Also ich verwende keine eigene cOsd::SetAreas.
Ich habe nur cOsd::Flush() und cOsd::SetActive().

Irgendwie sehe ich nichts, wo steht, daß SetAreas das OSD löschen soll.

./PLUGINS/src/dvbhddevice/HISTORY:- Fixed cHdffOsd::SetAreas() (didn't clear the OSD).

Hatte wohl das gleiche Problem.

Johns

Zitat von johns

Der Effekt ist ganz einfach, wenn ein neuer Untertitel angezeigt werden soll, dann überschreibt er nur den alten. Es erfolgt kein Löschen,
Ich gehe mal von TrueColor OSD aus, da dies normalerweise immer verwendet wird.

Also ich verwende keine eigene cOsd::SetAreas.
Ich habe nur cOsd::Flush() und cOsd::SetActive().

Irgendwie sehe ich nichts, wo steht, daß SetAreas das OSD löschen soll.

Ja, so direkt steht das da nicht. Werde ich ergänzen.

Zitat

Code

./PLUGINS/src/dvbhddevice/HISTORY:- Fixed cHdffOsd::SetAreas() (didn't clear the OSD).

Hatte wohl das gleiche Problem.

Genau ;-).

Implementiere einfach mal SetAreas() und lösche darin das OSD, so wie es auch in dvbdhdevice gemacht wird. Ich bin ziemlich sicher, daß dann das Problem weg ist.

Klaus

Anbei ein Patch für SoftHdDevice (GIT) was das Verhalten ändert.

eOsdError cSoftOsd::SetAreas(const tArea * areas, int n)
{
#ifdef OSD_DEBUG
    dsyslog("[softhddev]%s: %d areas \n", __FUNCTION__, n);
#endif
    // clear old OSD, when new areas are set
    if (!IsTrueColor()) {
        cBitmap *bitmap;
        int i;


        for (i = 0; (bitmap = GetBitmap(i)); i++) {
            bitmap->Clean();
        }
    }
    VideoOsdClear();


    return cOsd::SetAreas(areas, n);
}

Wäre schön wenn dies mal getestet würde, bevor es ins GIT kommt.
Johns

Zitat von Copperhead

Eine Frage zum OSD-Demo. Die ganzen Elemente sollen bis zum Schluss stehen bleiben, oder?
Das war vor dem Patch nämlich nicht so.

Ja, die sollen stehenbleiben, weil man dann mit Up, Down, Left, Right, Red, Green, 1 und 2 noch Effekte auslösen kann.

Klaus

Zitat von Copperhead

Edit: Jetzt passt da doch was nicht. Mit eingeschalteten Untertiteln wird das Hauptmenü nur noch in Bruchstücken angezeigt.

Das kommt daher, daß ich nur eine OSD Ebene kann und bisher wurden die Untertitel ignoriert, wenn das Menu offfen war.

// ignore sub-title if menu is open
    if (OsdLevel >= OSD_LEVEL_SUBTITLES && IsOpen()) {
        return;
    }

Wenn ich das noch in SetAreas einbaue, dann sollten in allen Funktionen die Untertitel gefiltert sein.

Edit: Patch.

Und ich denke die Untertitel verschwinden nun richtig! Also Zeitlich früher.

Johns

Zitat von johns

Das kommt daher, daß ich nur eine OSD Ebene kann und bisher wurden die Untertitel ignoriert, wenn das Menu offfen war.

Code

// ignore sub-title if menu is open
    if (OsdLevel >= OSD_LEVEL_SUBTITLES && IsOpen()) {
        return;
    }

Wenn ich das noch in SetAreas einbaue, dann sollten in allen Funktionen die Untertitel gefiltert sein.

Edit: Patch.

Und ich denke die Untertitel verschwinden nun richtig! Also Zeitlich früher.

Alles anzeigen

Ich frage mich, warum du dich überhaupt mit OSD-Levels befasst?!
Das wird doch komplett in osd.c (und nur dort) abgehandelt.

Klaus

Backtrace? SCNR
Du weißt ja jetzt, wie.

Lars.