Re: Bug in noiseprof and solution

[Eric Wong <normalperson@yhbt.net>]

Eric Seigne <eric.seigne@ryxeo.com> wrote:
> Here is a bug on "noiseprof" i've found with a qt/c++ program linked 
> with libsox: i'm french and my LC_ALL is fr_FR,UTF-8,

> here is the "/tmp/speech,noise-profile" file:
> 
> Channel 0: 3,047390, 1,534390, 4,114414, 3,004703, 1,243448, 2,580552, 

<snip>

> please have a look at "," wich is used twice, one for decimal separator 
> and one for fields separator ... so noiseprof does not work !
> 
> if i add
> 
>      setlocale(LC_ALL, "C");

I think you can use LC_NUMERIC instead of LC_ALL to limit the scope of
the locale change.

> and everything works well.
> 
> So is this bug resolved with last git version ?

It does not look to be, it looks like it's caused by the use of
"%f" with fprintf in src/noiseprof.c

I'm think the SoX maintainers would be interested in a portable fix
if you can provide one.

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk