develooper Front page | perl.perl5.changes | Postings from March 2021

[Perl/perl5] 890e99: XXX craig Unixish.h, doshish.h: Reorderterminatio...

From:
Karl Williamson via perl5-changes
Date:
March 23, 2021 22:10
Subject:
[Perl/perl5] 890e99: XXX craig Unixish.h, doshish.h: Reorderterminatio...
Message ID:
Perl/perl5/push/refs/heads/smoke-me/khw-locale/2ebd3e-4e0cdf@github.com
  Branch: refs/heads/smoke-me/khw-locale
  Home:   https://github.com/Perl/perl5
  Commit: 890e999ebb63cfd8309af56fd42010b61d32f979
      https://github.com/Perl/perl5/commit/890e999ebb63cfd8309af56fd42010b61d32f979
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M dosish.h
    M unixish.h

  Log Message:
  -----------
  XXX craig Unixish.h, doshish.h: Reorder terminations; simplify

The IO and memory terminations need to be after other things.  Add a
comment so that future maintainers won't make the mistakes I did.

Also refactor to that amiga os doesn't have a separate list to get out
of sync

I suspect that the amiga termination should be moved to earlier in
the sequence, but absent any evidence; I'm leaving it unchanged.


  Commit: 33d8e86afc4a5e697d8c3e2da4905f93659d3e66
      https://github.com/Perl/perl5/commit/33d8e86afc4a5e697d8c3e2da4905f93659d3e66
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Win32: Don't check folds validity

This code will check, when warnings are on, that the libc functions
return valid values.  But Windows platforms will always fail because
they have multiple divergences from the Posix standard.  The macros that
implement the case changing/folding in handy.h take extra steps to bring
Windows code more into alignment with Posix.  Those are too complicated
to easily duplicate the logic here.  The result of these checks is
looked at by our test suite, which has long, without anyone noticing,
skipped portions on Windows, even though handy.h should correct for
this.  So simply, don't do the checking under Windows, and find out what
handy.h has failed to fully correct for.


  Commit: fa19701c01845a62e00d1159d2b75a0172995336
      https://github.com/Perl/perl5/commit/fa19701c01845a62e00d1159d2b75a0172995336
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M lib/locale_threads.t

  Log Message:
  -----------
  XXX locale_threads


  Commit: 8740123dfec675b23d30946063dc69110feb8df0
      https://github.com/Perl/perl5/commit/8740123dfec675b23d30946063dc69110feb8df0
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  DEBUG_L now also looks at environment variable

Because locale initialization happens before command line processing,
one can't pass a -DL argument to enable debugging of locale
initialization.  Instead, an environment variable is read then, and is
used to enable debugging or not.  In the past, code specifically had to
test for this being set.  This commit changes that so that debugging can
automatically be enabled without having to write special code.  Future
commits will strip out those special checks.


  Commit: 357cc31999dc2dc0bcea46c0846ccffcf1865094
      https://github.com/Perl/perl5/commit/357cc31999dc2dc0bcea46c0846ccffcf1865094
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Replace most #ifdef DEBUGGING lines

THe previous commit enhanced the DEBUG macros so that they contain the
logic that previously had to be done with conditional compilation
statements.  Removing them makes the code easier to read.


  Commit: 06fc6a98adcec7afee67d31c1a45b84da29a0921
      https://github.com/Perl/perl5/commit/06fc6a98adcec7afee67d31c1a45b84da29a0921
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h
    M numeric.c
    M regcomp.c
    M regexec.c
    M utfebcdic.h

  Log Message:
  -----------
  Change handy.h macro names to be C standard conformant

C reserves symbols beginning with underscores for its own use.  This
commit moves the underscore so it is trailing, which is legal.  The
symbols changed here are most of the ones in handy.h that have few uses
outside it.


  Commit: a984d10547ced19c05768ff9f45e6e123409ba42
      https://github.com/Perl/perl5/commit/a984d10547ced19c05768ff9f45e6e123409ba42
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Remove only 2 calls to an internal macro

Replace isIDFIRST_LC and isWORD_CHAR_LC isIDFIRST_LC  with slightly
faster implementations.


  Commit: c5a9b7e18d74e0e71e739b1ddff10135a49fc7ee
      https://github.com/Perl/perl5/commit/c5a9b7e18d74e0e71e739b1ddff10135a49fc7ee
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Refactor some #ifdef's for commonality

This changes these compilation conditionals so that things in common
between Windows and other platforms are only defined once.

It changes the isIDFIRST_LC and isWORDCHAR_LC definitions for
non-Windows to match that platform superficially, though expanding to
what it previously did to.


  Commit: 906c6809e1a1964b746547af2be09693b1834524
      https://github.com/Perl/perl5/commit/906c6809e1a1964b746547af2be09693b1834524
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Add some branch predictions


  Commit: d510c1ff87ccf97f981bf9be21977ab01c1ccec8
      https://github.com/Perl/perl5/commit/d510c1ff87ccf97f981bf9be21977ab01c1ccec8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: White-space, comment only


  Commit: d84e8abd479e6a54cebe02a9d8cd59afb1b7dad9
      https://github.com/Perl/perl5/commit/d84e8abd479e6a54cebe02a9d8cd59afb1b7dad9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M charclass_invlists.h
    M handy.h
    M l1_char_class_tab.h
    M lib/unicore/uni_keywords.pl
    M perl.c
    M perl.h
    M regcomp.c
    M regcomp.h
    M regen/mk_PL_charclass.pl
    M regexec.c
    M sv.c
    M uni_keywords.h
    M utfebcdic.h

  Log Message:
  -----------
  Change handy.h macro names to be C standard conformant

C reserves symbols beginning with underscores for its own use.  This
commit moves the underscore so it is trailing, which is legal.  The
symbols changed here are many of the ones in handy.h that have
significant uses outside it.


  Commit: 2ab98be66429a0b70be8e60151b03144a3518d1a
      https://github.com/Perl/perl5/commit/2ab98be66429a0b70be8e60151b03144a3518d1a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Rmv internal macro

LC_CAST_ was my attempt at generality, but I didn't realize that the
POSIX standard specifies the type that this was meant to generalize, so
there isn't any need for it.


  Commit: 8c4189413882f8098380fac4c7d9b7c46e3c0681
      https://github.com/Perl/perl5/commit/8c4189413882f8098380fac4c7d9b7c46e3c0681
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Refactor some internal macros

This changes the parameters etc, in preparation for further changes


  Commit: 83466050c5c56e4808815d9ae9c5e506c2b05daa
      https://github.com/Perl/perl5/commit/83466050c5c56e4808815d9ae9c5e506c2b05daa
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Rmv unnecessary parameter to internal macros

The cast is required to be U8 by the POSIX standard.  There is no need
to have this added generality.


  Commit: c106c88047a5ed70086aa0a6424426998153fd42
      https://github.com/Perl/perl5/commit/c106c88047a5ed70086aa0a6424426998153fd42
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: #define one macro in terms of another

These two macros are equivalent as folding and lowercasing are the same
for this input domain.  Better to say so rather than to replicate the
definitions.


  Commit: 33d69b506c7b49d56416049b54aa1ece6a66161a
      https://github.com/Perl/perl5/commit/33d69b506c7b49d56416049b54aa1ece6a66161a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Add wrapper layer macros for isalnum() ...

This adds a new set of macros, forming a lower layer to what is currently
there to wrap the character classification libc functions, isdigit()
etc, and case changing ones, tolower(), toupper().

On most platforms these expand simply to the libc function call.  But on
windows, they expand to something more complex, to bring the Windows
calls into POSIX compliance.  Previously that was achieved at the higher
level, with the result that lower level calls were broken.  This
resulted in parts of the test suite being skipped on Windows.

The current level is rewritten to use the new lower layer, with the
result that it is simpler, as the complexity is now done further down.

I thought about calling these macros is_porcelain_isalnum or something
similar to emphaisze that they are close to the bare libc version, but
thought isU8_alnum() is shorter and conveys another truth, that being
the input is assumed to be a byte, without checking.


  Commit: 67602bf1177be1a7f1c2bd0767d250a9b5c8ed8a
      https://github.com/Perl/perl5/commit/67602bf1177be1a7f1c2bd0767d250a9b5c8ed8a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M vms/vms.c

  Log Message:
  -----------
  locale.c: Use new macros from the prev commit

This should result in Windows boxes now passing the locale sanity
checks.  Previously that failure would cause the test suite tests to be
skipped, and warnings generated to Windows users that actually were
invalid, as the flaws were actually compensated for in other code.


  Commit: 3264783d3e5327e872dc870e5e8b836470f8d33f
      https://github.com/Perl/perl5/commit/3264783d3e5327e872dc870e5e8b836470f8d33f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  XXX SEE IF WORKS handy.h: Change Windows macros


  Commit: 80c74f10252ed2ad16cf53279cfd025b9aa66adc
      https://github.com/Perl/perl5/commit/80c74f10252ed2ad16cf53279cfd025b9aa66adc
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M handy.h

  Log Message:
  -----------
  handy.h: Add isCASED_LC

As a convenience to other code.


  Commit: a2dd3a3527938ce42c40151ac84c615cc54c40ac
      https://github.com/Perl/perl5/commit/a2dd3a3527938ce42c40151ac84c615cc54c40ac
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M regexec.c

  Log Message:
  -----------
  regexec.c: Improve code

