[Patch] Beseitigung von Speicherlecks im IDE-Menü

Das Original Benutzerinterface Neutrino-SD incl. zapit, sectionsd, yWeb etc...
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Gaucho316 »

Der COnOffNotifier-Patch ist ohne die Änderungen für die Klasse CDriveSetup im CVS. Wenn die Speicherlecks erledigt sind, liefere ich den dann nach. Ich wollte die drei Patches von dbt nicht kaputt machen.

Der Patch fürs IDE-Menü aus dem ersten Posting war mein Versuch, die Speicherlecks zu schließen. Durch dbts Arbeit ist der überflüssig geworden.

Und size_empty.diff ist schon im CVS.
Mourice

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Mourice »

Danke, dann werde ich es mal angehen ...
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von rhabarber1848 »

dbt hat geschrieben:
dbt hat geschrieben:
patch1_cleanup_drivesetup_17.06.2012.patch
Dieser Patch schlägt im aktuellen CVS fehl, da
http://cvs.tuxbox-cvs.sourceforge.net/t ... 92&r2=1.93
später committed wurde. Es reicht, den ersten Patch des
o.g. commits im patch1, Zeile 88, manuell zu ergänzen.
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 19:18

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von dbt »

Bei mir siehts zeitlich nicht wirklich so gut aus, um alles unter zu kriegen, den Rest würde ich schon noch gerne erledigen. evtl ergibt sich wegen der gerade anstehenden Feiertage mal was.
@rhabarber1848, wenn du die 3 Patche comitten willst, kannst du das machen, meine dbox läuft damit schon seit dem ich die drin habe. Die Box ist zwar momentan nicht als Hauptgerät bei mir am Start, aber trotzdem regelmäßig in Verwendung. Da man diese Sache aber nicht ständig hin und her benutzt, könnte es aber sein das Fehler erst später bemerkt werden. In dem Fall wäre es wohl sinnvoll, zu comitten und Ergebnisse abzuwarten.
Mourice

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Mourice »

ok, dann lass ich es dann doch lieber ...
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von rhabarber1848 »

dbt hat geschrieben:@rhabarber1848, wenn du die 3 Patche comitten willst, kannst du das machen
Mit einem commit erledigt: http://article.gmane.org/gmane.comp.vid ... x.scm/3620
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Gaucho316 »

@dbt

Hast du schon weitergemacht? Falls nicht, würde ich gerne vorher noch alle Notifier-Klassen durch den COnOffNotifier ersetzen und außerdem addIntroItems() einbauen. Ich hatte das ja schon einmal abgeblasen, weil ich dir deine Arbeit nicht durcheinanderbringen wollte.
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 19:18

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von dbt »

Kannst beruhigt einbauen.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Gaucho316 »

Hier nun die angesprochenen Patches. Wenn hinterher alles noch so funktioniert und aussieht wie vorher, habe ich nichts falsch gemacht. Testen kann ich es ja nicht. Kompilieren tut es aber.

COnOffNotifier: add method to remove items
Link entfernt, da Patch in Git

Drive setup: use COnOffNotifier if possible
Link entfernt, da Patch in Git

Drive setup: use addIntroItems() in menus
Link entfernt, da Patch in Git


@dbt

Da hast du ja noch ein ganzes Stück Arbeit vor dir. Da sind ja noch so einige Speicherlecks drin. Und ich weiß jetzt auch, warum durch meinen Patch aus dem ersten Posting Segfaults aufgetreten sind. Es werden nämlich einige Menüelemente mehrfach in (verschiedene) Menuwidgets eingefügt. Löscht man die Menuwidgets dann, werden Menüelemente mehrfach gelöscht.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [Patch] Beseitigung von Speicherlecks im IDE-Menü

Beitrag von Gaucho316 »

Ich habe mir gedacht, dass ich dieses Thema nach ein paar Monaten ruhig mal wieder aus der Versenkung holen kann. Wie sieht's aus? :wink: