Document grok_atou as strtoul replacement.

Use grok_atou instead of strtoul (no explicit strtol uses).

Advertise grok_atou.

Add atoi to avoidables.

Add tests for grok_atou.

Use grok_atou instead of atoi.

Remaining atoi() uses include at least:
ext/DynaLoader/dl_aix.xs, os2/os2.c, vms/vms.c

Implement grok_atou as safe/strict atoi replacement.

For earlier discussion, see:
http://www.nntp.perl.org/group/perl.perl5.porters/2013/10/msg208680.html
https://rt.perl.org/Public/Bug/Display.html?id=116118#txn-1250187

grok_atou is completely new code, instead of trying to bolt
new parameters/flags to grok_number.  This makes it easier to
be extremely strict, and not worry about breaking grok_number.

Use %I64d instead of %lld for MinGW

For older versions of MSVCRT.dll %lld is still using only 32 bits, breaking
32-bit builds defining USE_64_BIT_INT. %I64d is still supported on all newer
MSVCRT.dll releases, so there is really no reason to use %lld with MinGW at
all.

Fixes https://rt.perl.org/Ticket/Display.html?id=120851

perlexperiment mentions features a lot, refer to the feature module

_LIB_VERSION scan was trying leading address twice.

Important with PERL_GLOBAL_STRUCT_PRIVATE.

Mention libperl.t, and explain nm output some more.

Move two variable assignments outside of parens.

Done for readability. Remove some unused variables. Move declaration of other
variable closer to where it is used. Rationalize indentation on lines
modified. Increment $VERSION.

For: RT #122282 (additional)

Open3.pm:spawn_with_handles remove alot of duplicate operators

dont call fileno twice, don't deref key "open_as" 3 extra times.

Avoid undefined warnings.

In FreeBSD memcmp symbol can disappear.

sv.h: Document SVt_INVLIST fields

utf8_heavy.pl: Fix nit in comment

cpan/Time-HiRes/t/itimer.t: better diagnostics

This test is failing some smokes. Add better diagnostics to see what's
going on.

Since this is a CPAN-first module, this commit either needs backing out at
some point, or merging upstream.

Disallow importing functions from UNIVERSAL

It's been deprecated since v5.12.

Regen podcheck db for new long verbatims in perlepigraph

perlop: Grammatical nit.

'either' refers only to one of two possibilities.  Since this
has more than that, 'either' is grammatically incorrect.

Upped version numbers for ModuleList

Bump version number from 5.21.2 to 5.21.3

New perldelta.

Installing Inline no longer installs Inline::C.

So, when testing cpan, install Inline::C.

Epigraph for 5.21.2

Merge branch 'release-5.21.2' into blead

Conflicts:
t/porting/libperl.t

Warn against tmpfile, and mention perlclib.

Skip symbols for which Configure found no support.

Comments for dlopen.

Expect sqrtl instead of sqrt if applicable.

Adjust 31e62fa1f35be3aceacc7fa62ce72d7131299a9a

It seems that 'sqrt' is not always called as a system call, in particular,
when compiling with -Dusemorebits

Add new release to perlhist

Acknowledgement section of perldelta

Listed updated modules.

Update Module::CoreList for 5.21.2

Revert "Update HTTP-Tiny to CPAN version 0.044"

This reverts commit 43bf4e6652b9fc743b1dbbc5153ced8b08779e84.

HTTP::Tiny contains tests that requires a connection to www.example.com.
These tests fail if run behind a firewall.

Add fgets+tmpfile to avoidables, and add comments.

Add freebsd support.

Parsing linux input in darwin, and vice versa.

Add comments.

Scan for expected and unwanted undef (libc) symbols.

We would expect to see e.g. chmod() being used,
but we do not want to see e.g. gets() being used, ever.

Sometimes Darwin has the string constants in cstring section.

At least in OS X 10.7.

Resolves the darwin x86 part of RT #12267 (the failing PL_no_mem test).

test.pl:eq_array() is enough.

No bad is_deeply() emulation necessary.

More debugging output.

Handle darwin ppc nm output, reenable test.

Not darwin ppc in particular: handle 32-bit nm output, in general.
Resolves part of RT #122267.

Allow reading nm output from a file (or STDIN).

Changes for 5.21.1 -> 5.21.2.

Not included the parts that are automated (module changes and acknowledgements).

Tick off the 5.20.0 and 5.20.1 releases.

"all question marks" is ambigious.

Because if just a single question mark is used, it means the release
manager is unknown. Only if multiple question marks are used it
means it isn't known whether this release will happen.

perlpolicy - Add encoding to fix podcheck.t following 0c6082f411

perlpolicy: update list of moderators

Note that ExtUtils::Manifest has been upgraded to 1.64

The EXCLUDED files are also now different.

perlpolicy - Note that minimal build fixes are acceptable for maint too