These case statements in a switch all had the same prelude for checking
if the locale is UTF-8 and handling that case separately.  A few commits
ago created macros closer to the base level.  This commit factors out
the common UTF-8 handling, and then puts the lower lever things in the
switch().  Perhaps the C optimizer will be smart enough to do this too,
but we might as well do it ourselves, now that it is convenient.


  Commit: d7ea56115d388ece1f562b6f08d53df2b0deabe1
      https://github.com/Perl/perl5/commit/d7ea56115d388ece1f562b6f08d53df2b0deabe1
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M regexec.c

  Log Message:
  -----------
  regexec.c: Refactor switch default()

It seems clearer to me to have the panic at the end of the routine
instead of as the default: of a switch().


  Commit: 7e71d1d56566047c3d472d34885e5f93c9574404
      https://github.com/Perl/perl5/commit/7e71d1d56566047c3d472d34885e5f93c9574404
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Declare three static arrays to be so.


  Commit: 021ac1e4ba6596948964cf19af35da5ae738e8da
      https://github.com/Perl/perl5/commit/021ac1e4ba6596948964cf19af35da5ae738e8da
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  Move some locale.c #defines to perl.h

This is in preparation for them to be used in macros from outside
locale.c


  Commit: 17f7d972e587e89cba9fcf58eee9dd302bae803d
      https://github.com/Perl/perl5/commit/17f7d972e587e89cba9fcf58eee9dd302bae803d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  Mark newly moved symbols as private

The previous commit made certain symbols that previously were local to
locale.c now available everywhere.  Add a trailing underscore to their
names to mark them as private.


  Commit: e8086b20d0a4b1dd8046e739a06a16f19488ddc7
      https://github.com/Perl/perl5/commit/e8086b20d0a4b1dd8046e739a06a16f19488ddc7
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M makedef.pl
    M perl.h

  Log Message:
  -----------
  Add USE_LOCALE_THREADS #define

This is in preparation for supporting configurations where there threads
are available, but the locale handling code should ignore that fact.

This stems from the unusual locale handling of z/OS, where any attempt
is ignored to change locales after the first thread is created.


  Commit: f1661c3d3cf4fe37c09f6c13f0e9512aa16a9646
      https://github.com/Perl/perl5/commit/f1661c3d3cf4fe37c09f6c13f0e9512aa16a9646
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs
    M ext/POSIX/lib/POSIX.pm
    M intrpvar.h
    M locale.c
    M makedef.pl
    M perl.c
    M perl.h
    M sv.c

  Log Message:
  -----------
  Regularize HAS_POSIX_2008_LOCALE, USE_POSIX_2008_LOCALE

A platform shouldn't be required to use the Posix 2008 locale handling
functions if they are present.  Perhaps they are buggy.  So, a separate
define for using them was introduced, USE_POSIX_2008_LOCALE.  But until
this commit there were cases that were looking at the underlying
availability of the functions, not if the Configuration called for their
use.


  Commit: d4fd0e46aef97449dad77a3397c6e5873f3671e8
      https://github.com/Perl/perl5/commit/d4fd0e46aef97449dad77a3397c6e5873f3671e8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Change macro name

Adopt the git convention of 'porcelain' meaning without special
handling.  This makes it clear that porcelain_setlocale() is the base
level.


  Commit: f7a475581fcb7123740cf2407dded962ae6335af
      https://github.com/Perl/perl5/commit/f7a475581fcb7123740cf2407dded962ae6335af
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Cast return of setlocale() to const

If they had it to do over again, the libc makers would have made the
return of this function 'const char *'.  We can cast it that way
internally to catch erroneous uses at compile time.


  Commit: 4a7c4757c69b5e708e40b95b0800cd267ba99d6e
      https://github.com/Perl/perl5/commit/4a7c4757c69b5e708e40b95b0800cd267ba99d6e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Create S_get_category_index()

libc locale categories, like LC_NUMERIC, are opaque integers.  This
makes it inconvenient to have table-driven code.  Instead, we have
tables that are indexed by small positive integers, which are a
compile-time mapping from the libc values.

This commit creates a run-time function to also do that mapping.  It
will first be used in the next commit.

The function does a loop through the available categories, looking for a
match.  It could be replaced by some sort of quick hash lookup, but the
largest arrays in the field have a max of 12 elements, with almost all
searches finding their quarry in the first 6.  It doesn't seem
worthwhile to me to replace a linear search of 6 elements by something
more complicated.  The design intent is this search will be used only at
the edges of the locale-handling code; once found the index is used in
future bits of the current operation.


  Commit: 229a0e18a078d8e7162d284772d955ebf937726a
      https://github.com/Perl/perl5/commit/229a0e18a078d8e7162d284772d955ebf937726a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Use get_category_index()

This creates the first uses of the function added in the previous commit.

It changes the name of a function that now takes an index to have the
suffix _i to indicate its calling parameter is a category index rather
than a category.  This will become a common paradigm in this file in
later commits.

