Simplify pg_collation.collversion for Windows libc - Mailing list pgsql-hackers

From Daniel Verite
Subject Simplify pg_collation.collversion for Windows libc
Date
Msg-id 40371128-91d0-4bed-bdb4-19ceeec918af@manitou-mail.org
Whole thread Raw
Responses Re: Simplify pg_collation.collversion for Windows libc
List pgsql-hackers
Hi,

Currently the libc collation version for Windows has two components
coming from the NLSVERSIONINFOEX structure [1]
dwNLSVersion and dwDefinedVersion

So we get version numbers looking like this (with 16 beta1):

postgres=# select collversion,count(*) from pg_collation group by
collversion;
  collversion  | count
---------------+-------
           |     5
 1539.5,1539.5 |  1457
(2 rows)

According to [1] the second number is obsolete, and AFAICS we should
expose only the first.

<quote>
dwDefinedVersion

    Defined version. This value is used to track changes in the repertoire
    of Unicode code points. The value increments when the Unicode
    repertoire is extended, for example, if more characters are defined.

    Starting with Windows 8: Deprecated. Use dwNLSVersion instead.
</quote>

PFA a patch implementing that suggestion.


[1]
https://learn.microsoft.com/en-us/windows/win32/api/winnls/ns-winnls-nlsversioninfoex


Best regards,
--
Daniel Vérité
https://postgresql.verite.pro/
Twitter: @DanielVerite

Attachment

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Let's make PostgreSQL multi-threaded
Next
From: "Jonathan S. Katz"
Date:
Subject: Re: Docs: Encourage strong server verification with SCRAM