[Patch] Beseitigung von Speicherlecks im IDE-Menü

[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]

Danke, dann werde ich es mal angehen ...

[rhabarber1848]

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]

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]

ok, dann lass ich es dann doch lieber ...

[rhabarber1848]

@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]

@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]

Kannst beruhigt einbauen.

[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]

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