Here, I am trying out both suggestions:  removing the checkbox from the GUI and replacing Create+Get with var_InheritInteger.  However, I have a problem.  Using var_InheritBool instead of Create+Get actually causes the frequency bands to change when stopping a file and playing it back again after changing the value in the Preferences window.  This approach proved very difficult, since I could not find a way to update the frequency labels in the GUI when VLC enters the "playing" state.  (I tried to implement several different callbacks in the extended_panels.cpp file with no success.)  Using the older approach of obtaining the variables (Create+Get) works, since it requires the user to restart the application, making the implementation in the GUI easier.  At this point, I am stuck, since it is undesirable to have incorrect frequency bands in the GUI and it is also undesirable to use Create+Get.<br>
<br><div class="gmail_quote">On Tue, Feb 21, 2012 at 3:35 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Feb 20, 2012 at 10:51:51PM -0600, Ron Wright wrote :<br>
<div class="im">> All right, it seems that all issues have been resolved (except for one).<br>
> There is no mention of Winamp anywhere, doubles have been changed to<br>
> floats, the method of creating the "equalizer-classicvlcfreqs" parameter is<br>
> explicitly set to inherit, the return value VLC_SUCCESS has been added to<br>
> the classic VLC frequency callback function, and C99-style declarations are<br>
> used in the "for" loops.<br>
<br>
</div>I think you should use var_InheritInteger and not Create+Get<br>
<div class="im"><br>
<br>
> However, I have two possibilities here:  should<br>
> the "Use Classic VLC Frequency Bands" checkbox be placed beside the<br>
> "Enable" and "2 Pass" checkbox at the cost of a wider window (version 1<br>
> patch), or should it be placed below the frequency adjustment sliders at<br>
> the cost of a taller window (version 2 patch)?<br>
<br>
</div>First, this should be split from the patch.<br>
Then, does it make any sense to change it live, outside of preferences?<br>
Do people change that often of bands?<br>
<div class="HOEnZb"><div class="h5"><br>
Best regards,<br>
<br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>