[vlc-devel] [PATCH v2 4/9] equalizer: Add support for extra sets of ISO bands

[Jean-Baptiste Kempf]

On 28 Nov, Ron Wright wrote :
> The EQZ_INCLUDE_MATH_ROUTINES define is needed because I wanted to make it
> optional to include math.h before including equalizer_presets.h when
> EqzGetTrueFrequency(), which requires math.h, is not used.  At the same

Why?

> >I'd say you need a selector with a few choices.
> What's wrong with the selector that's already present?  It allows the user
> to select from 10, 15, or 31 bands.

Because you have now one checkbox, plus a spinbox? And all the choices
of the spinbox are not valid...

With my kindest regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device