Two macros are also created to call that function; they have suffixes _c
(to indicate the parameter is a category known at compile time, and _r
(to indicate it needs to be computed at runtime).  This is in keeping
with the already existing paradigm in this file.


  Commit: fb01616112d07199fb0544ff2a43b6c00c3c7aed
      https://github.com/Perl/perl5/commit/fb01616112d07199fb0544ff2a43b6c00c3c7aed
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Change S_emulate_setlocale name and sig

It turns out this function is called only from places where we have the
category index already computed; so change the signature to use the
index and remove the re-calculation.

It renames it to emulate_setlocale_i() to indicate that the category
parameter is an index.

This also means, that it's very unlikely that it will be called with an
out-of-bounds value.  Remove the debugging statement for that case (but
retain the error return value).


  Commit: d4690dc14280501df27d962a9209b155a98fd736
      https://github.com/Perl/perl5/commit/d4690dc14280501df27d962a9209b155a98fd736
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M pod/perldelta.pod
    M pod/perldiag.pod

  Log Message:
  -----------
  locale.c: Simplify S_category_name

We can use the new function S_get_category_index() to simplify this.
Also, when I wrote it I didn't know about Perl_form(), and had
reimplemented a portion of it here; which is yanked as well.


  Commit: 7dc656dec7a3f4a02e4b1bf4dabeb61cc260fbd9
      https://github.com/Perl/perl5/commit/7dc656dec7a3f4a02e4b1bf4dabeb61cc260fbd9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Move unreachable code

It turns out this code, setting errno, is unreachable.  Move it to the
place where it would do some good, removing an extraneous, unreachable
return;


  Commit: ce413455d909042a6744c7294d513521ca2d2466
      https://github.com/Perl/perl5/commit/ce413455d909042a6744c7294d513521ca2d2466
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Comment clarifications, white space

Some of these are to make future difference listings shorter

Some of the changes look like incorrect indentation here, but anticipate
future commits.


  Commit: 306bc43928daa6248a7ed3640b99094cdc8385da
      https://github.com/Perl/perl5/commit/306bc43928daa6248a7ed3640b99094cdc8385da
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Separate query part of emulate_setlocale()

This splits a large function so that it is easier to comprehend, and is
in preparation for them to be separately callable.


  Commit: c0082f94b35f8bfd7aad9dc6c4740d36c34bf991
      https://github.com/Perl/perl5/commit/c0082f94b35f8bfd7aad9dc6c4740d36c34bf991
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Outdent previous commit

The previous commit kept the indentation level the same as it moved code
to a new function, even though an outer block was stripped off in the
process.  This was to minimize diff output.  This commit is white space
only.


  Commit: acf69e229658f4dd2314351f29df44c26f085114
      https://github.com/Perl/perl5/commit/acf69e229658f4dd2314351f29df44c26f085114
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Remove spaces around a '##' preprocessor directive

It turns out that at least my gcc preprocessor gets confused in some
contexts if spaces surround the ##.  CAT2() doesn't work for these.

It is working in this context, but future commits will introduce ones
where it won't, so this commit will help make things consistent within
this file

What seems to fail is #define f(x) (..., g(x ## y), ...) where 'x' is a
an already #defined symbol.  I want 'xy', but instead, for example if
'x' has been defined to be 1, I get '1y'


  Commit: d712fa4bd8f26ff02e3a82d2a30f5e1759959133
      https://github.com/Perl/perl5/commit/d712fa4bd8f26ff02e3a82d2a30f5e1759959133
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: #define some macros in terms of a base one

This is so changes to the lowest level automatically propagate to the
others


  Commit: 621d958bf6df8b995548d0ad00d2c4837c7b4ec0
      https://github.com/Perl/perl5/commit/621d958bf6df8b995548d0ad00d2c4837c7b4ec0
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Create new macros for just querying locale

There are two sets of names, which immediately indicate if the result
can be relied on to be thread level or must be assumed to be global to
the whole process.  At the moment they all expand to the same thing,
since on a threadless perl, it's a don't care; and on a threaded perl,
they are all already thread-level, in the Configurations we support.

Future commits will cause the macros to diverge, and comments will be
added then.

For POSIX 2008, this commit causes queries to go directly to the query
function, avoiding S_emulate_setlocale_i() completely.


  Commit: a38ff27fd4fc03835a61cee6ec3d4ea5117071bd
      https://github.com/Perl/perl5/commit/a38ff27fd4fc03835a61cee6ec3d4ea5117071bd
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Generalize certain Win32 calls

The old versions were windows-specific; the changes use a more generic
macro that currently expands to the same thing, but future commits will
change that.


  Commit: 1dc18fa7eb0f473e001884377681ae20cdb9905f
      https://github.com/Perl/perl5/commit/1dc18fa7eb0f473e001884377681ae20cdb9905f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add a convenience #define

This makes it clear if we are using an array that currently only happens
on non-querylocale systems, but that will change in future commits.


  Commit: f396386f5677565136b422fb500baf8bd051d3f9
      https://github.com/Perl/perl5/commit/f396386f5677565136b422fb500baf8bd051d3f9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add setlocale() return context macros

Future commits will benefit from knowing if the return value of
setlocale is to be ignored, just checked for if it worked, or the full
value is needed and can be relied on (or not) to be per-thread.


  Commit: 5c466daed2b6ab96fb7c0ac9fdbf8d0c7042cecf
      https://github.com/Perl/perl5/commit/5c466daed2b6ab96fb7c0ac9fdbf8d0c7042cecf
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add panic check/message

This panic is done when a setlocale unexpectedly fails.


  Commit: 42890e4a0a461b6fc3a7b444850a0dee88f6081b
      https://github.com/Perl/perl5/commit/42890e4a0a461b6fc3a7b444850a0dee88f6081b
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Use a function table to simplify code

Some locale categories require extra steps when they are changed.  This
moves that logic to a table, which gets rid of some code


  Commit: 69684f3ee2fb8ee59f19c2ce3ae2d4bc9744d2d9
      https://github.com/Perl/perl5/commit/69684f3ee2fb8ee59f19c2ce3ae2d4bc9744d2d9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  Perl_setlocale(): Same code for all param2 == NULL

Calling Perl_setlocale() with a NULL 2nd parameter returns the current
locale, rather than changing it.  Previously LC_NUMERIC and LC_ALL were
treated specially; other categories were lumped in with the code that
changes the locale.

Changing some categories involves a non-trivial amount of work.  This
commit avoids that by moving all queries to the same 'if' branch.
LC_NUMERIC and LC_ALL still have to be treated specially, but now it's
all within the same outer 'if', and the unnecessarily executing code
for when the locale changes is avoided.


  Commit: 1c171e44c3b0c769a24e9a9092e9f88a401f39d3
      https://github.com/Perl/perl5/commit/1c171e44c3b0c769a24e9a9092e9f88a401f39d3
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use low level macros at low level

Implementing Perl_setlocale, we can safely use the internal macros that
the public ones expand to call, without the overhead those public macros
impose (which they do to be more immune from improper calls from outside
code).


  Commit: 9dfded37dd834aac5f55f4c55dfc60c347655811
      https://github.com/Perl/perl5/commit/9dfded37dd834aac5f55f4c55dfc60c347655811
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Remove exploratory code

This code was to find out, in debugging builds, if an undocumented glibc
feature worked.  There were no reports that it didn't, and so, after,
several releases, it has served its purpose.  A future commit will allow
enabling this feature as a Configuration option.


  Commit: a113be71ed4f1b35ae2f46a7e98cbf99c7a48ef8
      https://github.com/Perl/perl5/commit/a113be71ed4f1b35ae2f46a7e98cbf99c7a48ef8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Expand scope of cpp conditional

This just doesn't bother with checking some locale-related stuff if not
paying attention to locales.


  Commit: e0a2f40541c073327084f86883c814a66f37eb00
      https://github.com/Perl/perl5/commit/e0a2f40541c073327084f86883c814a66f37eb00
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  locale.c: Create new convenience macro

glibc doesn't have the querylocale() function, available on some other
platforms, such as Darwin and *BSD.  However, it instead has the
equivalent functionality available through an undocumented feature.

This commit allows someone in the know to compile perl to use that
feature, and wraps its API with a macro so that the calling code doesn't
have to be aware of the different APIs of the two methods.

That macro's definition is now done in perl.h, as future commits will
use it in other files.

Since this is an undocumented feature, I am not currently documenting
this wrapper availability.  However, it has been used in the field
without complaint for a couple of releases, as follows:  A more
cumbersome substitute method continues to be used to get what it does.
But in the past both methods were tried and the program died if they
yielded different results.  Since no one has complained, I'm fairly
confident it works.  But sill I'm deferring its more general use.


  Commit: eba5595579c03df4029e0fe9938e5135d6ebb1ed
      https://github.com/Perl/perl5/commit/eba5595579c03df4029e0fe9938e5135d6ebb1ed
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M intrpvar.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: querylocale() doesn't work on LC_ALL

I had misread the man pages.  This bug has been in the field for several
releases now, but most likely hasn't shown up because it's almost always
the case that the locale categories will be set to the same locale.  And
so most implementations of querylocale() would return the correct
result.

This commit works by splitting the calculation of the value of LC_ALL
from S_emulate_setlocale_i() into a separate function, and extending it
to work on querylocale() systems.  This has the added benefit of
removing tangential code from the main line, making
S_emulate_setlocale_i easier to read.

calculate_LC_ALL() is the new function, and is now called from two
places.  As part of this commit, constness is added to PL_curlocales[]

Part of this change is to keep our records of LC_ALL on non-querylocale
systems always up-to-date, which is better practice

And part of this change is temporary, marked as such, to be removed a
few commits later.


  Commit: ca442da48a264421a9dba942a87a701aac533663
      https://github.com/Perl/perl5/commit/ca442da48a264421a9dba942a87a701aac533663
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use setlocale() for init, not P2008

We have found bugs in the POSIX 2008 libc implementations on various
platforms.  This code, which does the initialization of locale handling
has always been very conservative, expecting possible failures due to
bugs in it our the libc implementations, and backing out if necessary to
a crippled, but workable state, if something goes wrong.

I think we should use the oldest, most stable locale implementation in
these circumstances


  Commit: 9ef7a24100d97366960db0bf8f26ef2f23ccf41e
      https://github.com/Perl/perl5/commit/9ef7a24100d97366960db0bf8f26ef2f23ccf41e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Split aggregate LC_ALL from emulate_setlocale

This splits into a separate function the code necessary in some
Configurations to calculate LC_ALL from a potentially disparate
aggregate of categories having different locales.

This is being done just for readability, as this extensive code in the
middle of something else distracts from the main point.

A goto is hence replaced by a recursive call.


  Commit: 971610504c0c238e41c200280d1013a4475033be
      https://github.com/Perl/perl5/commit/971610504c0c238e41c200280d1013a4475033be
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Change internal variable name

The new name better reflects its purpose, so is less confusing


  Commit: 97006638b53a61a1bc36e571429bb8957d0e389e
      https://github.com/Perl/perl5/commit/97006638b53a61a1bc36e571429bb8957d0e389e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Clean up handling of a glibc bug

This commit moves all mention of this bug to just the code that requires
it, and inlines a macro, making it easier to comprehend


  Commit: 0d1c86383cc81d50de361b6d50abf93c254afe8a
      https://github.com/Perl/perl5/commit/0d1c86383cc81d50de361b6d50abf93c254afe8a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Split ancillary from S_emulate_setlocale

This takes the code to update LC_ALL, used only in some Configurations,
out of the main line, making the main line more readable.

It also allows the removal of temporary code added a few commits back


  Commit: d51ed717f00b84701181edfd32fac64223278e54
      https://github.com/Perl/perl5/commit/d51ed717f00b84701181edfd32fac64223278e54
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: locale "" can be disparate

Setting a locale "" means to get the value from environment variables.
These can set locale categories to different locales, and this needs to
be handled.  The logic before this commit only handled the disparate
case when the locale wasn't ""; but this was compensated for elsewhere.
A future commit will remove that compensation.


  Commit: 07cbc2371d8f86f8bca03746a27d763c67372515
      https://github.com/Perl/perl5/commit/07cbc2371d8f86f8bca03746a27d763c67372515
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  Split off setting locale to "" from S_emulate_setlocale

This is done for readability, to move the special casing of setting a
locale to the empty string (hence getting it from the environment) out
of the main line code.


  Commit: fce4912e004e5469a2d59f92c067f52f74bf1b6d
      https://github.com/Perl/perl5/commit/fce4912e004e5469a2d59f92c067f52f74bf1b6d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M sv.c

  Log Message:
  -----------
  sv.c: Duplicate more variables during cloning

These locale-related ones should be getting initialized in the new
thread, but be certain.


  Commit: db3455a0871fd71a43a16e97af00d8af1906f86f
      https://github.com/Perl/perl5/commit/db3455a0871fd71a43a16e97af00d8af1906f86f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M embedvar.h
    M intrpvar.h
    M locale.c
    M makedef.pl
    M perl.c
    M proto.h
    M sv.c

  Log Message:
  -----------
  locale.c: Add fcn to hide edge case undefined behavior

The POSIX 2008 API has an edge case in that the result of most of the
functions when called with a global (as opposed to a per-thread) locale
is undefined.

The duplocale() function is the exception which will create a per-thread
locale containing the values copied from the global one.

This commit just calls duplocale, if needed, and the caller need not
concern itself with this possibility


  Commit: 0fc026643f0cc46a9447ffe64109c556fa49bad4
      https://github.com/Perl/perl5/commit/0fc026643f0cc46a9447ffe64109c556fa49bad4
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add DEBUGGING information

These functions are called as expansions of macros.  It may be useful to
know where in the file the macro occurred.


  Commit: 65c2bfa295d97ac576c55a97262cd11e16bd6b56
      https://github.com/Perl/perl5/commit/65c2bfa295d97ac576c55a97262cd11e16bd6b56
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Separate out two Win fcns from a larger one

This makes the larger one easier to understand, and prepares for
possible independent calls to the two, which are potentially useful on
their own.


  Commit: 8c7077f4bbd1b024233b36499752cb2bbc71eecb
      https://github.com/Perl/perl5/commit/8c7077f4bbd1b024233b36499752cb2bbc71eecb
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs

  Log Message:
  -----------
  POSIX.xs: Use macro to reduce complexity

This #defines a macro and uses it to populate a structure, so that
strings don't have to be typed twice.


  Commit: 675083dded4be1c5ebb08595efebe9e5be11595b
      https://github.com/Perl/perl5/commit/675083dded4be1c5ebb08595efebe9e5be11595b
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs

  Log Message:
  -----------
  POSIX.xs: White-space only

Properly indent some nested preprocessor directives


  Commit: 4c41e9c08f9a67467f02b563703410d1b251164e
      https://github.com/Perl/perl5/commit/4c41e9c08f9a67467f02b563703410d1b251164e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M ext/POSIX/POSIX.xs
    M locale.c
    M proto.h

  Log Message:
  -----------
  Move code from POSIX.xs to locale.c

This avoids duplicated logic.


  Commit: a0d4f4756c977895035a61fed82b668da907ba14
      https://github.com/Perl/perl5/commit/a0d4f4756c977895035a61fed82b668da907ba14
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Reorder cases in a switch

This moves handling the CODESET to the end, as future commits will make
its handling more complicated.  The cases are now ordered so the
simplest (based on the direction of future commits) are first


  Commit: 39774c68e744d82721a036c48b8a28f5f49e7d51
      https://github.com/Perl/perl5/commit/39774c68e744d82721a036c48b8a28f5f49e7d51
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Make statics of repeated string constants

These strings are (or soon will be) used in multiple places; so have
just one definition for them.


  Commit: 1c63e8870f795606f707dd232e8fdfde6054eb06
      https://github.com/Perl/perl5/commit/1c63e8870f795606f707dd232e8fdfde6054eb06
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add two #defines

This makes sure that we handle having any variant of nl_langinfo() or
localeconv().


  Commit: 339069923823e88ba803415414a5b619811afd20
      https://github.com/Perl/perl5/commit/339069923823e88ba803415414a5b619811afd20
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Return defaults for uncomputable langinfo items

Return the values from the C locale for nl_langinfo() items that aren't
computable on this platform.  If the platform has nl_langinfo(), then
all of them are computable, but if not, some can't be computed, and
others can be, but only if there are alternative methods available on
the platform.

As part of this commit, S_my_nl_langinfo() and S_save_to_buffer() are no
longer used when USE_LOCALE is not defined, so don't compile them.


  Commit: 616d08bfa4fa227a825e71982e274b9e2ad6f111
      https://github.com/Perl/perl5/commit/616d08bfa4fa227a825e71982e274b9e2ad6f111
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Rmv reimplementation of my_strftime()

Prior to this commit, there was a near duplicate copy of the code from
util.c that implements my_strftime().  This was done because the util.c
version zaps the wday field, which made it incompatible.

But it dawned on me that if the arbitrary date we use to do our
calculations were such that it was for a year in which January 1 falls
on a Sunday, then the util.c version automatically works.


  Commit: b77e37adb7918c3ef4bb679c921cde04e7954cb9
      https://github.com/Perl/perl5/commit/b77e37adb7918c3ef4bb679c921cde04e7954cb9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Shorten static function name

The extra syllable(s) are unnecessary noise


  Commit: 2f5a5872ade1f0d01a5b11833482e82ed7ded170
      https://github.com/Perl/perl5/commit/2f5a5872ade1f0d01a5b11833482e82ed7ded170
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Extend a static function

This will allow it to be used in situations where the buffer it controls
is single use, and we don't need to keep track of the size for future
calls.


  Commit: acf1b80737ca176e3fcae108072de46e0642b030
      https://github.com/Perl/perl5/commit/acf1b80737ca176e3fcae108072de46e0642b030
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use typedef to simplify

This allows some preprocessor conditionals to be removed


  Commit: f6d833e5ada3e3d4d49931268409af43fbb07c60
      https://github.com/Perl/perl5/commit/f6d833e5ada3e3d4d49931268409af43fbb07c60
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Rmv redundant cBOOL()

strEQ and && already return booleans


  Commit: 7cbea3a7bd40df24c013846f2941d7cef03d6d09
      https://github.com/Perl/perl5/commit/7cbea3a7bd40df24c013846f2941d7cef03d6d09
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Fix currency symbol derivation

On platforms without nl_langinfo(), we derive the currency symbol from
localeconv().  The symbol must be tweaked to conform to nl_langinfo()
standards.  Prior to this commit, it guessed at how to tweak a rare
circumstance.  I now have seen evidence this guess was wrong, so give up
on it.

This also no longer returns just an empty string in certain cases.
nl_langinfo() itself doesn't, so conform to that.


  Commit: e60947483d51ad21ca77277502eda72b2f35ccb1
      https://github.com/Perl/perl5/commit/e60947483d51ad21ca77277502eda72b2f35ccb1
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Don't add CP to Windows code page names

The actual name appears to be just the number for purposes of
nl_langinfo()-ish things.


  Commit: 75a759662c0bfbbe3456ea60d735957adffdd1a8
      https://github.com/Perl/perl5/commit/75a759662c0bfbbe3456ea60d735957adffdd1a8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Don't ask a static fcn to be inlined

It's too complicated to really be inlined, and the compiler can figure
things out itself given it is a static function


  Commit: 7278fffce1fa06bb82d20ed8617a34fd1f132f9e
      https://github.com/Perl/perl5/commit/7278fffce1fa06bb82d20ed8617a34fd1f132f9e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Rmv no longer used param from static fnc

Previous commits have gotten rid of this parameter to S_save_to_buffer


  Commit: 55810cddba795daf0383f241e21d99da18cc87da
      https://github.com/Perl/perl5/commit/55810cddba795daf0383f241e21d99da18cc87da
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Don't change locale if already there

Changing the locale is cheap for some categories, but expensive for
others.  Changing LC_COLLATE is most expensive, requiring recalculation
of the collation transformation mapping.

This commit checks that we aren't already in the desired locale before
changing locales. and does nothing if no change is needed.


  Commit: c10e4c8e0278ae2e214471b69342f82b4edee45f
      https://github.com/Perl/perl5/commit/c10e4c8e0278ae2e214471b69342f82b4edee45f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M intrpvar.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  Make three locale PL_ strings const char*

This adds some compile safety to these.


  Commit: 5ff462198dc02473218de5383d49e7ebfeedd2bc
      https://github.com/Perl/perl5/commit/5ff462198dc02473218de5383d49e7ebfeedd2bc
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use a scratch buf; instead of resuing old

This is in preparation for the next commit


  Commit: ec4c576dbb9ed15b222dc3e21f41b2aced9bd3fc
      https://github.com/Perl/perl5/commit/ec4c576dbb9ed15b222dc3e21f41b2aced9bd3fc
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Make static fcn reentrant

This makes my_langinfo() reentrant by adding parameters specifying where
to store the result.

This prepares for future commits, and fixes some minor bugs for XS
writers, in that the claim was that the buffer in calling
Perl_langinfo() was safe from getting zapped until the next call to it
in the same thread.  It turns out there were cases where, because of
internal calls, the buffer did get zapped.


  Commit: 9e051f035824848a3fa87168f534dbdaef76baaa
      https://github.com/Perl/perl5/commit/9e051f035824848a3fa87168f534dbdaef76baaa
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: langinfo: Use Windows fcn to find CODESET

There is a Windows function, available for quite a long time, that will
return the current code page.  Use this for the nl_langinfo() CODESET,
as that libc function isn't implemented on Windows.


  Commit: d04b501d972328f686537c4e4abfd8ab485d701f
      https://github.com/Perl/perl5/commit/d04b501d972328f686537c4e4abfd8ab485d701f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add static fcn to analyze locale codeset

It determines if the name indicates it is UTF-8 or not.  There are
several variant spellings in use, and this hides that from the the
callers.

It won't be actually used until the next commit


  Commit: 156528db6184fa041782480a1732772f797ac85a
      https://github.com/Perl/perl5/commit/156528db6184fa041782480a1732772f797ac85a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/I18N-Langinfo/Langinfo.pm
    M locale.c

  Log Message:
  -----------
  locale.c: Improve non-nl_langinfo() CODESET calc

Prior to this commit, on non-Windows platforms that don't have a
nl_langinfo() libc function, the code completely punted computation of
the CODESET item.  I have not been able to figure out how to do this,
even going to the locale definition files on disk (which may vary
anyway), but we can do a lot better than punting.

This commit adds three checks:

1) If the locale name is C or POSIX, we know the codeset

