Bug#582421: [Patch] Floating point is broken if :lang is called in gvim
Bram Moolenaar
Bram at Moolenaar.net
Fri May 21 10:03:24 UTC 2010
James -
> On Thu, May 20, 2010 at 05:01:01PM -0400, James Vega wrote:
> > On Thu, May 20, 2010 at 10:12:50PM +0200, Bram Moolenaar wrote:
> > > > If the locale is changed in gvim via ":lang" to a locale which uses
> > > > comma for the decimal separator, then floating point numbers are no
> > > > longer usable. This only happens with the ":lang $locale" variant,
> > > > since that sets $LANG before calling gtk_set_locale().
> > > >
> > > > env LANG=C gvim
> > > > :lang es_ES.UTF-8
> > > > :lang messages C " so errors are in English
> > > > :echo ceil(1.2)
> > > > E806: using Float as a String
> > > > E116: Invalid arguments for function ceil(1.2)
> > > > E15: Invalid expression: ceil(1.2)
> > > > :echo str2float('1.2')
> > > > 1,0
> > > > :echo str2float('1,2')
> > > > 1,2
> > > >
> > > > Attached patch fixes the issue by ensuring $LC_NUMERIC is set to C if
> > > > the LC_ALL version of :lang is called.
> > >
> > > Thanks. In other places we do this:
> > >
> > > setlocale(LC_NUMERIC, "C");
> > >
> > > Thus I think we should do the same in this place.
> >
> > That won't work here. The issue is specifically to do with the
> > environment variables present when gtk_set_locale is called. In fact,
> > there's already an instance of âsetlocal(LC_NUMERIC, "C");â a few lines
> > above the code my patch adds.
>
> Alternatively, it may be simpler to simply tell Gtk not to call
> setlocale(LC_ALL, "") at all. This would remove the need for the change
> introduced in 7.2.318 as well as the call to gtk_set_locale in
> ex_language().
>
> Attached patch does this and appears to work fine. Gtk dialogs created
> after running ":lang es_ES.UTF-8" are properly translated.
OK, that simplifies things. Looks a bit risky, but in the 7.3 branch we
have time to try it out for everyone.
- Bram
--
hundred-and-one symptoms of being an internet addict:
87. Everyone you know asks why your phone line is always busy ...and
you tell them to send an e-mail.
/// Bram Moolenaar -- Bram at Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ download, build and distribute -- http://www.A-A-P.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
More information about the pkg-vim-maintainers
mailing list