regexec.c: Split EXACT, folding nodes in regrepeat

As I started working on fixing more bugs in regrepeat, I realized that
the EXACT node had enough different things going on from the folding
nodes that it was better to give it its own case statement.  This patch
does this and refactors the remaining code to compensate, and for
clarity.

PL_fold wrong for EBCDIC platforms.

The PL_fold table map on EBCDIC only works on the ASCII-subrange
characters, not the full native Latin1.

To fix this, I moved the table to utfebcdic.h for EBCDIC platforms, and
actually changed it to three tables, one for each of the code pages
known to Perl.

There is no EBCDIC platform available to test on.  What I did was hack
together a program from existing code that does EBCDIC transforms.  I
ran it in ASCII mode, and verified that the generated table was
identical to the Latin1 table I had previously constructed by hand and
extensively tested.  I then ran it on each of the three EBCDIC
transforms, and verified that each matched the places in the original
table that I knew were correct, all the ASCII alphabetics, the controls,
and a few other code points.

So these tables are at least as correct as the existing one, as they are
identical to it for [A-Z], [a-z].

perl.h: Expand comment

re/pat.t: Skip tests on EBCDIC

There's no convenient way to translate to EBCDIC in these tests, since
they don't use the normal test routines which have this facility.
Therefore, have to skip these tests on those platforms.

regcomp.sym: Fix descriptions

requires regen

regex free up bit in ANYOF node

This patch causes all locale ANYOF nodes to have a class bitmap (4
bytes) even if they don't have a class (such as \w, \d, [:posix:]).
This frees up a bit in the flags field that was used to signal if the
node had the bitmap.  I intend to use it instead to signal that loading
a swash, which is slow, can be bypassed.  Thus this is a time/space
tradeoff, applicable to not just locale nodes: adding a word to the
locale nodes saves time for all nodes.

I added the ANYOF_CLASS_TEST_ANY_SET() macro to determine quickly if
there are actually any classes in the node.

Minimal code was changed, so this can be easily reversed if another bit
frees up.

Another possibility is to share with the ANYOF_EOS bit instead, as this
is used just in the optimizer's start class, and only in regcomp.c.  But
this requires more careful coding.

Another possibility is to add a byte (hence likely at least 4 because of
alignment issues) to store extra flags.

And still another possibility is to add just the byte for the start
class, which would not need to affect other ANYOF nodes, since the EOS
bit is not used outside regcomp.c.  But various routines in regcomp
assume that the start class and other ANYOF nodes are interchangeable,
so this option would require more code changes.

regcomp.h: Add comment

regcomp.c: Remove references to old #define

Two #defines were recently collapsed to mean the same thing.
Standardize on using one of them.

regcomp.h: Reorder statements for clarity

Reorder #defines of bits so are in numerical order

Newly-created stashes may need effective names added

%x can't be used directly with pointers.

(I keep forgetting this.) Change the four occurrences in the new custom
op tests to UVxf with a PTR2UV around the pointer.

Remove redundant check

There is a similar check about fifty lines back.
And it is silly to check HvARRAY on a named hash.

mro_package_moved must act on all effective names

See the test case in the commit. It passes in 5.8.x and blead (as of
this commit), but not 5.10-5.13.7.

In every case the name to be passed to mro_gather_and_rename is cre-
ated using an SV, so we might as well pass that instead of extracting
the char array and length from it.

That allows us to pass an AV instead, if there are multiple names to
take into account.

Move the test for Win32's FindExt.pm to t/porting, and run it automatically.

The test was written as part of the work on migrating modules to cpan/ and
dist/, but at that time at least one of FindExt.pm and Configure was buggy with
the classification of at least one module (XS or non-XS). As that is now fixed,
move the test to t/porting, and run it routinely. This will ensure that the
Win32 code's classification of modules will remain consistent with Configure's.

Add flags param to hv_ename_*

We will need this for making the API UTF8-aware in 5.16
or whenever.

‘If you are upgrading from 5.13.6...’

Only clean up the generated perldelta

S_hfreeentries: keep OOK off unless adding something to aux

This small optimisation allows hv_undef to skips its if(SvOOK()) block
and all the checks inside it much of the time.

mktables: Clean up warning msg