2) We can detect if a locale is UTF-8.  If it is, that is the codeset.
Many modern locales are of this ilk.

3) Failing that, some locales have the codeset appear in the name,
following a dot.

It isn't perfect, but it's a lot better than completely punting.


  Commit: ce8507caed55a86d39066773baeba82fc4b0b494
      https://github.com/Perl/perl5/commit/ce8507caed55a86d39066773baeba82fc4b0b494
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  New signature for static fcn my_langinfo()

This commit changes the calling sequence for my_langinfo to add the
desired locale (or a sentinel to indicate to use the current locale),
and the locale category of the desired item.

This allows the function to be able to return the desired value for any
locale, avoiding some locale changes that would happen until this
commit, and hiding the need for locale changes from outside functions,
though a couple continue to do so to avoid potential multiple changes.


  Commit: c4e123123d41543079bc3bd6568ace9f31a9752a
      https://github.com/Perl/perl5/commit/c4e123123d41543079bc3bd6568ace9f31a9752a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add is_locale_utf8()

Previous commits have added the infrastructure to be able to determine
if a locale is UTF-8.  This will prove useful, and this commit adds
a function to encapsulate this information, and uses it in a couple of
places, with more to come in future commits.

This uses as a final fallback, mbtowc(), which some sources view was a
late adder to C89, and others as not really being available until C99.
Future commits will add heuristics when that function isn't available.


  Commit: 0d6b1ed6bf30eac7a5a77dbbef52bdf3328d6e8d
      https://github.com/Perl/perl5/commit/0d6b1ed6bf30eac7a5a77dbbef52bdf3328d6e8d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add fcn for UTF8ness determination