Changes like 31114fe991, which has just been cherry-picked into maint-5.20,
should be allowed.

uconfig.h again

I am so sorry

cut-n-paste error in comment for dladdr in config_h.SH for Condifure

Update HTTP-Tiny to CPAN version 0.044

  [DELTA]

0.044     2014-07-16 23:46:09-04:00 America/New_York

    [CHANGED]

    - Providing a custom 'Host' header is now a fatal exception.  Previously, it
      was silently ignored, as the RFC mandates that Host be set from the
      URL, but ignoring it could lead to unexpected, confusing errors.

    - optimized URL splitting

    - Passing 'undef' for any proxy attribute will prevent HTTP::Tiny from
      setting the proxy from the environment.

Merge branch '122300-extutils-manifest' into blead

use just the import sub from Exporter

Need to increment $VERSION in ExtUtils::Manifest.

use warnings; convert "use vars" to "our"

add vim swap files to the ignored list for MANIFEST.SKIP.

RT 72933: Ignore files from prove

This should be automated!

Regen Configure after backports

File::Copy does not overwrite read-only files

refactor pp_ref

similar to commmit b3cf48215c

-removed:
  -4/-8 pop on SP
  +4/+8 push on SP
  PUTBACK
  1 non vol register save/restore (TARG not saved across the sv_ref())

TARG is not computed if the SV isn't a reference, so the PL_sv_no branch is
slightly faster. On VC 2003 32 bit miniperl, this func dropped from 0x6D to
0x58 bytes of machine code.

optimize PathTools to not try to load XS on miniperl

this will slightly speed up perl's build time by not loading XSLoader.pm
and DynaLoader.pm into the process, and then die()+eval trap when trying
to load the XS module.

Tony: update perldelta message to avoid a broken link

skip_all on darwin until nm peculiarities can be straighted out.

For: RT #122267

locale.c: Improve some comments

locale.c: Fix some unused code for potential future use

This code extends the heuristics used to determine if a locale is UTF-8
or not on older platforms.  It has been #ifdef'd out because it only
added a little value on dromedary.  Now the previous commit has added
new heuristics, and tests on dromedary show that this adds nothing to
that.  But I'm leaving it in the source in case it might ever prove
useful.  In order to test it, I compiled it and found some problems with
the earlier version that this now fixes.

locale.c: Add new heuristic for finding if locale is UTF-8

On older platforms that don't conform to POSIX 2001 nor C99, heuristics
are employed to try to determine if a locale is UTF-8 or not.  This
commit improves those heuristics by looking at names of the months and
days of the week to see if they are UTF-8 or not.  This is done if
looking at the currency symbol failed to help.

locale.c: White-space only

Indent and outdent blocks of code to conform to newly formed or removed
braces

locale.c: Refactor UTF8ness of currency symbol code

On older platforms that aren't C99 nor POSIX 2001, locale.c uses the
currency symbol to try to see if a locale is UTF-8 or not.  This commit
refactors it somewhat to make it cleaner, and which fixes several
problems.  The least issue was that it sometimes did a setlocale()
unnecessarily.  Others are that in some circumstances it called
localeconv() and/or looked at the result while within the wrong locale.

locale.c: Use ptr's value before freeing it, not after

This only affected runs with the -DL parameter to perl set.

locale.c: Use safer code practice

The interior-most function can return NULL.  Currently savepv() which is
the next outer function handles this correctly, as does the next outer
function, but it is dangerous to rely on that behavior.  So we test for
NULL before calling functions on a NULL ptr.

locale.c: Skip compiling fallback code on modern platforms

In the function that determines if a POSIX locale is UTF-8 or not, if
either nl_langinfo or MB_CUR_MAX are defined, it can reliably determine
the answer.  If they are not defined, it uses heuristics to figure
things out as best it can.  This code doesn't add value for those
platforms where one of the two symbols is defined, so can just be
ifdef'd out

util.c: Add comment

locale.c: name should be last resort when deciding if locale is utf8

Looking at if the currency symbol is UTF-8 should come ahead of looking
at the locale name.

locale.c: Prepare for rearrangement of code blocks

This section of code just returned generally,.  This commit changes it
so that it drops off the end if it can't determine if the current locale
is UTF-8 or not, so that additional tests can be added later.  The
function defaults to not UTF-8 if this drops off the end, so there
should be no functionality change

perl.h: Actually #define USE_LOCALE_TIME

This symbol is used, but was never defined

Upgrade IO-Socket-IP from version 0.30 to 0.31

(perldelta: Say we're upgrading from 0.29 since we've already upgraded from
that since 5.21.0.)

Remove IGNORABLE files from IO-Socket-IP as per 8db6555210

locale.c: Fix misplaced parenthesis

Commit a39edc4c877304d4075679b1d8de1904671a9c37 got a parenthesis
misplaced so it wasn't really looking at the next character, like it was
supposed to be doing

