[SkinNopacity] Aktuelle Probleme

kamel5 · 1. März 2021

Zitat von S:oren

Was funktioniert da nicht richtig? Mir ist bisher nichts aufgefallen...

Das fällt nur bei bestimmten Themen auf. Das Problem war hier, das die Grafiken für den imagecache bei der Initialisierung erzeugt wurden. Zu dieser Zeit war allerdings dem VDR der Skin und damit auch das Theme noch nicht bekannt. Das war auch der Grund für die ursprüngliche Implementierung mit dem CreateCacheDelayed in displaychannel.

Die letzten beiden Commits in meinem Devel-Branch sollten das beheben.

Den Devel-Branch bitte neu herunterladen, ich habe da nach Deinen Hinweisen noch ein wenig geändert.

Zitat von S:oren

Hab ich am Wochenende mal gefixt,

Mal schauen, ob ich das für master so übernehmen kann.

Grüße

kamel5

kamel5 · 1. März 2021

Zitat von S:oren

Was funktioniert da nicht richtig? Mir ist bisher nichts aufgefallen...

Ich habe mal einen Branch Devel-Light angelegt. Da sind die dazu gehörenden Änderungen und noch einige Andere, die auch für die Light-Version passen sollten, enthalten.

Grüße

kamel5

S:oren · 1. März 2021

Danke erstmal fuer Devel-Light.

Leider sind die Menues damit alle kaputt, Darstellung einzelner Items außerhalb des sichtbaren Bereichs. Liegt vielleicht daran, dass ich Breite und Hoehe des OSD um ein paar Prozent verkleinert habe.

Soll ich ein bisect starten?

Gruss,

S:oren

S:oren · 1. März 2021

Code

root@rockpro64:/data/vdr-2.5.1/PLUGINS/src/skin-nopacity# git bisect good a051c940c5812aab8dd968adfe3f1e6af08db66b
Bisecting: 2 revisions left to test after this (roughly 1 step)
[f7218d04ba871dd6b9d9f1bd3177e7d09fb15ab0] Make imgCache global
root@rockpro64:/data/vdr-2.5.1/PLUGINS/src/skin-nopacity# git bisect good
Bisecting: 0 revisions left to test after this (roughly 1 step)
[c0268e72319902e41f1791d18d4734ea89dfeb75] Fix imgCache->CreateCacheDelayed()
root@rockpro64:/data/vdr-2.5.1/PLUGINS/src/skin-nopacity# git bisect skip
Bisecting: 1 revision left to test after this (roughly 1 step)
[3f91a8449498d3980b9d0e0a2706c1cfca609bdb] Eliminate cGeometryManager::SetOSDSize
root@rockpro64:/data/vdr-2.5.1/PLUGINS/src/skin-nopacity# git bisect skip
There are only 'skip'ped commits left to test.
The first bad commit could be any of:
3f91a8449498d3980b9d0e0a2706c1cfca609bdb
c0268e72319902e41f1791d18d4734ea89dfeb75
54315f88c5ce57d349b92136b99a83016564b26c
We cannot bisect more!

Alles anzeigen

Leider lassen sich einzelne Commits nicht uebersetzen. Sowas ist extrem schlecht und verhindert ein sinnvolles git-bisect...

Gruss,

S:oren

kamel5 · 1. März 2021

Mist...

Eine Zeile vergessen und schon funktioniert das nicht richtig.

Ich muss wirklich alles doppelt testen.

Ich habe den Devel-Light upgedated, jetzt sollte es übersetzen.

Zitat von S:oren

Soll ich ein bisect starten?

Das wäre nett.

Grüße

kamel5

S:oren · 1. März 2021

Zitat von kamel5

Das wäre nett.

Protokoll steht doch direkt drueber...

Edit: Ach so, mit der neuen Version...

S:oren · 1. März 2021

6fc9c40bd686348b6761a3c97f6f61ac6a9d59be is the first bad commit

fehlende Hintergrundgrafiken

81e016a561f21e4e051a904729d7a46c27536589 is the first bad commit
wieder mit Hintergrund, aber falsche Geometrie

Ergibt das ueberhaupt einen Sinn, dieses in 2 Patches zu teilen? Ich wuerde die beiden squashen. Aber der Bug bleibt dann natuerlich trotzdem.

Ich kann nochmal in den Code reinschauen, aber nicht mehr heute...

Gruss,

S:oren

kamel5 · 1. März 2021

Zitat von S:oren

Ich kann nochmal in den Code reinschauen, aber nicht mehr heute...

Ich habe gerade nochmal getestet und denke, ich habe den Fehler schon gefunden (Menü wird links oben dargestellt).

Ich ändere das morgen Nachmittag und fasse die beiden anderen Patches zusammen.

Grüße

kamel5

S:oren · 2. März 2021

Zitat von S:oren

Ich wuerde die beiden squashen.

Zitat von kamel5

und fasse die beiden anderen Patches zusammen.

War wohl gestern schon ein bisschen spaet. Anscheinend sind die Patches schon zusammengefasst, die sich mit CreateCacheDelayed(); befassen. Die beiden letzten sind schon komplett verschieden und sollten getrennt bleiben.

