All right, it seems that all issues have been resolved (except for one).  There is no mention of Winamp anywhere, doubles have been changed to floats, the method of creating the "equalizer-classicvlcfreqs" parameter is explicitly set to inherit, the return value VLC_SUCCESS has been added to the classic VLC frequency callback function, and C99-style declarations are used in the "for" loops.  However, I have two possibilities here:  should the "Use Classic VLC Frequency Bands" checkbox be placed beside the "Enable" and "2 Pass" checkbox at the cost of a wider window (version 1 patch), or should it be placed below the frequency adjustment sliders at the cost of a taller window (version 2 patch)?<br>
<br><div class="gmail_quote">On Mon, Feb 20, 2012 at 1:50 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 Sun, Feb 19, 2012 at 10:55:10PM -0600, Ron Wright wrote :<br>
<div class="im">> All right, so I guess introducing another frequency band isn't such a great<br>
> idea after all if it requires using a registered trademark to describe the<br>
<br>
</div>Just call it "Classic VLC bands" :)<br>
I think it is a good idea to provide both, just not to call it Winamp.<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>