[Erledigt] fix possible memleak in audio_select.cpp

[GetAway]

Notifier geht auch ohne new().
Nebenwirkungen habe ich keine feststellen können.

Patch: fix_possible_memleak_audio_select.cpp.diff

[Gaucho316]

Nebenwirkungen habe ich keine feststellen können.

Ich schon. Es funktioniert nämlich nicht mehr, weil das Objekt audioSetupNotifier auf dem Stack angelegt wird und nach dem Verlassen des umgebenden if-Statements nicht mehr existiert. Ich hab's mal korrigiert.

Audio select: fix analog output selection
Link entfernt, da Patch im CVS

[seife]

Würde es nicht reichen, den weiterhin auf dem Stack anzulegen, aber weiter oben (da wo du jetzt den pointer auf NULL setzt, damit er im richtigen Scope ist?

[Gaucho316]

Na klar, das reicht auch. Das war auch mein erster Gedanke. Ich habe mich dann aber umentschieden, weil ich verhindern wollte, dass der Notifier unnötigerweise angelegt wird, wenn g_settings.audio_left_right_selectable false ist. Aber trotzdem habe ich das auch mal als Patch parat. Ihr könnt euch ja aussuchen, welcher von beiden eingecheckt wird.

Audio select: fix analog output selection
Link zum Patch entfernt

[rhabarber1848]

committed:

audio-select_fix-analog-output-selection_2012-06-21_0822.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3488