regcomp.c: Remove a cargo-culted #undef SPSTART

regcomp.c: Remove a #undef op

regcomp.c: Remove a useless use of I_STDARG

Module::CoreList export %delta and document it

test that $\ is localized and restored even if it was undef

This was broken before 4bac9ae47b5ad7845a24e26b0e95609805de688a
but there is no test, and it is not clear that we knew about
the bug or the fix.

RMG - Add a note about merging the release branch back into blead

I was momentarily confused and almost did the wrong thing when I saw that
"git merge release-5.19.4" had produced a merge commit (71d5a36340),
whereas the merge of 5.19.3 a month ago did not (see f865d60069).

The cause was simply that a change (ac239e1c3e) had been pushed to blead
since I'd created the release branch, which had not happened when merging
the 5.19.3 release branch last month.

Optimise if/unless wrt OP_AND/OP_OR/OP_DOR. Also optimise OP_OR/OP_DOR chains.

An OP_AND/OP_OR/OP_DOR in void context provides a short circuit
through ->op_other that can be used if AND/OR/DOR ops contained
within it jump out early. Use that short circuit.

Previously:

  $ ./perl -Ilib -MO=Concise -e 'if ($aa || $bb) {}'
  8  <@> leave[1 ref] vKP/REFC ->(end)
  1     <0> enter ->2
  2     <;> nextstate(main 3 -e:1) v:{ ->3
  -     <1> null vK/1 ->8
  6        <|> and(other->7) vK/1 ->8
  -           <1> null sK/1 ->6
  4              <|> or(other->5) sK/1 ->6              <-- Not optimised
  -                 <1> ex-rv2sv sK/1 ->4
  3                    <$> gvsv(*aa) s ->4
  -                 <1> ex-rv2sv sK/1 ->-
  5                    <$> gvsv(*bb) s ->6
  -           <@> scope vK ->-
  7              <0> stub v ->8

Now:

  $ ./perl -Ilib -MO=Concise -e 'if ($aa || $bb) {}'
  8  <@> leave[1 ref] vKP/REFC ->(end)
  1     <0> enter ->2
  2     <;> nextstate(main 3 -e:1) v:{ ->3
  -     <1> null vK/1 ->8
  6        <|> and(other->7) vK/1 ->8
  -           <1> null sK/1 ->6
  4              <|> or(other->5) sK/1 ->7               <-- Short circuited
  -                 <1> ex-rv2sv sK/1 ->4
  3                    <$> gvsv(*aa) s ->4
  -                 <1> ex-rv2sv sK/1 ->-
  5                    <$> gvsv(*bb) s ->6
  -           <@> scope vK ->-
  7              <0> stub v ->8

[perl #3112] Stop last from returning values

In push @a, last, it can try to return the @a, copying it like a sca-
lar in the process, resulting in Bizarre copy of ARRAY in last.

In do{{&{sub{"Just another Perl hacker,\n"}},last}}, it returns "Just
another Perl hacker,\n".

The former is clearly a bug.  The latter depends on a side-effect of
the same bug.

‘last’ really should not be trying to return the values that the same
statement has accumulated so far.

corelist.pl - Fix the addition of a new perl release following 9f92b9bec5

Now that the check is working, it was working slightly too well and actually
picked up 5.019005 in the %delta because that had been added already and
thus didn't add it to %released when it should have done!

One simple fix is to move the processing of %released so that it's done
first, which makes sense since it appears first in the file anyway.

Prepare Module::CoreList for 5.19.5 and bump its $VERSION

Module::CoreList 2.99 is now on CPAN

Bump version for 5.19.5

Add new perldelta for 5.19.5

Add 5.19.4 epigraph

Merge branch 'release-5.19.4' into blead

Use hv_fetch_ent() instead of hv_fetch() in S_finalize_op().

This makes the source code slightly shorter and clearer, and the object code
smaller.

Update perlhist for Perl 5.19.4

Finalize perldelta

Update Module::CoreList for Perl 5.19.4

Update RMG - Bump Module::CoreList* $VERSIONs sooner

The corelist.pl program picks up the $VERSION bumps for you if you bump
them first.

Fix line nums when multiline ${expr} spans here-doc

<<end . ${

end
"bar"};
warn __LINE__ # 3, not 5

This was caused by commit a49b10d0a, which make scan_ident in toke.c
reallocate the parser’s current line buffer (SvPVX(PL_linestr)) to
search for whitespace surrounding an identifier.

In case there is an arbitrary expression, it temporarily records the
line number and resets it at the end if that turns out to be the case.
However, it was not resetting PL_parser->herelines, which records how
many line numbers to skip when next incrementing it (to skip past
here-doc bodies).

So save and restore that value, too.

Fix parser buffer corruption with multiline *{...}

Since commit a49b10d0a, it has been possible for scan_ident in toke.c
to reallocate the parser’s buffer (SvPVX(PL_linestr)) when scanning
for multiline whitespace.

For the sake of those cases where it finds an arbitrary expression,
not just an identifier, it records a pointer to the first opening
brace, which it returns to the parser after finding out that there is
indeed an expression.

That pointer was not being updated when the buffer was being
allocated.

The solution is to record an offset, rather than a pointer, of the
opening brace relative to the beginning of the current line of input.

This one-liner:

$ ./miniperl  -e '*{' -e '         XS::APItest::gv_fetchmeth_type()' -e '}'

was giving me:

Unrecognized character \x80; marked by <-- HERE after 2<-- HERE near column 24 at -e line 2.

(There were nine nulls before the 2, but git stripped them out.)

perldelta - Wrap to 79 columns

perldelta for the prev. commit

Don’t free initial src from @INC sub too early

As noted in <20130919225357.GA18474@mars.tony.develop-help.com>, com-
mit 839a0e5 introduces uninitialized warnings (and sometimes other
warnings) in inccode.t.

What actually is happening is that \PVBM is no longer being inlined
and returning the same reference each time, but is returning a new
mortal reference.  Commit 839a0e5 uncovers an existing bug.

A sub in @INC can return a reference to a scalar containing initial
source code.  That returned value would be freed prematurely if it was
not referenced elsewhere.

We have to increment its reference count before leaving the scope, and
then mortalise it afterwards.

perldelta - A couple of corrections and tweaks

corelist.pl - Update RMG to reflect recent changes

corelist.pl - Prefer more recent versions where many have the same delta

When adding a new section to %delta in CoreList.pm and Utils.pm if we find
several previous versions which we can take the delta from then prefer the
most recent version. This has the effect that when adding the 5.019004
section to Utils.pm we now choose 5.019003 rather than a random one of
5.019000 (or 5.019), 5.019001, 5.019002 or 5.019003.

corelist.pl - Put the new utilities delta section in place

This includes removing any existing stub section for this $perl_vnum first.

corelist.pl - Fix make_coreutils_delta()

corelist.pl - Various tidy-ups

corelist.pl - Apply BinGOs's commit "Two" at updating Utils.pm

This is commit 402e920b67.

corelist.pl - Apply BinGOs's "First stab" at updating Utils.pm

This is commit bdb83d4b01.

corelist.pl - Correct the indentation of %delta's 'removed' entries

The indentation was only 11 characters instead of 12 to match the 'changed'
entries.

corelist.pl - Improve the updating of %delta and %deprecated

Both of these hashes needed to have the existing stub sections for the new
$perl_vnum manually removed prior to running corelist.pl otherwise a second
section for $perl_vnum got added. The script now deletes any such existing
section itself before adding the new (real) one.

corelist.pl - Fix the updating of %released

The code to not update %released if $perl_vnum is already in it didn't work
correctly. Fix this.

Teach configure.com about Perl_drand48 and friends.

Follow-up to 890c2948b6847.

Add newline to new switchd test for VMS.

On VMS, you're going to get a newline at EOF willy nilly, which
was making the expected output fail to match the actual output.
The simplest solution is just to put an explicit newline on the
print statement, which yields the same result everywhere.

Follow-up to 261cbad16f5ed8.

Upgrade CPAN::Meta from version 2.132510 to 2.132620

perldelta - 'nonexistent' should be 'non-existent'

perldelta - Remove all but one XXX notices

perldelta - Fill in currently remaining TODO items

9d32676e doesn't need one since it only fixes things that were added post
5.19.3, but [perl #117265] is worth mentioning somewhere so I've added that
to the warnings entry.

perldelta - Update entry for b29f65fce6

perldelta for 3428cdc0d3

perldelta - More copy-editing

perldelta - Un-TODO 9c7618be

As noted by Karl Williamson, it's very unlikely that any user would have
encountered the bug which this fixed, so it's not worth mentioning in
perldelta.

Add a USING_MSVC6 macro to identify Microsoft Visual C++ 6.0

This simplifies some of the logic necessary for coping with its various
problems.

Suggested by Nicholas Clark.

Remove Cwd from the "actually architecture dependant files" special cases.

This probably hasn't been needed since the file Cwd.pm was moved from lib/
into the directory containing its XS code.

Added is_core(), which returns true if the module was/is in core

Default to checking against $^V, but you can optionally specify
the perl release, and can also optionally specify a minimum
version of the module.

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

Remove 4 redundant #undefs, missed by commit 79be8fb4ac8fa995.

That commit removed all the code that defined PERL_NEED_MY_HTOLE64,
PERL_NEED_MY_LETOH64, PERL_NEED_MY_HTOBE64 and PERL_NEED_MY_BETOH64, but
missed removing code in perl.h which conditionally undefined them.

perldelta for 88e3936f.

Fix the VC6 build on Windows following commit e25d460c74

As noted in commit 38aa66aabf, VC6 on Windows does not support the
64-bit-int build option, and likewise is broken if we don't undef HAS_QUAD
in the core.

Update win32/config_H.[gv]c canned configuration files

The win32/config.[gv]c files are generally kept up to date these days (and
we have tests to check that) so no changes are required in them.

The win32/config_H.[gv]c files are regenerated as per instructions in the
win32/Makefile and win32/makefile.mk, being careful to restore a couple of
things otherwise lost from the config_H.gc file. The files are now in sync
with the top-level master configuration file, config_h.SH.

Add Toby Inkster to AUTHORS

Exclude perlfaq1, perlglossary, etc from the list of pragmata - these just pod, not modules.

Really fix precedence problem in IO::Socket::connect() from 80d2c56d79

Thanks to Dagfinn Ilmari Mannsåker for paying more attention than me.

perldelta - Correction to new slice syntax

Thanks to Michael Schroeder for the spot.

Make changes_between() in Module::CoreList API the same as the other functions

Fix precedence problem in IO::Socket::connect() from 80d2c56d79

Thanks to Andreas Koenig for the spot.

Merge branch 'hugmeir/disallow_cntrl' into blead

[perl #119123] disallow literal control character variables

This branch introduces a deprecation warning on variables like $^T,
where ^T is a literal control character in the source code.

It also reworks how the handling of whitespace in ${...} works,
fixing a number of bugs, such as ${\nfoo\n} not increasing the
line number.

toke.c, scan_ident: No need for a while loop when using PEEKSPACE

PEEKSPACE already gobbles up all the remaining spaces until it
either finds a non-space character, or EOF.

toke.c, scan_ident(): use PEEKSPACE() to skip over whitespace.

This fixes a number of bugs regarding whitespace and line numbers
in scan_ident(), such as ${\nfoo\n} not increasing the line number,
or ${\ntime\n[1]} not working.

It goes through a number of hoops to get the correct line number for
warnings emmitted from scan_ident, and reverts CopLINE to its
original value if scan_ident() is giving up and returning from the
point of the opening bracket, like in the case of ${\n\nfoo()}.

toke.c, S_scan_ident(): Don't take a "end of buffer" argument, use PL_bufend

All but one of scan_ident()'s callers already passed PL_bufend as
the removed argument; The one deviant was intuit_more(), which was
setting the "end of buffer" argument, to the next close-bracket.
This commit modifies intuit_more() to temporarily set PL_bufend and
then restore it.

This was done as groundwork for the following commit, which will add
more uses of PEEKSPACE() to scan_ident() in order to fix some whitespace
and line number bugs, and PEEKSPACE() modifies PL_bufend directly
if it encounters a newline at the end of the buffer -- that last bit
being why changing intuit_more() to modify-and-restore PL_bufend is
safe, since the end of the buffer will always be a ']'

[perl #119123] disallow literal control character variables

This introduces a deprecation warning on things like $^T, where ^T
is a literal control character in the source code.

toke.c, S_scan_ident: Skip over newlines as well as spaces and tabs.

Not doing this lead to some bugs, such as 'eval "*{\nOIN}"'
returning a glob for "\nOIN" rather than just "OIN", or
'eval "\${\cT\n}"' giving spurious syntax errors.

Note however that this is not a full fix, since that latter
test case still fails outside of an eval.

Reworked t/base/lex.t to use less hardcoded test numbers.

The later two thirds of the file now consistently use $test, which
should make future tweaks to the file simpler.

Merge the cast fixes that enable drand48 to work on 32-bit HP-UX builds.

Update the comment in util.h related to HAS_QUAD and U64TYPE.

Use U64_CONST() for the drand48 constants that are larger than 2**32.

Without this the constants are truncated to 32 bits on (at least) HP-UX
with 32 bit IVs, which results in some very wrong output from rand().

For PERL_CORE, don't undefine HAS_QUAD when building with 32 bit IVs.

When commit de1c2614758a00c0 added support for $Config{d_quad} and HAS_QUAD
in Oct 1999, it also undefined HAS_QUAD. It's been that way ever since.

This isn't actually correct on a platform with a 64 bit type available, but
changing it now will cause too much breakage on CPAN for too little gain
(See RT #119753). However, we do need HAS_QUAD in the core for use by the
drand48 code, so make the undef conditional on PERL_CORE not being set.

[perl #119539] clarify that copy() doesn't create the directory

bump File::Copy's $VERSION

add Nathan Glenn to AUTHORS

Added note about directories not being created with File::Copy::copy

perldelta for acb340504/#119501

perldelta for 650b1534

perldelta: Two to-dos that don’t deserve mention

9cefd268 [perl #118747] Allow in-place s///g when !!PL_sawampersand
c04e4151 op.c:leave_scope: use mg_free before sv_force_normal [#119295]

In both cases, it is a module relying on undocumented internals.

In the former, we do have an optimisation kicking in in cases where it
didn’t in 5.18 and earlier.  But those cases are rare enough that an
entry in ‘Performance Enhancements’ would be misleading.

In the latter, the order in which clearing and ‘free’ magic are trig-
gered is more predictable, but I don’t believe that has ever been
guaranteed, and only creepy modules will be affected by it.

depend on the new win32 sys/errno2.h header

Sort the contents of %Modules in Porting/Maintainers.pl

(No changes other than the ordering of the entries.)

perldelta for f1deee3312

perldelta - Three module upgrades which haven't come from new CPAN releases

Use non-dev version number in IO::Socket

There was no need for 80d2c56d79 to use a dev number. The IO distribution's
modules all have differing version numbers, mostly all well ahead of the
last CPAN release (UPSTREAM is 'blead' anyway), and no others are dev
numbers.

Upgrade parent from version 0.227 to 0.228

RT-119817 - Treat =back as the end of a warning description.

This keeps any trailing data in the file from showing up as part
of the last warning's description.

perldelta - Fill in one more item and add currently remaining possible TODOs

perlhack: Make MAINTENANCE BRANCHES a link

[perl #119501] \(1+2) always referencing the same sv

2484f8dbbb hid the fact that constant folding happens by making 1+2
fold to  a PADTMP, an SV that is never used as an lvalue and gets cop-
ied if one tries to use it that way.

The PADTMP mechanism is what allows \"$x" to return a new value each
time, even though ops like "$x" actually reuse the same scalar repeat-
edly to return values.

Because \ copies PADTMPs, \(1+2) ends up folding 1+2 to 3 (marked
PADTMP), which is then copied by the \ at compile time.  (Constant
folding works by evaluating certain ops at compile time and then
inlining their returned value.)  The result is that we have a folded
\3 where 3 is *not* marked PADTMP (the reference is, instead); hence
\(1+2) gives the same scalar each time, producing a value that can be
modified, affecting future evaluations.

The solution is to skip folding \ if its argument is a PADTMP.

Rewrap one perldiag entry for better splain output

The right margin is slightly less uneven this way (once the pod
formatting is stripped).

Upgrade CPAN from version 2.00 to 2.03-TRIAL

Merge fixes needed to build with 32 bit IVs and HAS_QUAD defined.

However, don't actually eliminate the #undef HAS_QUAD for 32 bit IVs, as
the costs (in CPAN breakage) look to be higher than the gains.
(See RT #119753 for more detailed analysis.)

Eliminate POPq, POPuq, TOPq, TOPuq, dPOPqv, dTOPqv, qPOPuqv, dTOPuqv.

These shortcut macros are unused in the core, and unused by any code on CPAN.
If any XS code we can't see *is* using them, it will now fail to compile, and
can easily be fixed by replacing the macros with their expansion.

The choice of 7 or 13 byte extended UTF-8 should be based on UVSIZE.

Previously it was based on HAS_QUAD, which is not (as) correct.

Use IVSIZE not HAS_QUAD to enable 'q' and 'Q' formats in pack.

Whilst the code for 'q' and 'Q' in pp_pack is itself well behaved if enabled
on a perl with 32 bit IVs (using SvNV instead of SvIV and SvUV), the
regression tests are not. Several tests use an eval of "pack 'q'" to
determine if 64 bit integer support is available (instead of
$Config{ivsize}), and t/op/pack.t fails many tests. While these could be
fixed (or skipped), unfortunately the approach of evaling "pack 'q'" is
fairly popular on CPAN, so the breakage isn't just in the perl core, and
might also be present in code we can't see or submit patches for.

Use IVSIZE not HAS_QUAD to enable "long long" formats in sv_vcatpvfn_flags().

Without this, enabling HAS_QUAD on 32 bit IV systems causes a lot of test
failures. Potentially those failures could be addressed, but it seems like
a lot more work for little gain.

Atari FreeMiNT: MiNT does not have .exe for _exe

[perl #89502]: Add support for Atari FreeMiNT platform

perldelta - Correction to new Win32 entry in Platform-Specific Notes

perldelta for #119857

add reference to maintenance branches in perlpolicy

perldelta - Document two new test scripts

perldelta for d4c027436f