[Patch] Sprachauswahl ohne CMenuOptionLanguageChooser

[Gaucho316]

Ich habe das Sprachauswahlmenü überarbeitet. Optisch und funktionstechnisch ändert sich nichts. Allerdings kann nun die Klasse CMenuOptionLanguageChooser entfallen, die sowieso irgendwie gaga ist. Außerdem wird so automatisch ein Speicherleck geschlossen, da in der Methode COsdLangSetup::showSetup() vor meiner Änderung ein Wert mit strdup kopiert und im Falle, dass er an CMenuOptionLanguageChooser übergeben wird, nie wieder gelöscht wird. Ich habe den Patch getestet und bei mir funktioniert er.

Link entfernt, da Patch im CVS

[GetAway]

Positiv getestet.

[seife]

Ist denn der pointer, der jetzt ge-free()d wird noch Gültig, wenn das Menüdingens aufgerufen wird? (Im Patch fehlte mir der Kontext, aber das wäre auf jeden Fall zu prüfen...)

[Gaucho316]

Ist denn der pointer, der jetzt ge-free()d wird noch Gültig, wenn das Menüdingens aufgerufen wird?

Das Kopieren mit strdup() ist jetzt gar nicht mehr drin und damit der Fehler auch nicht.

[seife]

Ja, aber ist namelist[count] das du dem "new CMenuForwarderNonLocalized" übergibst nicht schon gefree()d worden, wenn das Menü aufgebaut wird? Oder wird das im Konstruktor kopiert?

[Gaucho316]

Ja, das wird an beiden Stellen in ein std::string kopiert.

[rhabarber1848]

neutrino_rework-language-selection_2012-03-15_2100.diff

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3313
http://article.gmane.org/gmane.comp.vid ... x.scm/3314

[Gaucho316]

Noch ein kleiner Nachtrag. Irgendwie fehlte in meinem Patch das Entfernen der Zeile 39 (und 40)

Code: Alles auswählen

#include <system/setting_helpers.h>

aus osdlang_setup.h. Die ist dort überflüssig.

[rhabarber1848]

osdlang_setup.h. Die ist dort überflüssig.

committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3320