t/run/locale.t: setting to POSIX may be same as C

On some (? all) systems, setting the locale to POSIX may cause it to
actually set it to C.  In these tests we want to exclude C, hence we
have to exclude POSIX as well.

Some systems may not have any other locales, so this commit also
properly skips tests on those.

Thanks to Matthew Horsfall for figuring this out

t/loc_tools.pl: Don't return a locale empty string

A locale name shouldn't be an empty string; this makes sure
that some glitch that introduces one doesn't get passed on

ext/POSIX/POSIX.xs: White space only, align column

POSIX::setlocale(LC_NUMERIC) returns proper locale

Commit 4c28b29c7a1549f173177cdf69e91cc1acaa0969 broke calling setlocale
with a single parameter and hoping to get the correct LC_NUMERIC
property returned from it (including LC_ALL calls).  This is because the
numeric locale is now kept as C except around the code that wants it to
be the underlying locale.  It turns out that querying the current locale
is such a code area, so LC_NUMERIC is now set to the underlying locale
around this.

locale.c: White-space only

Outdent because the previous commit removed the enclosing block.

locale.c: Remove conditionals.

These two functions are supposed to normally be called through macro
interfaces which check whether they actually should be called or not.
That means the conditionals removed by this commit are redundant from
the normal interface.   By removing them, we allow the exceptional case
where the code should be executed unconditionally, to happen, by just
calling the functions directly, not using the macro interface.

locale.c: Keep better track of C/non-C locale

Perl uses three interpreter-level (but private) variables to keep track
of numeric locales.  PL_numeric name is the current underlying locale.
PL_standard is a boolean to indicate if we are switched to the C (or
POSIX) locale, and PL_local is a boolean to indicate if we are switched
to the underlying one.  The reason there are two booleans is if the
underlying locale is C, both can be true at the same time.  But the code
that is being changed by this commit didn't realize this, and
could unnecessarily set the booleans to FALSE.  This could cause
unnecessary switching of locales.

locale.c: Make a common idiom into a macro

lib/locale.t: Fix debug statement

This was printing using the wrong function.  It is a string, and should
use the string output function.

utf8.h: Add comment

Update which stable release series are now "officially" supported

Fix typo in op.h

This caused all OP structures to be larger than intended; for example, it
made `struct op` 48 bytes rather than 40 on Mac OS X x86-64.

[Merge] add support for ops to link to parent

This series of commits provides experimental support for ops in the optree
to be able to locate their parent op, if any.

First, it adds a new general-purpose function, op_sibling_splice(),
which allows for general manipulation of an op_sibling chain by analogy
with the perl splice() function. op.c has then been heavily refactored to
use this function rather than manipulating ops directly.

Then the boolean field op_lastsib has been added to OPs. Within the core,
this is set on the last op in an op_sibling chain (so it is synonymous
with op_sibling being null). By default, its value is set but not used.

In addition, a new build define has been added (not yet enabled by default),
-DPERL_OP_PARENT, that forces the core to use op_lastsib to detect the
last op in a sibling chain, rather than op_sibling being NULL. This frees
up the last op_sibling pointer in the chain, which rather than being set
to NULL, is now set to point back to the parent of the sibling chain (if
any).

Then add a C-level op_parent() function and B parent() method; under
default builds they just return NULL, under PERL_OP_PARENT they return the
parent of the current op.

Collectively this provides a facility not previously available from B:: nor
C, of being able to follow an op tree up as well as down.

Also, some assertions are added to finalize_op() that check the
consistency of the optree; in particular that op_last points to the last
sibling. Under -DPERL_OP_PARENT, more checks are enabled.

avoid adding two OP_HINTSEVAL ops

Under argless eval (i.e. defaults to $_) in the presence of hints,
the perl core was recursively calling ck_eval, which had the side-effect
of attempting to append an OP_HINTSEVAL op twice. Formerly due to the way
the appending was done, the first OP_HINTSEVAL op was silently discarded;
after doing things "properly" with op_sibling_splice(), both ops ended up
being appended, and at run-time, the hints hash would be interpreted as
the SV to eval.

This wasn't tested in the core tests, and just ended up sometimes failing
warnings.t or Digest-SHA tests (but never on my system!)

minor fixups to new op_sibling work

remove pTHX args to op_parent() and op_sibling_splice() since they don't
use them, and cast an op type assignment. Also, minor fixups to the pod
for op_sibling_splice().

Improve optree sanity checking code in finalize_op

The previous couple of commits added DEBUGGING code to finalize_op() that
checked, for BINOP/LISTOPs etc, that all the op's children had consistent
flags and that op_last pointed to the last op (and vice-versa under
PERL_OP_PARENT).  Extend most of this consistency checking to the various
classes of OP that have op_first but not op_last (like UNOP, LOGOP).