This message actually printed, and I discovered the various components
ran together, needing white space to make it more readable.

mktables: comment typo and expand other comments

mktables: Smaller tables if not -DDEBUGGING

This patch frees up about .5Mb of disk space.  First, it omits an
unnecessary trailing tab in the output tables, at the cost of an extra
test.  Secondly, for non-debugging Perls, it omits the
helpful-to-developer comments at the beginning of each table file, and
changes the default to not output the other helpful-to-developer
comments that give the number of code points in each output range in the
tables.

Revert "Globs that are in the symbol table can be unglobbed"

This reverts b9e00b79 except for the tests.

This extra checking and saving of the FAKE flag is no longer necessary
as of the previous commit.

[perl #77926] Glob reification during localisation

This was supposed to have been fixed by 2acc3314e31a9, but the code it
added was in the wrong spot. (This is the code that makes *{} return a
new glob if its argument is FAKE.)

Consequently, local *$foo was localising $foo, not *$foo.

This changes the order of the statements and adds a test.

Correct test count in UCD.t

Increase Unicode'UCD::s version

UCD.pm: Add info about named sequence alternatives

The namedseq function is essentially obsolete, as the core has better
incorporated its abilities.  This adds documentation as to the
alternatives.

UCD.pm: Don't use CompositionExclusions.txt

The motiviation for this patch was to remove dependence of UCD on
another Unicode DB .txt file.

But the subroutine that uses it is out-of-date, now that this property,
and an even more convenient one are accessible from the core.  So the
documentation is also updated to educate people.

Instead of using the file, the routine just uses the core's access
method

UCD.pm: Don't use NamedSequences.txt, saves disk

This changes UCD to not use this file.  Instead it takes advantage of
the recent addition of named sequences being accessible through the \N{}
construct.  In one case where it returns a hash of all the named
sequences, it uses the same .pl file that \N{} does.  My guess is that
this routine's usefulness is now past, as named sequences are now
incorporated into the core.

Eliminate the newname param from mro_package_moved

Nothing is using this any more, as of the previous commit.

Make hv_undef leave HvENAME alone

unless called from sv_clear.

This is necessary as and undeffed stash, though it nominally becomes
just a plain hash and is not a stash any more, is still to be found
in the symbol table. It may even be in multiple places. HvENAME’s
raison d’être is to keep track of this. If the effective name is
deleted, then things can get out of sync as the test in the commit
demonstrates. This can cause problems if the hash is turned back
into a stash.

This does not change the deletion of the HvNAME, which is the only
difference between hv_clear and hv_undef on stashes that is visible
from Perl. caller still returns (unknown) or __ANON__::....

I tried to make this into several small commits, but each part of it
breaks things without the other parts, so this is one big commit.

These are the various parts:

• hv_undef no longer calls mro_package_named directly, as it deletes
  the effective name of the stash. It must only be called on sub-
  stashes, so hfreeentries has been modified to do that.

• hv_name_set, which has erased the HvENAME when passed a null arg
  for the value ever since effective names were added (a special case
  put it just for hv_undef), now leaves the HvENAME alone, unless the
  new HV_NAME_SETALL flag (set to 2 to allow for UTF8 in future)
  is passed.

• hv_undef does not delete the name before the call to hfreeentries
  during global destruction. That extra name deletion was added when
  hfreeentries stopped hiding the name, as CVs won’t be anonymised
  properly if they see it. It does not matter where the CVs point if
  they are to be freed shortly. This is just a speed optimisation, as
  it allows the name and effective name to be deleted in one fell
  swoop. Deleting just the name (not the effective name) can require a
  memory allocation.

• hv_undef calls mro_isa_changed_in as it used to (before it started
  using mro_package_moved), but now it happens after the entries are
  freed. Calling it first, as 5.13.6 and earlier versions did, was
  simply wrong.

• Both names are deleted from PL_stashcache. I inadvertently switched
  it back and forth between the two names in previous commits. Since
  it needed to be accounted for, it made no omit it, as that would
  just complicate things. (I think PL_stashcache is buggy, though I
  have yet to come up with a test case.)

• sv_clear now calls Perl_hv_undef_flags with the HV_NAME_SETALL
  flag, which is passed through to the second hv_name_set call,
  after hfreeentries. That determines whether the effective names
  are deleted.

• The changes at the end of hv_undef consist of pussyfooting to avoid
  unnecessary work. They make sure that everything is freed that needs
  to be and nothing is freed that must not be.

hv_undef .= _flags

Add flags param to hv_undef.

There is no mathom, as the changes that this will support
are by no means suitable for maint.

Fix hv_name_set when there is an HvENAME

This code was completely wrong and could even crash. This is not cur-
rently reached.

Add the new shiny perldelta file I neglected to add.

Update TOC for perl5.13.8

Created perldelta for v5.13.8

v5.13.7 epigram added.

Update the totals in perldelta with the last few commits

Add QNX Neutrino RTOS to list of platforms in perlport

Update Module::CoreList for v5.13.7

make regen; make regen_perly

Ahem. *Final* finalisation.

There was still an XXX section.

Finalisationtion of perldelta with updates for committers and stats

Fix a perldelta typo

perldelta for 249534c3

First pass at finalising perldelta, removing unrequired boilerplate

Bump the perl version in various places for v5.13.7

Bump Data::Dumper version as previous commit made a change

Typo in perldelta

Fix the test fix for older perls (5499dc3d6e47ea78). We're still on 5.13.6

Skip [perl #72332] DD test on 5.10.1

$ perl5.10.1 -le 'print *{*STDERR{IO}}'
Bus error

That crash is beyond DD’s control, so the test should be skipped.

Compare to lib.pm 0.63 on CPAN

Update mailing list and website for cpantesters in perlport

Update Maintainers.pl with new releases

'Upgrade' to AutoLoader 5.71

That release was a core => CPAN sync, so this is just a version bump.

Fix Data::Dumper test on old perls

Bump Math::BigInt::FastCalc to dev release as per Florian's request

Bump Math::BigRat to dev release as per Florian's request

Sync version of dist/Math-BigInt modules to 1.99_01 as per Florian's request

Add Unicode's minor 6.0 correction

Unicode just released a minor correction to 6.0.  This patch adds that.
Unfortunately, a test thought the old behavior was correct, and so had
to be modified as well.

mktables: Leave Unicode1 BELL name alone

It occurred to me that I was changing both the name and Unicode 1 name
fields, and only the name field is in conflict.  So change just it.

perluniprops.pod: nits

It is generated by mktables

Update to perlpolicy.pod to clarify that minor updates to dual-life
modules to fix build/install/test issues are ok

perdleta tpo fxies

Update the perldelta for ExtUtils::Constant.

Also note the size shrinkage of B's shared object, which is partly due to now
using code generated by ExtUtils::Constant.

Update perlhist, adding Miyagawa to 'THE KEEPERS OF THE PUMPKIN' and updates for v5.13.7

perldelta up to f869c5854

All I did was review the changes and add 249534c34 to the list of
unrecorded commits.

The most recent MRO changes are not noteworthy.

perldelta: clarify tie $glob

perldelta for custom ops

Work-around Uni 6.0 issues with 'BELL'

Unicode version 6.0 has co-opted the name BELL for a different character
than traditionally used in Perl.  This patch works around that by adding
ALERT as a synonym for BELL, and causing a deprecated warning for uses
of the old name.

The new Unicode character will be nameless in Perl 5.14, unless I can
(unlikely) get Unicode to grant a synonym that they will support.

charnames.t: indent newly formed block

This is a white-space only patch to indent the code that was put into an
if block by the previous commit

charnames.t: PERL_RUN_SLOW_TESTS runs more tests

This patch makes this .t look for this environment variable, and if set
run more tests.  There are two levels of setting, as explained in the
comments

More updates to point to Unicode 6.0

Unicode 6.0 DB

mktables: Upgrade to handle new Unicode 6.0 tables

Update Archive-Tar to CPAN version 1.72

  [DELTA]

  This merely brings the core and CPAN versions into sync.
  No code changes apart from version update.

Note that support for building with VC++ 2010 is not yet complete

perldelta entry for 8c8488cd4fce90cb5c03fb3f89e89c05e5275498.

Refactor newATTRSUB()'s logic for grafting a sub definition to an existing stub

Previously it was using cv_undef() to (partially) free the target CV (the
pre-existing stub), before donating it the padlist and outside pointers from
the source CV (the definition, just compiled), and then freeing up the remains
of the source CV.

Instead, explicitly exchange padlist and outside pointers, explicitly assign
other fields that need changing (file and stash), and assert that various
CvFLAGS are as we expect them.

Test that the stash and file of a sub's definition overrides those of its stub.

This is the current behaviour for Perl_newATTRSUB(), and it turns out that we
have no test for it.

Test that lack of prototype on a sub's definition overrides any on its stub.

This is the current behaviour for Perl_newATTRSUB(), and it turns out that we
have no test for it.

Update Changes file ahead of v5.13.7 release

Update perldelta to note the Visual C++ 2010 support

Support Visual C++ 2010

Also add a couple of things to win32/makefile.mk which were only previously
applied to win32/Makefile

Make perlio line buffer VMS record-oriented files on output.

When perlio flushes down to the unix layer, it can introduce a
spurious record boundary when writing to a record-oriented file.
Perl may create such files when doing edit-in-place or any other
context where the file format is inherited from a previous
version of the file.

The problem can be eliminated by enabling line buffering on such
files when they are opened.  This was a regression in 5.10.0 since
before that stdio's buffering performed the same function.

N.B.  Lines longer than the size of the perlio buffer will still
result in multiple records -- a larger buffer may be necessary.

For more details and discussion see:

http://www.nntp.perl.org/group/perl.vmsperl/2010/11/msg15419.html

Thanks to Martin Zinser for the problem report.

Don't install unnecessary unicore files

Many of the files in lib/unicore are no longer needed
after testing.  This patches installperl to skip those
files, cutting down the installed size of unicore
from about 17M to about 7M.

In Perl_cv_undef(), only check potential pads against PL_comppad

Don't even try checking the address of the pad name AV against PL_comppad, and
don't try checking the address of pad AVs against PL_comppad_name. Neither will
ever match.

In S_pad_check_dup(), no need to check the 0th name entry.

The 0th entry in a pad is for @_, and the name corresponding to it is NULL,
so save a check.

Convert newSUB() to a macro wrapping Perl_newATTRSUB()

Provide a Perl_newSUB() function in mathoms.c for anyone referencing it by its
full name.

Spelling/grammar nits

Create proper Cygwin $ENV{PATH} in test.pl

For Cygwin, runperl() in test.pl adds ':/bin' to $ENV{PATH}.  However, if
$ENV{PATH} is initially empty, the leading colon causes the following error:

Insecure directory in $ENV{PATH} while running with -T switch at ./test.pl line 566.

This fix sets $ENV{PATH} to just '/bin' when it's intially empty.

Get cpan/CGI/t/http.t working on VMS again

This stopped working with the 84601d63a (the 3.50 upgrade).

This patch was not applied to the CGI.pm repository until
after the new release.

Clean up multiplying perldelta deletions.

Only the one copied as part of the build should be removed in the
clean target.  TODO: buildtoc is probably busted.

Remove unnecessary check in mro_package_moved

This was supposed to have been removed by 80ebaca, when negative val-
ues for the newname_len argument stopped being used.

Don’t skip mro_package_moved if the parent stash is renamed

This stops S_hv_delete_common from skipping the call to
mro_package_moved if the HvNAME of the stash containing the deleted
glob is no longer valid, but the stash is still attached to some other
part of the symbol table.

Inline Perl_pad_undef() into its only caller, Perl_cv_undef().

Perl_pad_undef was never in the API, and has no users (elsewhere in core,
on CPAN, or anywhere else visible to Google codesearch).

Move Perl_cv_undef() from op.c to pad.c

This will allow the non-API function Perl_pad_undef to be inlined into it.

Add Grant McLean to AUTHORS, following commit a27417a027da2da0.

Update references to targrep to ptargrep in ptargrep

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

Note in perlfaq2 that  perl.com is no longer part of O'Reilly

In Perl_pad_new(), allocate a 2 element array for padlist.

Most subroutines never recurse, hence only need 2 entries in the padlist
array - names, and depth=1.  The default for av_store() is to allocate 0..3,
and even an explicit call to av_extend() with <3 will be rounded up, so we
inline the allocation of the array here.

Running ./installman allocates 7K less with this change.