[Mplayer-cvslog] CVS: main/libmpcodecs vf_smartblur.c,1.1,1.2

Michael Niedermayer michaelni at gmx.at
Sun Nov 24 10:21:25 CET 2002


Hi

On Sunday 24 November 2002 07:24, Rémi Guyomarch CVS wrote:
> Update of /cvsroot/mplayer/main/libmpcodecs
> In directory mail:/var/tmp.root/cvs-serv29954
>
> Modified Files:
> 	vf_smartblur.c
> Log Message:
>
> Locale fix.
[...]
> +#ifdef USE_SETLOCALE
> +    setlocale( LC_NUMERIC, "C" );
> +#endif
[...]
> +#ifdef USE_SETLOCALE
> +    setlocale( LC_NUMERIC, "" );
> +#endif
[...]

hmm, it solves the problem but do u really think this is a good idea? do u 
want to add these around every sscanf("%f...)

why dont u just add a 

mp_sscanf(){
#ifdef USE_SETLOCALE
    setlocale( LC_NUMERIC, "C" );
#endif

sscanf();

#ifdef USE_SETLOCALE
    setlocale( LC_NUMERIC, "" );
#endif
}

btw even better would be some parsing code without any *local() stuff

Michael



More information about the MPlayer-cvslog mailing list