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

BooStar · June 21, 2015 at 12:52 AM

Hallo..
da ich im Moment daran arbeite den tryouts-Skin an das plex-plugin anzupassen, bin ich dank chriszero und frodo auf dem letzen Stand von frodo-testing-vdr.
Leider gibts seit dem letzten Update ein Problem, das plex-plugin stürtzt beim beenden ab, jemand eine Idee woran das liegen könnte?

louis · June 21, 2015 at 7:00 AM

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

Frodo · June 21, 2015 at 10:34 AM

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.

mini73 · June 21, 2015 at 10:43 AM

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

Lars

louis · June 21, 2015 at 10:55 AM

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

Frodo · June 21, 2015 at 11:05 AM

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

BooStar · June 21, 2015 at 11:19 AM

Und genau die sind installiert:

Code

vdr-plugin-plex:
  Installiert: 1:0.1.4-32-g7688256-0frodo0~precise
  Kandidat:    1:0.1.4-32-g7688256-0frodo0~precise
  Versionstabelle:
     1:0.1.4-32-g7688256-0yavdr0~precise 0
        500 http://ppa.launchpad.net/yavdr/testing-vdr/ubuntu/ precise/main amd64 Packages
 *** 1:0.1.4-32-g7688256-0frodo0~precise 0
       1002 http://ppa.launchpad.net/frodo-vdr/testing-vdr/ubuntu/ precise/main amd64 Packages
        100 /var/lib/dpkg/status

Code

vdr-plugin-skindesigner:
  Installiert: 0.5.3-6-g50fe393-0frodo0~precise
  Kandidat:    0.5.3-6-g50fe393-0frodo0~precise
  Versionstabelle:
     0.5.3-6-g50fe393-0yavdr0~precise 0
        500 http://ppa.launchpad.net/yavdr/testing-vdr/ubuntu/ precise/main amd64 Packages
 *** 0.5.3-6-g50fe393-0frodo0~precise 0
       1002 http://ppa.launchpad.net/frodo-vdr/testing-vdr/ubuntu/ precise/main amd64 Packages
        100 /var/lib/dpkg/status

louis · June 21, 2015 at 11:36 AM

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

Ciao Louis

chriszero · June 21, 2015 at 1:59 PM

Schaue ich mir heute abend mal an.

chriszero · June 22, 2015 at 7:20 PM

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?

chriszero · June 22, 2015 at 8:16 PM

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 :-/

louis · June 22, 2015 at 10:02 PM

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

Ciao Louis

chriszero · June 23, 2015 at 8:49 PM

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

BooStar · June 23, 2015 at 11:06 PM

Danke... nun funktioniert es wieder...

louis · June 24, 2015 at 8:39 AM

Quote from 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

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

Participate now!

Users Viewing This Thread