All right, so I guess introducing another frequency band isn't such a great idea after all if it requires using a registered trademark to describe the frequency band that VLC uses.  In this submission, I have changed all doubles to floats, removed the option to use the other frequency band (which eliminates the issue with the Winamp trademark and the inherit issue), and changed the for loop so that the "i" variable declaration is made within the "for" statement.  Everything else (besides removal of unused structures and variables) should be the same.<br>
<br><div class="gmail_quote">On Sun, Feb 19, 2012 at 5:46 PM, 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 Fri, Feb 17, 2012 at 02:40:38PM -0600, Ron Wright wrote :<br>
> +#define WINAMP_BANDS_TEXT N_( "Use Winamp frequency bands" )<br>
> +#define WINAMP_BANDS_LONGTEXT N_( \<br>
> +         "Use the frequency bands used in Winamp. Otherwise, use the ISO " \<br>
> +         "Standard frequency bands." )<br>
<br>
> +    add_bool( "equalizer-winampfreqs", true, NULL, WINAMP_BANDS_TEXT,<br>
> +              WINAMP_BANDS_LONGTEXT, true )<br>
<br>
No. Winamp is a registered trademark.<br>
<br>
> +    double d_rate = (double) i_rate;<br>
> +    double d_nyquist_freq = 0.5 * d_rate;<br>
> +    double d_octave_factor = pow( 2.0, 0.5 * d_octave_percent );<br>
<br>
Do you need double here?<br>
<br>
> +    int i;<br>
<br>
Declare it in the for loop. We are C99.<br>
<br>
> +    EqzCoeffs( i_rate, 1.0,<br>
> +               var_CreateGetBool( p_aout, "equalizer-winampfreqs" ), &cfg );<br>
<br>
Sure you do not need inherit here?<br>
<br>
> +    int i;<br>
as above.<br>
<br>
> diff -purN a/modules/gui/qt4/ui/equalizer.ui b/modules/gui/qt4/ui/equalizer.ui<br>
> --- a/modules/gui/qt4/ui/equalizer.ui 2010-04-12 19:22:27.000000000 -0500<br>
> +++ b/modules/gui/qt4/ui/equalizer.ui 2011-07-06 18:20:46.371654207 -0500<br>
> @@ -36,6 +36,13 @@<br>
>        </widget><br>
>       </item><br>
>       <item><br>
> +      <widget class="QCheckBox" name="winampFreqCheck" ><br>
> +       <property name="text" ><br>
> +        <string>Use Winamp Frequency Bands</string><br>
> +       </property><br>
> +      </widget><br>
> +     </item><br>
> +     <item><br>
>        <spacer><br>
<br>
I do not see why we need to be able to change those freq live. This is<br>
not something you change all the time.<br>
<br>
Best regards,<br>
<span class="HOEnZb"><font color="#888888"><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>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>