Entschuldigung fuer den voreiligen Kommentar...

S:oren

kamel5 · 2. März 2021

S:oren ,

der Fehler müsste jetzt in Devel-Light im letzten Commit behoben sein.

Da hatte ich etwas zu kurz gedacht mit der letzten Anpassung, osdLeft und osdTop muss bei jeder Geometrieänderung natürlich auch geprüft und gesetzt werden.

Grüße

kamel5

S:oren · 2. März 2021

Die beiden Fehler in den letzten beiden Commits sehe ich unveraendert.

Gruss,

S:oren

kamel5 · 2. März 2021

Mhhh, was ist bei Dir anders als bei mir.

Zumindest mit dem letzten Commit vom Devel-Light konnte ich bei meinen Tests keine Anzeigeprobleme feststellen.

Kannst Du vielleicht mal eine Bildschirmkopie machen, damit ich eine Vorstellung habe, was da nicht stimmt.

Ist das eventuell Theme-abhängig.

Grüße

kamel5

S:oren · 2. März 2021

Zum vorletzten Commit:

Wo soll denn cNopacity::Show(void) ueberhaupt aufgerufen werden? Das ist doch gar keine geerbte Methode, oder was habe ich da uebersehen?

Gruss,

S:oren

kamel5 · 2. März 2021

Show(void) wird in osdbase.h definiert.

Grüße

kamel5

S:oren · 2. März 2021

Aber cNopacity ist doch ein cSkin und kein cOsdObject!?

Gruss,

S:oren

kamel5 · 2. März 2021

Die genauen Zusammenhänge habe ich da auch noch nicht erkannt.

Ich hatte da eine Debugausgabe eingebaut und jedesmal wenn ich das Menü oder eine andere Anzeige aufgerufen habe, wurde diese Funktion aufgerufen.

Was ich auch nicht verstehe, das es bei mir funktioniert und bei Dir nicht.

Ich würde mal den vorletzten Commit ausklammern. Ich probiere das morgen noch mal. Und mit dem letzten Commit funktioniert das ja.

Warum ist damit aber die Geometrie noch falsch?

Grüße

kamel5

S:oren · 2. März 2021

Zitat von kamel5

Die genauen Zusammenhänge habe ich da auch noch nicht erkannt.

Dann wuerde ich vorschlagen, diese beiden Commits wegzulassen. Irgendwas auszuprobieren, ohne zu verstehen was man tut, kann zu keinem stabilen Code fuehren.

Und wenn die Themes jetzt kaputt sind, waere ein Revert der entspechenden alten Aenderung wohl am sinnvollsten...

Gruss,

S:oren

kamel5 · 2. März 2021

Zitat von S:oren

Aber cNopacity ist doch ein cSkin und kein cOsdObject!?

Du hast recht, das kann nicht funktionieren. Ich weiß jetzt auch warum ich das gedacht habe, ich hatte die Debugausgabe an einer anderen Stelle eingebaut.

Den vorletzten Commit streiche ich komplett und das Start() nehme ich auch wieder raus.

Ich passe das im Anschluss mal an.

Grüße

kamel5

kamel5 · 2. März 2021

Zitat von S:oren

Dann wuerde ich vorschlagen, diese beiden Commits wegzulassen.

Den vorletzten Commit habe ich rausgenommen. Der letzte ist aber OK so.

Ich habe auch den Fehler mit der Geometrie gefunden, der war schon im commit "Eliminate cGeometryManager::SetOSDSize".

Bitte nochmal testen, jetzt müsste alles passen.

Zitat von S:oren

Irgendwas auszuprobieren, ohne zu verstehen was man tut, kann zu keinem stabilen Code fuehren.

Das ist wohl war. Ich weiß jetzt auch wie ich darauf gekommen bin. Parallel mache ich noch was an einem Plugin und da ist das so gelöst...

Danke für die Hilfe.

Grüße

kamel5

S:oren · 2. März 2021

Zitat von kamel5

Ich habe auch den Fehler mit der Geometrie gefunden, der war schon im commit "Eliminate cGeometryManager::SetOSDSize".

Dieser Commit ergibt jetzt ueberhaupt keinen Sinn mehr. Statt des zweimaligen Aufrufs von SetOSDSize(); wird der Code zweimal hineinkopiert. Auch die Commit-Beschreibung passt nicht mehr.

Hier gab es mit der Version vorher bei mir auch keinen Fehler (was aber an meinen Settings liegen kann).

Zitat von kamel5

Der letzte ist aber OK so.

Nein, immer noch der selbe Fehler.

Es scheint mir, in diesem Commit werden mehrere verschiedene Sachen geaendert, die auch so nicht in der Beschreibung erklaert werden.

Wenn Du den Commit aufteilst (die NULL-Geschichten, Fix der Debug-Logs, was da jetzt wieso anders initialisiert wird) und das jeweils erklaerst, dann kann man vielleicht auch nachvollziehen, was da schief geht.

Gruss,

S:oren

[SkinNopacity] Aktuelle Probleme

Jetzt mitmachen!

Teilen

Tags