[gelöst][frodo]testing-vdr und plex

Moin,

ist das die aktuelle Git Version (update gestern) oder die 0.5.3 vom Skindesigner? In der aktuellen Version (nach der es ausschaut) habe ich in der libskindesignerapi im Destruktor von cViewGrid ein "view->ClearGrids()" hinzugefügt, damit auch blinkende Icons korrekt abgeräumt werden. Der view Pointer scheint hier nicht mehr gültig zu sein...da müsste wohl Chris mal schauen.

Ciao Louis

Immer wieder libskindesignerapi ...

Ich würde es bevorzugen wenn diese Library in einem eigen git steckt dann würde ich erkennen wenn es dort Änderungen gab womit in der Regel alle abhängigen Plugins neu gebaut werden müssen.

Die Änderung am Destruktor sollte kein Neubau der abhängigen Plugins nach sich ziehen.
Das muss ein anderes Problem sein.

Lars

Welche Version isses denn nu? Wie Lars schon schreibt habe ich nur in einem c File und keinem Headerfile was geändert, da müssen die abhängigen Plugins nicht neu gebaut werden.

Ich vermute das Problem eher in dem neuen Code...Chris scheint da was anders zu machen als ich im TVGuide Plugin. Da macht das zusätzliche Clear keine Probleme.

Ciao Louis

In meinem Repository liegen folgende Versionen:
vdr-plugin-plex = 0.1.4-32-g7688256
vdr-plugin-skindesigner = 0.5.3-6-g50fe393

Ok, also die aktuelle Version mit dem ClearGrids().

Ciao Louis

Schaue ich mir heute abend mal an.

hmm, das kracht bei einem delete auf ein skindesignerapi::cOsdView* beim schließen des OSD? War vorher aber definitiv nicht...
Ich muss doch meinen Heap wieder aufräumen dürfen?

Ich weiß jetzt woran das liegt. Du räumst den Zeiger nochmals weg, Im Clear der viewgrid.c
Ich war in der Annahme das der cOsdView* von GetOsdView() auch wieder gelöscht werden sollte.
In C++11 gibt es dafür std::shared_ptr

edit: Falscher Alarm, das machst du ja auch im tvguideng, delete rootview :-/

Hm, kann es sein das die Reihenfolge der deletes eine Rolle spielt? Der cOsdView sollte zuletzt gelöscht werden...

Ciao Louis

Liegt wirklich daran in welcher Reihenfolge die Objekte freigegeben werden.
das cViewGrid muss vor dem cOsdView freigegeben werden. Das wäre eine Info im Wiki wert gewesen

Zitat von chriszero

das cViewGrid muss vor dem cOsdView freigegeben werden. Das wäre eine Info im Wiki wert gewesen

Werde ich mal aufnehmen, das wurde ja erst mit meiner letzten Änderung so notwendig...

Ciao Louis