[Bug localedata/23831] New: nl_NL missing LC_NUMERIC thousands_sep

[Bug localedata/23831] New: nl_NL missing LC_NUMERIC thousands_sep

[sourceware.org at troep dot com]

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

            Bug ID: 23831
           Summary: nl_NL missing LC_NUMERIC thousands_sep
           Product: glibc
           Version: unspecified
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: sourceware.org at troep dot com
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

Currently the thousands_sep is empty "" this should be changed to "."

I've compiled a small list of sources:
- https://onzetaal.nl/taaladvies/getallen-in-letters-of-cijfers/
- https://onzetaal.nl/taaladvies/getallen-uitschrijven
- https://taaladvies.net/taal/advies/vraag/1
- https://www.taaltelefoon.be/getallen-komma-punt-spatie
- https://en.wikipedia.org/wiki/Decimal_separator#Examples_of_use ->
Netherlands (most common)
- https://www.google.nl/search?q=test (3.180.000.000 resultaten)
- https://www.bing.com/search?q=test&setlang=nl-nl (30.100.000 Resultaten)
- Microsoft Windows also uses . as the default thousands separator.
- All our close neighbours use . (eg: nl_BE fr_BE de_DE da_DK)

Weird why no one bothered to file a bug for the last 22 year:
https://github.com/bminor/glibc/blame/master/localedata/locales/nl_NL#L93

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

[Bug localedata/23831] nl_NL missing LC_NUMERIC thousands_sep

[pander at users dot sourceforge.net]

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

Pander <pander at users dot sourceforge.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pander at users dot sourceforge.ne
                   |                            |t

--- Comment #1 from Pander <pander at users dot sourceforge.net> ---
I can confirm that this needs to be fixed. Is a patch file needed for this or
can this be fixed directly by a maintainer?

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

[Bug localedata/23831] nl_NL missing LC_NUMERIC thousands_sep

[digitalfreak at lingonborough dot com]

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

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

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

--- Comment #2 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Patch is not needed unless you want to see your name in the changelog and in
the commit log. :-) There was a little discussion in bug 21614 (yes,
incorrectly placed) about the correct value of thousands_sep and
mon_thousands_sep.  In short:

* currently thousands_sep is empty and mon_thousands_sep is a narrow no-break
space,
* in this report it has been said that thousands_sep should be a single dot,
* nothing has been said about mon_thousands_sep, is it correct now?

It seems to me that some people prefer a space, some prefer a dot.  CLDR which
is our authoritative source says it should be a dot but it is not clear to me
whether the dot should be applied to mon_thousands_sep (monetary format) or
thousands_sep (other numeric formats) or maybe both.  Is there a context where
a space (especially a narrow no-break space) would be preferred?

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

[Bug localedata/23831] nl_NL missing LC_NUMERIC thousands_sep

[sourceware.org at troep dot com]

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

--- Comment #3 from Jim <sourceware.org at troep dot com> ---
Yes both money and regular numbers should be the same:

mon_thousands_sep         "."
thousands_sep             "."

mon_grouping              3;3
grouping                  3;3

The links I've added in the previous post concur this.

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

[Bug localedata/23831] nl_NL missing LC_NUMERIC thousands_sep

[cvs-commit at gcc dot gnu.org]

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

--- Comment #4 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=fefa21790b5081e5d04662a240e2efd18603ef86

commit fefa21790b5081e5d04662a240e2efd18603ef86
Author: Rafal Luzynski <digitalfreak@lingonborough.com>
Date:   Fri Jun 21 20:18:48 2019 +0200

    nl_{AW,NL}: Correct the thousands separator and grouping (bug 23831).

    According to CLDR 35.1 and the bug report the thousands grouping
    separator should be always "." (a single dot) and digits should be
    grouped by 3.

        [BZ #23831]
        * localedata/locales/nl_AW (mon_thousands_sep): Set to ".".
        * localedata/locales/nl_NL (mon_thousands_sep): Likewise.
        (thousands_sep): Likewise.
        (grouping): Set to 3;3.

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

[Bug localedata/23831] nl_NL missing LC_NUMERIC thousands_sep

[digitalfreak at lingonborough dot com]

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

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.30

--- Comment #5 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Thank you for the report, Jim.  It's fixed now.

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