Talking to myself again...<br><br><div class="gmail_quote">On 30 August 2012 21:39, Mark Lee <span dir="ltr"><<a href="mailto:mark.lee@capricasoftware.co.uk" target="_blank">mark.lee@capricasoftware.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><br>1. If you enable the equalizer filter *after* you start playback, it will 
'forget' to keep the equalizer enabled after you stop/start.<br><br>2. If you enable the equalizer filter *before* you start playback, it will 'remember' and keep the filter enabled even after you stop/start the media multiple times.<br>
</div></div></blockquote><div><br>src/audio_output/common.c relates...<br><br>1. If you enable the filter after you start playback, an aout exists and a new filter chain string gets set on the aout object. When you subsequently stop the media the aout goes away - then when you start again you get a new aout with no filter chain and hence no enabled filter.<br>
<br>2. If you enable the filter before you start playback, an aout does *not* exist and the filter chain string gets set in "config". When you subsequently stop the media the aout goes away - then you start again you get a new aout and this time the previous filter chain is restored from config and the audio filter is enabled.<br>
<br>In aout_ChangeFilterString() in common.c there is this code, where psz_new is the new filter chain string:<br><br>743    if( p_aout )<br>744        var_SetString( p_aout, psz_variable, psz_new );<br>745    else<br>746        config_PutPsz( p_obj, psz_variable, psz_new );<br>
<br>My first idea to fix this issue is simply to always store the audio filter chain in the config even if there is an aout, so:<br><br>743    if( p_aout )<br>744        var_SetString( p_aout, psz_variable, psz_new );<br>
745    config_PutPsz( p_obj, psz_variable, psz_new );<br><br>This works... but I'm not sure how correct it is. <br><br>If that is acceptable I will send another patch, if not I'll dig some more.<br><br>Any comments or other ideas?<br>
<br></div></div>