get_locale_string_utf8ness_i() will determine if the string it is passed
in the locale it is passed is to be treated as UTF-8, or not.


  Commit: 5fc036102c346bd39291534ef2ab6586f490acc8
      https://github.com/Perl/perl5/commit/5fc036102c346bd39291534ef2ab6586f490acc8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M ext/POSIX/POSIX.xs
    M locale.c
    M proto.h

  Log Message:
  -----------
  XXX perldelta Move POSIX::localeconv() logic to locale.c

The code currently in POSIX.xs is moved to locale.c, and reworked some
to fit in that scheme, and the logic for the workaround for the Windows
broken localeconv() is made more robust.

This is in preparation for the next commit which will use this logic
instead of (imperfectly) duplicating it.

This also creates Perl_localeconv() for direct XS calls of this
functionality.


  Commit: ab3016a6eb1b299120adab9e1991995f6d8030d3
      https://github.com/Perl/perl5/commit/ab3016a6eb1b299120adab9e1991995f6d8030d3
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Collapse duplicate logic into one instance

The previous commit move the logic for localeconv() into locale.c.  This
commit takes advantage of that to use it instead of repeating the logic.

On Windows, there is alternative way of finding the radix character for
systems that have a localeconv() that could cause a race.  Prior to this
commit, if that failed to find something that looked like the radix, it
returned a '?'.  Now it will drop down to using this new code, as the
likelihood of the race is small.

Notably, this commit removes the inconsistent duplicate logic that had
been used to deal with the Windows broken localeconv() bug.


  Commit: 16628a22fa26d1bb6a8008e777ef46520f451e70
      https://github.com/Perl/perl5/commit/16628a22fa26d1bb6a8008e777ef46520f451e70
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Fix windows bug with broken localeconv()

localeconv() was broken on Windows until VS 2015.  As a workaround, this
was using my_snprintf() to find what the decimal point character is,
trying to avoid our workaround for localeconv(), which has a (slight)
chance of a race condition.

The problem is that my_snprintf() might not end up calling snprintf at
all; I didn't trace all possibilities in Windows.  So it doesn't make
for a reliable sentinel.

This commit now specifically uses libc snprintf(), and if it fails, drops
down to try localeconv().


  Commit: e35634959d50aaef771465202969e70ac91901a9
      https://github.com/Perl/perl5/commit/e35634959d50aaef771465202969e70ac91901a9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M ext/POSIX/POSIX.xs
    M locale.c
    M proto.h

  Log Message:
  -----------
  XXXdelta Add my_strftime8()

This is like plain my_strftime(), but additionally returns an indication
of the UTF-8ness of the returned string


  Commit: 360b266fc8ef04788c1837e8f12f6376acf0a734
      https://github.com/Perl/perl5/commit/360b266fc8ef04788c1837e8f12f6376acf0a734
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Add utf8ness return param to static fcn

my_langinfo_i() now will additionally return the UTF-8ness of the
returned string.


  Commit: a1450e1a694fb23ccdcc76bf0022b36b808bdca6
      https://github.com/Perl/perl5/commit/a1450e1a694fb23ccdcc76bf0022b36b808bdca6
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M ext/I18N-Langinfo/Langinfo.xs
    M locale.c
    M proto.h

  Log Message:
  -----------
  XXXdelta Add Perl_langinfo8()

This is like Perl_langinfo() but additionally returns information about
the UTF-8ness of the returned string.


  Commit: ac3d14afb3b08659ea36837e9331ca8b7fe41403
      https://github.com/Perl/perl5/commit/ac3d14afb3b08659ea36837e9331ca8b7fe41403
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add fallbacks if no mbtowc()

This add heuristics that work well for non-English locales to determine
if a locale is UTF-8 or not when mbtowc() isn't available.  It would be
a very rare compiler that didn't have that these days, but this covers
that case as best as I have been able to figure out.


  Commit: f2dc178b0d1c075c878527b9078893ebd4969618
      https://github.com/Perl/perl5/commit/f2dc178b0d1c075c878527b9078893ebd4969618
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use Strerror(), not strerror()


  Commit: 60206b66b32f2aaaaf6c9b6c65ea23fa0ddd9608
      https://github.com/Perl/perl5/commit/60206b66b32f2aaaaf6c9b6c65ea23fa0ddd9608
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Refactor #ifdef's for clarity

The my_strerror() function has effectively 5 different implementations
depending on the capabilities of the platform.  Only a few lines are
common to all, the set-up and the return.  The #ifdefs obscure the
underlying logic.  So this commit separates them out into 5 different
functions, with the result that it's clear what is going on in each.


  Commit: 9184a82678d1333cf046a9ec687f98b3ea4b222c
      https://github.com/Perl/perl5/commit/9184a82678d1333cf046a9ec687f98b3ea4b222c
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  Avoid mojibake in "$!"

In stress testing, I discovered that the LC_CTYPE and LC_MESSAGES
locales need to be the same locale, or strerror() can return
question marks or mojibake instead of the proper message.

This commit refactors the handling of stringifying "$!" to make the
locales of both categories the same during the stringification.

Actually, I suspect it isn't the locale, but the codeset of the locale
that needs to be the same.  I suspect that if the categories were both
in different UTF-8 locales, or both in single-byte locales, that things
would work fine.  But it's cheaper to find the locale rather than the
locale's codeset, so that is what is done.


  Commit: 5841a79f79b2c5c08ef56193a8936af6f3efcdec
      https://github.com/Perl/perl5/commit/5841a79f79b2c5c08ef56193a8936af6f3efcdec
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M makedef.pl
    M mg.c
    M proto.h

  Log Message:
  -----------
  Move utf8ness calc for $! into locale.c from mg.c

locale.c has the infrastructure to handle this, so remove repeated
logic.

The removed code tried to discern better based on using script runs, but
this actually doesn't help, so is removed.


  Commit: 59b21addaeeef2d84a3355070ea9e83e88e63168
      https://github.com/Perl/perl5/commit/59b21addaeeef2d84a3355070ea9e83e88e63168
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M mg.c

  Log Message:
  -----------
  mg.c: White-space only

Indent newly formed block from the previous commit.


  Commit: a0b9e4ce423e9652b1f0b6b9a680a6b9e18e6fef
      https://github.com/Perl/perl5/commit/a0b9e4ce423e9652b1f0b6b9a680a6b9e18e6fef
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M embedvar.h
    M intrpvar.h
    M locale.c
    M proto.h
    M sv.c

  Log Message:
  -----------
  locale.c: Rmv no longer used code; UTF8ness cache

What these functions do has been subsumed by code introduced in previous
commits, and in a more straight forward manner.

Also removed in this commit is the cache of the knowing what locales are
UTF-8 or not.  This data is now cheaper to calculate when needed, and
there is now a single entry cache, so I don't think the complexity
warrants keeping it.

It could be added back if necessary, split off from the remainder of
this commit.


  Commit: 833c2efadced0e1f118599731f62d543c41dc6f1
      https://github.com/Perl/perl5/commit/833c2efadced0e1f118599731f62d543c41dc6f1
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  Don't discard locale info in starting P2008

The program is started in the global locale, and then is converted to
the POSIX 2008 per-thread locale API.  Prior to this commit the startup
locale was discarded.  It really should be the foundation for the 2008
locales.  I don't know of any current paths through the code that this
makes a difference for, but it is a potential hole that is easy to plug.


  Commit: ca3737e5d2d1f5c6040e7d45e8f3fc16a922534b
      https://github.com/Perl/perl5/commit/ca3737e5d2d1f5c6040e7d45e8f3fc16a922534b
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M perl.h
    M proto.h

  Log Message:
  -----------
  Add a common locale panic macro and functions

This will make sure that all the necessary clean up gets done.


  Commit: b6d1144cd2d75f240b5aeceb9a07dce11cf1ca35
      https://github.com/Perl/perl5/commit/b6d1144cd2d75f240b5aeceb9a07dce11cf1ca35
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Revamp sync_locale()

This rarely used function was actually failing to do what it purported
in some Configurations.


  Commit: 1d15a6c323d646eced10edf39846252e298e4e5f
      https://github.com/Perl/perl5/commit/1d15a6c323d646eced10edf39846252e298e4e5f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Clean up thread_locale_init()

We can use internal functions to this file instead of the API ones here.
This commit also calls  sync_locale() to avoid repeated logic.


  Commit: 97e0ed335e821f6ae75d04a8b8397d1b82bf48b8
      https://github.com/Perl/perl5/commit/97e0ed335e821f6ae75d04a8b8397d1b82bf48b8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  Revamp switch_to_global_locale()

Prior to this commit, the global locale was not always getting populated
with the values from the thread being switched.


  Commit: 8f955ac1fe8e2d1b2c84c131036c08c82a90fef1
      https://github.com/Perl/perl5/commit/8f955ac1fe8e2d1b2c84c131036c08c82a90fef1
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Omit an extra copy

In this case in Perl_setlocale(), we can just return the plain result
from setlocale(), as, if something further needs to be done that would
destroy it, that is taken care of already at the time.

On per-thread locale platforms, the result already is in a per-category
buffer.


  Commit: b26d482ca33c94ca3df038c1710d530716e35e30
      https://github.com/Perl/perl5/commit/b26d482ca33c94ca3df038c1710d530716e35e30
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embedvar.h
    M intrpvar.h
    M locale.c
    M makedef.pl
    M perl.c
    M sv.c

  Log Message:
  -----------
  locale.c: Cache the current LC_CTYPE locale name

This is now used as a cache of length 1 to avoid having to lookup up the
UTF-8ness as often.

There was a complicated cache previously, but changes to the logic
caused that to be much less necessary, and it is no longer actually
used, and will be removed in a later commit.

But it's pretty easy to keep this single value around to cut further
down the new scheme's need to look it up


  Commit: d40ccb3c2e68cb20ad3f0a817e8a2f613dcc2735
      https://github.com/Perl/perl5/commit/d40ccb3c2e68cb20ad3f0a817e8a2f613dcc2735
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M intrpvar.h

  Log Message:
  -----------
  intrpvar.h: Initialize a variable

I don't believe there is a bug with this PL_numeric_name being
uninitialized, but this is an easy precaution.


  Commit: 0b6704427489e4c8169e553df260f2d5a56c6bb3
      https://github.com/Perl/perl5/commit/0b6704427489e4c8169e553df260f2d5a56c6bb3
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  Swap the ordering of two locale category indices

Perl internally uses a mapping of locale category values into a
consecutive sequence of indices starting at 0.  These are used as
indexes into arrays.  The reason is that the category numbers are
opaque, vary by platform, aren't necessarily sequential, and hence are
hard to make table driven code for.

This commit makes the LC_CTYPE index 0, and LC_NUMERIC equal to 1;
swapping them.  The reason is to cause LC_CTYPE to get done first in the
many loops through the categories.  The UTF8ness of categories is an
often needed value, and most of the time the categories will have the
same locale.  LC_CTYPE is needed to calculate the UTF8ness, and by doing
it first and caching the result, the other categories likely
automatically will use the same value, without having to recalculate.


  Commit: 4ae9209482fde80426cf03ed1fdc4e06d8510299
      https://github.com/Perl/perl5/commit/4ae9209482fde80426cf03ed1fdc4e06d8510299
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use new mechanism to save/restore errno

Instead of explicitly saving the errno around debugging statements, the
new more general mechanism is used.


  Commit: 0945158addc2c67a8a29c68065732214e1ed805e
      https://github.com/Perl/perl5/commit/0945158addc2c67a8a29c68065732214e1ed805e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Move DEBUG location info

This commit takes advantage of the new mechanism to add common DEBUGGING
code to print the __FILE__ and __LINE__ of every debugging statement.
This allows those to be removed from each statement, and have them
implicitly added.

This make things consistent, and easier to read and add new statements.


  Commit: 06edac05cd2b97aad920f128b4e9afd61911916f
      https://github.com/Perl/perl5/commit/06edac05cd2b97aad920f128b4e9afd61911916f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add some asserts


  Commit: c26e73bc4b1b0ce648253defcdc721330a69fef0
      https://github.com/Perl/perl5/commit/c26e73bc4b1b0ce648253defcdc721330a69fef0
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Reorder code, rmv unneeded conditional

Previous commits have made the conditional about being able to find the
radix character unnecessary.  The called function my_langinfo_c()
handles the case properly.

This commit also makes the trivial case first in a conditional, as that
is easier to comprehend.


  Commit: 61a7ff45fdd40498499dbef5cd83fd29f79ab12b
      https://github.com/Perl/perl5/commit/61a7ff45fdd40498499dbef5cd83fd29f79ab12b
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Reorder 'if' branches

It's better for understandability to have positive tests than negative
ones


  Commit: cd0f6fff604701f15a4f4e3c965c2419270cd436
      https://github.com/Perl/perl5/commit/cd0f6fff604701f15a4f4e3c965c2419270cd436
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Refactor a static function

S_new_numeric() is called after the LC_NUMERIC category is changed, to
update various ancillary information Perl keeps.

This reorders the function so that on POSIX 2008 platforms, the numeric
object is created earlier.  This allows for fewer operations on those
platforms, as we already have the correct value in place for querying
what the radix and thousands separator characters are.

Explanatory comments are also added.


  Commit: 6f6a6a8155a074134d26deebbdd4fec6c7a48373
      https://github.com/Perl/perl5/commit/6f6a6a8155a074134d26deebbdd4fec6c7a48373
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Change assert() into STATIC_ASSERT()


  Commit: 6381e9ded6defef331cfc3a2f719af44f9e63579
      https://github.com/Perl/perl5/commit/6381e9ded6defef331cfc3a2f719af44f9e63579
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use standard fold table for C locale

Copy the standard compiled-in ASCII fold table when the locale is C or
POSIX, instead of looping through all 256 characters and computing them.
This saves some time as well as ensures that any platform bugs become
irrelevant.


  Commit: e11d8e7b239537d2fc7699412408e963fdb32b8c
      https://github.com/Perl/perl5/commit/e11d8e7b239537d2fc7699412408e963fdb32b8c
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add check that strxfrm didn't fail

The code failed to take into account that strxfrm() can fail for reasons
besides buffer length.  It does not return errors, and the only way to
check is to set errno to 0 beforehand, and check that it is still 0
afterwards.


  Commit: ac0bcea08ef07ec4782ed7fa8dbe613082c94681
      https://github.com/Perl/perl5/commit/ac0bcea08ef07ec4782ed7fa8dbe613082c94681
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Don't assume LC_CTYPE, LC_COLLATE are same

This code is using isCNTRL_LC which depends on LC_CTYPE to verify that
something in the LC_COLLATE locale is a control.  That only works
properly if the two locales are the same.  This commit adds code to
ensure they are.


  Commit: 7c4523c859084a4e30b89646b712aa6acceb2cdb
      https://github.com/Perl/perl5/commit/7c4523c859084a4e30b89646b712aa6acceb2cdb
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: strxfrm() requires LC_CTYPE eq LC_COLLATE

The libc functions strxfrm() on some platforms requires the LC_CTYPE
locale to be the same as the LC_COLLATE locale (or rather, probably that
they have the same code set, but checking for locale is cheaper).
Otherwise mojibake would result, or more likely the function will fail,
setting errno.

This commit brings the locales into alignment if necessary


  Commit: f4fa947c3d65bea19b972f13eb03cf31789cd3ba
      https://github.com/Perl/perl5/commit/f4fa947c3d65bea19b972f13eb03cf31789cd3ba
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M Configure
    M Cross/config.sh-arm-linux
    M Cross/config.sh-arm-linux-n770
    M NetWare/config.wc
    M Porting/config.sh
    M config_h.SH
    M configure.com
    M metaconfig.h
    M plan9/config_sh.sample
    M uconfig.h
    M uconfig.sh
    M uconfig64.sh
    M win32/config.gc
    M win32/config.vc

  Log Message:
  -----------
  Configure: strxfrm_l


  Commit: 5a52d9b63bef2ed5dc03d9d643d8445593346e15
      https://github.com/Perl/perl5/commit/5a52d9b63bef2ed5dc03d9d643d8445593346e15
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M lib/locale.t

  Log Message:
  -----------
  XXX temp: Windows debug


  Commit: b3924c5138dfb84dfe5a0a6e891de181a4f8d1b4
      https://github.com/Perl/perl5/commit/b3924c5138dfb84dfe5a0a6e891de181a4f8d1b4
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Use strxfrm_l() if available

This more modern version of the function doesn't require us to change
locales.


  Commit: 2e3cfc409b2dc74c38c70d497fe297d61ee2fbb0
      https://github.com/Perl/perl5/commit/2e3cfc409b2dc74c38c70d497fe297d61ee2fbb0
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M mathoms.c
    M proto.h
    M sv.c

  Log Message:
  -----------
  Change name of internal function

This is in preparation for working on it; the new name, mem_collxfrm_ is
in compliance with the C Standard; the old was not.


  Commit: cec34168251e3bfc98869ef0705db996f2d0ae6d
      https://github.com/Perl/perl5/commit/cec34168251e3bfc98869ef0705db996f2d0ae6d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M ext/POSIX/POSIX.xs
    M ext/POSIX/lib/POSIX.pod
    M locale.c
    M proto.h

  Log Message:
  -----------
  XXXdelta Fix POSIX::strxfrm()

This function takes an SV containing a PV.  The encoding of that PV is
based on the locale of the LC_CTYPE locale.  It really doesn't make
sense to collate based off of the sequencing of a different locale, which
prior to this commit it would do if the LC_COLLATION locale were
different.


  Commit: f4920c34782a0af9ceef4cef9fb5b13a9bd5ad41
      https://github.com/Perl/perl5/commit/f4920c34782a0af9ceef4cef9fb5b13a9bd5ad41
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Improve debugging for mem_collxfrm()

This prints out more information, better organized.

It also moves up the info from -DLv to plain -DL


  Commit: 1ec3c9017463705e65ee6a496c1979d5b2bf76ce
      https://github.com/Perl/perl5/commit/1ec3c9017463705e65ee6a496c1979d5b2bf76ce
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Add debug statement for collation failure

Perhaps this should be a warning to the user that we couldn't calculate
collation info for the locale, but at least there should be a way to
get that info from a DEBUG statement


  Commit: 41680ec418dd02b2a0d3572c86be6c5ae1119a83
      https://github.com/Perl/perl5/commit/41680ec418dd02b2a0d3572c86be6c5ae1119a83
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c: Print code point in hex, not decimal

Hex is the more familiar form


  Commit: 688e24cb15f3f939ca656fb50878f89e922ed2a9
      https://github.com/Perl/perl5/commit/688e24cb15f3f939ca656fb50878f89e922ed2a9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs
    M locale.c
    M perl.h

  Log Message:
  -----------
  Mark certain mutex lock macros as private

