On 20/03/16 05:02, Michael Paquier wrote:
On Sun, Mar 20, 2016 at 8:06 AM, Andrew Dunstan <and...@dunslane.net> wrote:

Still to do: the non-perl pieces.

The patch to address locales is the sensitive part. The patch from
Petr is taking the correct approach though I think that we had better
simplify it a bit and if possible we had better not rely on the else
block it introduces.

That would be ideal, not particularly sure that both are possible at the same time. We can definitely remove the else block but it involves enumerating system locales which makes the patch orders of magnitude bigger and uglier. Do you have any better approaches in you mind?

As a side note, I have to say the current locale API in Windows is one huge mess and if it was on me I'd just redefine the locale_t struct correctly as that is simplest solution and the APIs involved are not deprecated or anything, it's just that the public headers are botched in current version and apparently nobody important is using them to force MS to fix them.

  Petr Jelinek                  http://www.2ndQuadrant.com/
  PostgreSQL Development, 24x7 Support, Training & Services

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to