[Buildroot] [PATCH v2] libmad: Add optimization config options

["Benoît Thébaudeau" <benoit.thebaudeau@advansee.com>]

Hi Thomas,

> > Changes for v2:
> >    - Factorize the 'able' in the $(if ).
> 
> I prefer the solution in v1 with regard to this, as it is much easier
> to read.

As you prefer.

> > +choice
> > +	prompt "Speed vs. accuracy"
> > +	default BR2_PACKAGE_LIBMAD_SPEED_ACCURACY
> 
> What about calling those options:
> 
>  BR2_PACKAGE_LIBMAD_OPTIMIZATION_SPEED
>  BR2_PACKAGE_LIBMAD_OPTIMIZATION_ACCURACY
>  BR2_PACKAGE_LIBMAD_OPTIMIZATION_DEFAULT
> 
> > +config BR2_PACKAGE_LIBMAD_SPEED_ACCURACY
> > +	bool "Optimize for both speed and accuracy"
> 
> This should rather be:
> 
> 	bool "Default"
> 
> or something like that. Technically, this option does not enable any
> special optimization for both speed and accuracy, it just uses the
> default options.

Actually, libmad's README says that the default options (neither --enable-speed
nor --enable-accuracy) imply an optimization of both:

"Note that you need not specify one of --enable-speed or --enable-accuracy; in
its default configuration, MAD is optimized for both."

But it refuses to configure with both options, and there is indeed technically
nothing special done by default.

I'll post a v3 doing all that.

Regards,
Beno?t