mbtowc() mblen(), and wctomb() should not be directly used by XS
writers; instead use the POSIX versions.  Don't encourage the direct use
by having public macros to aid in their use.


  Commit: f3618cc34a15735aaad2a62ed3175bd3842d9cdd
      https://github.com/Perl/perl5/commit/f3618cc34a15735aaad2a62ed3175bd3842d9cdd
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Move some code around

This is purely to make future commits have smaller real difference
listings, and involves a temporary (complemented) copy of a preprocessor
conditional.


  Commit: 790f629714e22d71d998bf421ceaeb3647009ad4
      https://github.com/Perl/perl5/commit/790f629714e22d71d998bf421ceaeb3647009ad4
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Reorder cpp branches

Disposing of the trivial case first makes things easier to read.


  Commit: b4b208ed99db73095e498db58cc91f35bc5937c4
      https://github.com/Perl/perl5/commit/b4b208ed99db73095e498db58cc91f35bc5937c4
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embedvar.h
    M intrpvar.h
    M locale.c
    M makedef.pl
    M perl.h
    M sv.c

  Log Message:
  -----------
  Make the locale mutex a general semaphore

Future commits will use this new capability, and in Configurations where
no locale locking is currently necessary.


  Commit: cd92591632bfd9438914883937ea3d51c51ca2e3
      https://github.com/Perl/perl5/commit/cd92591632bfd9438914883937ea3d51c51ca2e3
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embedvar.h
    M intrpvar.h
    M makedef.pl
    M perl.h
    M perlvars.h
    M sv.c

  Log Message:
  -----------
  Use general locale mutex for numeric operations

This commit removes the separate mutex for locking locale-related
numeric operations on threaded perls; instead using the general locale
one.  The previous commit made that a general semaphore, so now suitable
for use for this purpose as well.

This means that the locale can be locked for the duration of some
sprintf operations, longer than before this commit.  But on most modern
platforms, thread-safe locales cause this lock to expand just to a
no-op; so there is no effect on these.  And on the impacted platforms,
one is not supposed to be using locales and threads in combination, as
races can occur.  This lock is used on those perls to keep Perl's
manipulation of LC_NUMERIC thread-safe.  And for those there is also no
effect, as they already lock around those sprintf's.


  Commit: fb579818b2a58cb11860da03826fd05314c16fbd
      https://github.com/Perl/perl5/commit/fb579818b2a58cb11860da03826fd05314c16fbd
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  Add locale macro to wrap static-space-using fncs

Some functions return a result in a global-to-the-program buffer, or
they have an internal global buffer.  Other threads must be kept from
simultaneously using that function.  This macro is to be used for all
such ones dealing with locales.  Ideally, there would be a separate mutex
for each such buffer space.  But these functions also have to lock the
locale from changing during their execution, and there aren't that many
such functions, and they actually are rarely executed.  So a single lock
will do.

This will allow future commits to have more targeted locking for
functions that don't affect the global locale.


  Commit: 8817908c7b488c748d1952dfd7630dc92adeba2d
      https://github.com/Perl/perl5/commit/8817908c7b488c748d1952dfd7630dc92adeba2d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  Redefine the POSIX.xs locale macros using prev commit

This commit uses the new macro introduced in the previous commit to
define the internal locale mutex macros in POSIX.xs


  Commit: d484434bbe1c2c8960c9c265b6429a6d5b46b151
      https://github.com/Perl/perl5/commit/d484434bbe1c2c8960c9c265b6429a6d5b46b151
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  perl.h: Remove NL_LANGINFO_LOCK

This is needed in precisely one place in the code, so move it to there.


  Commit: 17fd9844deef6ac1b43063aee151c150268795c5
      https://github.com/Perl/perl5/commit/17fd9844deef6ac1b43063aee151c150268795c5
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  perl.h: Remove LOCALECONV_LOCK

This is needed in just one function, in locale.c, so more it there.


  Commit: 9da840ecf2b137b9bf71c27c29132dd468996052
      https://github.com/Perl/perl5/commit/9da840ecf2b137b9bf71c27c29132dd468996052
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c
    M perl.h

  Log Message:
  -----------
  XXX perlembed Add PORCELAIN_SETLOCALE_LOCK/UNLOCK

This macro is used to surround raw setlocale() calls so that the return
value in a global static buffer can be saved without interference with
other threads.

There are a few very rarely occurring instances in locale.c that are
converted to use this.  These previously could have been races.

The raw setlocales in the initialization function are not guarded, as
these happen early in the Perl process initialization, before threading
is enabled.

This is buggy if there are multiple embedded perls.  It can't be helped.
perlembed is being updated to indicate this.


  Commit: 8c856a356a7d819a840a22da7dd4c310ac1a2a6a
      https://github.com/Perl/perl5/commit/8c856a356a7d819a840a22da7dd4c310ac1a2a6a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Move #defining SETLOCALE_LOCK

This simplifies slightly, and will allow further simplification


  Commit: eb33acbdf0e48ac6093943a82d74305713e19b91
      https://github.com/Perl/perl5/commit/eb33acbdf0e48ac6093943a82d74305713e19b91
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Move LOCALE_READ_LOCK #definition

To enable future simplifications


  Commit: f11571f5393a52ae5173ddba8beab3a5ee1be127
      https://github.com/Perl/perl5/commit/f11571f5393a52ae5173ddba8beab3a5ee1be127
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M intrpvar.h
    M locale.c
    M makedef.pl
    M perl.c
    M perl.h
    M sv.c

  Log Message:
  -----------
  locale.c: Move #define to perl.h; use it elsewhere

 Rather than recalculate this combined conditional, do it once in
 perl.h.


  Commit: 4e04583d575057651a0d606a1b2d3160b49d8374
      https://github.com/Perl/perl5/commit/4e04583d575057651a0d606a1b2d3160b49d8374
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M embed.fnc
    M embed.h
    M locale.c
    M proto.h

  Log Message:
  -----------
  locale.c: Mitigate unsafe threaded locales

This a new set of macros and functions to do locale changing and
querying for platforms where perl is compiled with threads, but the
platform doesn't have thread-safe locale handling.

All it does is:

1) The return of setlocale() is always safely saved in a per-thread
buffer, and
2) setlocale() is protected by a mutex from other threads which are
using perl's locale functions.

This isn't much, but it might be enough to get some programs to work on
such platforms which rarely change or query the locale.


  Commit: ba2c0e5ef36d82652b2dd13f153cf40ebb8289f2
      https://github.com/Perl/perl5/commit/ba2c0e5ef36d82652b2dd13f153cf40ebb8289f2
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  XXX make sure comments get moved appropriately perl.h: Remove now empty block

Previous commits have left this empty except for comments.


  Commit: 27c38dba6af47cbae650f401901af9005388cb9d
      https://github.com/Perl/perl5/commit/27c38dba6af47cbae650f401901af9005388cb9d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M pp.c

  Log Message:
  -----------
  XXX pp.c: do %g print under mutex,


  Commit: 6a48ca39a1aefe5cf955a8c3cd36d3d0b0cbd771
      https://github.com/Perl/perl5/commit/6a48ca39a1aefe5cf955a8c3cd36d3d0b0cbd771
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ebcdic_tables.h
    M embedvar.h
    M globvar.sym
    M inline.h
    M intrpvar.h
    M perl.h
    M regen/ebcdic.pl

  Log Message:
  -----------
  Make fc(), /i thread-safe on participating platforms

A long standing bug in Perl that has gone undetected is that the array
is global that is created when changing locales and tells fc() and qr//i
matching what the folds are in the new locale.

What this means is that any program only has one set of fold definitions
that apply to all threads within it, even if we claim that the locales
are thread-safe on the given platform.  One possibility for this going
undetected so long is that no one is using locales on multi-threaded
systems much.  Another possibility is that modern UTF-8 locales have the
same set of folds as any other one.

It is a simple matter to make the fold array per-thread instead of
per-process, and that solves the problem transparently to other code.

I discovered this stress-testing locale handling under threads.  That
test will be added in a future commit.


  Commit: 3c228f54c3a8bb50f7d8ed2a9257159026cbc97b
      https://github.com/Perl/perl5/commit/3c228f54c3a8bb50f7d8ed2a9257159026cbc97b
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M inline.h
    M locale.c

  Log Message:
  -----------
  XXX temp debug? locale.c, inline.h:foldEQ_locale


  Commit: f96ed8a08b69f9cd7b125c950a3f04caaf2d1f66
      https://github.com/Perl/perl5/commit/f96ed8a08b69f9cd7b125c950a3f04caaf2d1f66
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  locale.c comments


  Commit: 9b39601c90b0273447c065676e7bf20ed73509ff
      https://github.com/Perl/perl5/commit/9b39601c90b0273447c065676e7bf20ed73509ff
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M locale.c

  Log Message:
  -----------
  XXX prob drop; done before anything so no races


  Commit: 478562762c36b262f9bf337ad871e67d35cea90c
      https://github.com/Perl/perl5/commit/478562762c36b262f9bf337ad871e67d35cea90c
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Add #define for gwENVr_LOCALEr_UNLOCK

This is for functions that read the locale and environment and write to
some global space.


  Commit: c3d8cef5b9e718110b9c7667617e9acd0237cc53
      https://github.com/Perl/perl5/commit/c3d8cef5b9e718110b9c7667617e9acd0237cc53
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h
    M time64.c

  Log Message:
  -----------
  Remove ENV_LOCALE_LOCK/UNLOCK macros

These are subsumed by gwENVr_LOCALEr_LOCK created in the previous
commit.


  Commit: 183e2c3c7513b061d555e95981ed635fa6971e9d
      https://github.com/Perl/perl5/commit/183e2c3c7513b061d555e95981ed635fa6971e9d
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h
    M time64.c
    M util.c

  Log Message:
  -----------
  Change ENV/LOCALE locking read macro names

