21. helmikuuta 2012 11.35 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> kirjoitti:<br><div class="gmail_quote"><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></div></div></blockquote><div><br></div><div>I personally like it live since it is much easier to hear the differences.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
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>