[Bug localedata/25233] New: Consider . as the thousands separator for sl_SI (Slovenian)

[Bug localedata/25233] New: Consider . as the thousands separator for sl_SI (Slovenian)

[fweimer at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

            Bug ID: 25233
           Summary: Consider . as the thousands separator for sl_SI
                    (Slovenian)
           Product: glibc
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: fweimer at redhat dot com
                CC: libc-locales at sourceware dot org
  Target Milestone: ---
             Flags: security-

Originally reported here:
https://sourceware.org/ml/libc-locales/2019-q4/msg00090.html

Cursory research confirms this.

<https://www.gov.si/assets/ministrstva/MDDSZ/VZD-/Resolucija-o-nacionalnem-programu-VZD.pdf>

linked from

<https://www.gov.si/podrocja/zaposlovanje-delo-in-upokojitev/varnost-in-zdravje-pri-delu/>

uses a dot as the separator:

| Zaradi duševnih motenj je bilo pri izbranih zdravnikih v Sloveniji med
| letoma 2009 in 2013 v povprečju 18.471 primerov bolniškega dopusta na
| leto, kar je 2,5 odstotka vseh bolniških dopustov.

Translated as (also linked available from the page):

| Doctors in Slovenia registered an average of 18,471 cases of sick
| leave per year due to mental disorders between 2009 and 2013, which is
| 2.5 per cent of all cases of sick leave.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/25233] Consider . as the thousands separator for sl_SI (Slovenian)

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |digitalfreak@lingonborough.
                   |                            |com

--- Comment #1 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
This is correct according to CLDR [1] as well.  The patch is here: [2]

Some remarks:

* The old thousands separator is not a regular space (<U0020>), it's a <U202F>
character, narrow no-break space, which indeed may be displayed as a space if a
constant width font is used.
* This patch fixes both LC_MONETARY (monetary formatting, strfmon) and
LC_NUMERIC (non-monetary numeric formatting, sprintf and other functions of the
family).
* Note that in LC_NUMERIC section of sl_SI grouping is set to "0;0" which means
that no thousands grouping will be used, at least by the libraries which
respect this setting.

[1] https://st.unicode.org/cldr-apps/v#/sl/Symbols/
[2] https://sourceware.org/ml/libc-alpha/2020-01/msg00042.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/25233] Consider . as the thousands separator for sl_SI (Slovenian)

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Rafal Luzynski <rl@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=135540285cb44f820dac41c2b95531db33658f9f

commit 135540285cb44f820dac41c2b95531db33658f9f
Author: Rafał Lużyński <digitalfreak@lingonborough.com>
Date:   Fri Jan 3 12:11:20 2020 +0100

    sl_SI locale: Use "." as the thousands separator (bug 25233)

    This is correct according to CLDR [1] and Florian Weimer's quick
    research. [2]

    [1] https://st.unicode.org/cldr-apps/v#/sl/Symbols/
    [2] https://sourceware.org/bugzilla/show_bug.cgi?id=25233#c0

    Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/25233] Consider . as the thousands separator for sl_SI (Slovenian)

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Fixed by the commit 135540285cb44f820dac41c2b95531db33658f9f.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/25233] Consider . as the thousands separator for sl_SI (Slovenian)

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |2.31

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/25233] Consider "." as the thousands separator for sl_SI (Slovenian)

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=25233

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Consider . as the thousands |Consider "." as the
                   |separator for sl_SI         |thousands separator for
                   |(Slovenian)                 |sl_SI (Slovenian)

-- 
You are receiving this mail because:
You are on the CC list for the bug.