The old name was confusing.


  Commit: d6c525d10ede8ba10b41fe5d5186968801cd60c3
      https://github.com/Perl/perl5/commit/d6c525d10ede8ba10b41fe5d5186968801cd60c3
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Move some statements

So they are closer to related statements


  Commit: 82c204a9bfd828d91a92fa9b1816c268aca6cc9c
      https://github.com/Perl/perl5/commit/82c204a9bfd828d91a92fa9b1816c268aca6cc9c
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h
    M util.c

  Log Message:
  -----------
  perl.h: Finish implementing combo ENV/LOCALE mutexes

There are cases where an executing function is vulnerable to either the
locale or environment being changed by another thread.  This commit
implements macros that use mutexes to protect these critical sections.
There are two cases that exist:  one where the functions only read; and
one where they can also need exclusive control so that a competing
thread can't overwrite the returned static buffer before it is safely
copied.

5.32 had a placeholder for these, but didn't actually implement it.
Instead it locked just the ENV portion.  On modern platforms with
thread-safe locales, the locale portion is a no-op anyway, so things
worked on them.

This new commit extends that safety to other platforms.  This has long
been a vulnerability in Perl.


  Commit: d0eb08487325526579696944fff1ab1e559716e8
      https://github.com/Perl/perl5/commit/d0eb08487325526579696944fff1ab1e559716e8
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M time64.c

  Log Message:
  -----------
  time64.c: Remove no longer needed code

This code defined some macros; those are now defined by perl.h


  Commit: 9ea68c0f9195b7c2f5db2ee38fcfe0cf375b92e7
      https://github.com/Perl/perl5/commit/9ea68c0f9195b7c2f5db2ee38fcfe0cf375b92e7
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M pp_sys.c

  Log Message:
  -----------
  XXX need to StructCopy pp_sys mutexes


  Commit: cd48e9e535c71dd6ec7cfd1e09c5e5462d1cab1a
      https://github.com/Perl/perl5/commit/cd48e9e535c71dd6ec7cfd1e09c5e5462d1cab1a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M win32/win32.c

  Log Message:
  -----------
  win32.c: Add mutexes around some calls

These could have races.


  Commit: b05d8dd992fd5231eab7022e7dd9d92361411a4a
      https://github.com/Perl/perl5/commit/b05d8dd992fd5231eab7022e7dd9d92361411a4a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M util.c

  Log Message:
  -----------
  util.c: mktime needs to run under a mutex

per the Posix standard


  Commit: f251b634d2ee388d24c038c64dc9f2b06892a709
      https://github.com/Perl/perl5/commit/f251b634d2ee388d24c038c64dc9f2b06892a709
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M dist/ExtUtils-ParseXS/lib/perlxs.pod
    M t/porting/known_pod_issues.dat

  Log Message:
  -----------
  perlxs


  Commit: 4391e54da9a6902b804e641ffdd915cb401f21e9
      https://github.com/Perl/perl5/commit/4391e54da9a6902b804e641ffdd915cb401f21e9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs

  Log Message:
  -----------
  POSIX.xs env locks, check file for more


  Commit: 00511c6dc9da971ddabe7bacc5a46d8543599748
      https://github.com/Perl/perl5/commit/00511c6dc9da971ddabe7bacc5a46d8543599748
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cygwin/cygwin.c

  Log Message:
  -----------
  cygwin


  Commit: 1050e302be9bac0917759a1cedfeff2a7743060a
      https://github.com/Perl/perl5/commit/1050e302be9bac0917759a1cedfeff2a7743060a
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M os2/os2.c

  Log Message:
  -----------
  os2: Use many reader lock instead of exclusive

This is just reading the environment, not changing it, so a many readers
can be accessing it at the same time.


  Commit: 01d8d3ff23149972d8f05715140fc92d3b46b6cf
      https://github.com/Perl/perl5/commit/01d8d3ff23149972d8f05715140fc92d3b46b6cf
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.pm
    M cpan/Time-Piece/Piece.xs

  Log Message:
  -----------
  XXX cpan PR Time-Piece: Add locks

This add mutex locking around some unsafe thread operations to make this
module thread-safe.


  Commit: d09bbee82d07e352015e841c03e0ba89765aa9fd
      https://github.com/Perl/perl5/commit/d09bbee82d07e352015e841c03e0ba89765aa9fd
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.xs

  Log Message:
  -----------
  Time-Piece: Use foldEQ_locale() if available

This supported core function is thread-safe and knows about Perl
internals, so is preferable to the similar libc function, which is now
used only as a fallback.  This commit also bomb proofs the code by
adding an additional fallback, specified in C89, which isn't a great
substituted, but far better than nothing.


  Commit: 393d52a26eec7b69b16d77d4374c08fc8f9a0052
      https://github.com/Perl/perl5/commit/393d52a26eec7b69b16d77d4374c08fc8f9a0052
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.xs

  Log Message:
  -----------
  Time-Piece: Use isSPACE, not isspace

The latter gives results that are dependent on the program's underlying
locale, and so may be inconsistent.

If locale dependence is actually desired, isSPACE_LC should be used, as
it knows about various things the module writer shouldn't have to
concern themselves with.  It is supported since 5.004


  Commit: a00a3c20c917a213037d1e741a9d41789b823f36
      https://github.com/Perl/perl5/commit/a00a3c20c917a213037d1e741a9d41789b823f36
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.xs

  Log Message:
  -----------
  Time-Piece: Use isDIGIT, not isdigit

The latter gives results that are dependent on the program's underlying
locale, and so may be inconsistent.

If locale dependence is actually desired, isDIGIT_LC should be used, as
it knows about various things the module writer shouldn't have to
concern themselves with.  It is supported since 5.004


  Commit: b9df8b7f8786357cd7c09f1adacfade874f53329
      https://github.com/Perl/perl5/commit/b9df8b7f8786357cd7c09f1adacfade874f53329
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.xs

  Log Message:
  -----------
  Time-Piece: Use isUPPER, not isupper

The latter gives results that are dependent on the program's underlying
locale, and so may be inconsistent.

If locale dependence is actually desired, isUPPER_LC should be used, as
it knows about various things the module writer shouldn't have to
concern themselves with.  It is supported since 5.004


  Commit: 929a9dbd9ece80ca325ae8d74cdb285bdbe16aa5
      https://github.com/Perl/perl5/commit/929a9dbd9ece80ca325ae8d74cdb285bdbe16aa5
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M pod/perlhacktips.pod

  Log Message:
  -----------
  XXX incomplete perlhacktips:


  Commit: 66c4ccf4287dc34755dea071efbd752b2c46f1a4
      https://github.com/Perl/perl5/commit/66c4ccf4287dc34755dea071efbd752b2c46f1a4
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M dist/IO/IO.pm
    M dist/IO/IO.xs

  Log Message:
  -----------
  XXX check if using ppport IO.xs: Remove fallback code furnished by ppport


  Commit: 3ff6b2a5e08a87b3ee9de85bc752d84d38cc7969
      https://github.com/Perl/perl5/commit/3ff6b2a5e08a87b3ee9de85bc752d84d38cc7969
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M hints/freebsd.sh

  Log Message:
  -----------
  XXX check with freebsd: hints/freebsd.sh


  Commit: edb5e5647a4c63693253474325cdf0d567c65ca9
      https://github.com/Perl/perl5/commit/edb5e5647a4c63693253474325cdf0d567c65ca9
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M thread.h

  Log Message:
  -----------
  thread.h: White-space, braces only


  Commit: cca809eaa7afc30824dea7b2108e504b3d64ce5e
      https://github.com/Perl/perl5/commit/cca809eaa7afc30824dea7b2108e504b3d64ce5e
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M thread.h

  Log Message:
  -----------
  XXX thread.h Save errno around lock/unlock


  Commit: 8a742e7e134bce77f2a8781d48f24d128d2405db
      https://github.com/Perl/perl5/commit/8a742e7e134bce77f2a8781d48f24d128d2405db
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  XXX perl.h: Debugging mutex lock'


  Commit: 3b28dde27bc63a8e308f881c6a9f52bede019272
      https://github.com/Perl/perl5/commit/3b28dde27bc63a8e308f881c6a9f52bede019272
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M cpan/Time-Piece/Piece.xs
    M handy.h
    M iperlsys.h
    M locale.c
    M perl.h
    M regen/reentr.pl
    M regexec.c
    M util.c

  Log Message:
  -----------
  Notes


  Commit: b4e58ec8e0c125682a05b6f24ec8692682952e73
      https://github.com/Perl/perl5/commit/b4e58ec8e0c125682a05b6f24ec8692682952e73
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M ext/POSIX/POSIX.xs
    M locale.c
    M perl.h

  Log Message:
  -----------
  locks


  Commit: 570106f38532c78f08fa584be4218bc10b34c64f
      https://github.com/Perl/perl5/commit/570106f38532c78f08fa584be4218bc10b34c64f
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M lib/locale_threads.t

  Log Message:
  -----------
  l


  Commit: 4e0cdf33af536d0472d816ea4eaee428146da681
      https://github.com/Perl/perl5/commit/4e0cdf33af536d0472d816ea4eaee428146da681
  Author: Karl Williamson <khw@cpan.org>
  Date:   2021-03-23 (Tue, 23 Mar 2021)

  Changed paths:
    M lib/locale_threads.t
    M locale.c
    M t/run/locale.t

  Log Message:
  -----------
  l


Compare: https://github.com/Perl/perl5/compare/2ebd3e1e4aee...4e0cdf33af53



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About