Dual-life I18N::Collate

Add probe for sa_len availability in sockaddr struct

Sorry for the huge config_h.SH re-order. Don't know (yet) what caused that

perldelta for [perl #19135]

legacy.pm is no more

It has been removed in 1863b87966ed39b042c45e12d1b4e0b90b9cc071. feature.pm
replaces it.

perlfaq is not the only exception; just say "few"

Remove ; from use statements for readability. (Obeying the documentation
policy)

Typo fix

First draft of a documentation policy.

Slightly update perl.pod's "DESCRIPTION"

Don't explicitly talk about our support for DBM files as a feature.
It's not 1995 anymore.

remove more a2p/s2p from perl.pod

Remove a now-inaccurate statement about setuid scripts

Remove a paragraph from perl.pod pimping it as a sed/awk replacement.

Wordsmithing of Abigail's prose.

Added a statement about the examples in perl's documentation

Dual-life Devel::SelfStubber

Storable's t/malice.t now needs to use a minor version >4 ahead for testing.

The "safety margin" of 4 has been used up, now that Storable 2.25 can read
v2.8 files, but only writes out v2.4 on 5.004. All tests now pass on 5.004
(albeit with a bunch of warnings about ambiguous use of barewords, which are
no longer warnings).

Also, update $Test::Builder::Level to make diagnosing failing tests easier.

Refactor Storable::read_magic to avoid 4 arg substr.

This makes it fractionally shorter, and restores all compatibility with 5.004

Fix failing ExtUtils-CBuilder test

The change is also being applied upstream, but this will fix
tests until the next EU::CB release is merged to blead

Document use of strlen() by is_ascii_string(), is_utf8_string() and friends.

[perl #68658] attributes turn "state" into "my"

This is for two reasons:

• In S_my_kid, the attribute-handling code comes before the code that
  marks the padop as being a state instead of a my, which it knows to
  do based on the value of PL_parser->in_my. The attribute-handling
  code begins by setting PL_parser->in_my to FALSE, preventing the
  code that follows from doing its job.

  So now PL_parser->in_my is read at the top of S_my_kid, before the
  attribute code, with the statehood recorded in a boolean. Then the
  code that marks the padop as being state checks that boolean
  instead of in_my.

• A lexical variable declaration that has an attribute and is assigned
  to in the same expression compiles to something similar to:

    (attributes->import(... \$x ...), my $x) = 3;

  where the list is actually in scalar context, returning the my $x
  which is then assigned to (something that cannot be expressed
  directly in Perl syntax). So Perl_ck_sassign needs to take that list
  op into account when creating the ‘once’ op that actually makes
  state assignment work. Up till now it was just looking for a padsv
  on its LHS. This commit makes it check also for a list op whose last
  item is a padsv.

Revert change to AUTHORS #ea316027e

#2a25d7b already adds Paul Evans at the correct spot.

Fix file permissions mangled by #0e5d25bf

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

Upgrade Win32 from CPAN (from 0.39 to 0.40)

Yes, still has CR/LF line endings; I'll fix it in a CPAN
release with no changes but the line-endings fix first.

perldelta for 558b442/[perl #66104]

Modified unlink_all in t/test.pl to return the count of unlinked files This will make it so that it can be a drop-in replacement for unlink

Replaced '1 while unlink' with 'unlink_all' in t/uni/write.t

Replaced '1 while unlink' with 'unlink_all' in t/run/switches.t

Replaced 'unlink' with 'unlink_all' in t/re/qr.t

Replaced 'unlink' with 'unlink_all' in t/op/sysio.t

Replaced '1 while unlink' with 'unlink_all' in t/op/stat.t

Replaced 'unlink' with 'unlink_all' in t/op/magic.t

Replaced 'unlink' with 'unlink_all' in t/op/goto.t

Replaced 'unlink' with 'unlink_all' in t/op/filetest.t

Replaced 'unlink' with 'unlink_all' in t/lib/deprecate.t

Replaced '1 while unlink' with 'unlink_all' in t/io/perlio.t

Replaced 'unlink' with 'unlink_all' in t/io/nargv.t

Replaced '1 while unlink' with 'unlink_all' in t/io/fs.t

Replaced '1 while unlink' with 'unlink_all' in t/io/argv.t

Replaced '1 while unlink' with 'unlink_all' in t/op/write.t

[perl #19135] string eval turns off readonlyness on lexicals

Don’t turn off readonliness on lexicals when freeing pad entries.

The readonliness is (prior to this commit) turned off explicitly in
pad_free under ithreads. See also bug #19022, which resulted from the
same change. There is some discussion there, too, but nobody seemed to
know exactly why the readonliness needed to be turned off.

Change 4761/2aa1bed, from January of 2000, added that SvREADONLY_off.
It is supposed to make sure that pad entries that were constants will
not be constants the next time they are used.

Dave Mitchell writes:
> I think...[this]...fix is correct (just removing the SvREADONLY_off).
> The issue it was trying to fix appears to have been properly fixed
> later by 3b1c21fabed159100271bd60bac3f870f5ac16af, which is why it's
> safe to remove it.

So this commit just deletes that code.

In Storable.xs fix #80074, caused by the Perl stack moving when expanded.

cbc736f3c4431a04 refactored Storable::{net_,}pstore to simplify the logic in
their caller, Storable::_store(). However, it introduced a bug, by assigning
the result of do_store() to a location on the Perl stack, which fails if the
Perl stack moves, because it was reallocated. Fix this assumption, and add a
test which causes the Perl stack to expand during the call to do_store().

regexec.c: Fix locale and \s

The handling for locale \s and \S both assume that the character in
ASCII platforms at 0x20 is a space.  This is not necessarily so.

I'm guessing that the code was originally just copied and pasted from
the non-locale space handling code without thinking.  That code hard-coded
in the space character, probably to avoid an expensive swash fetch for a
common situation.

regexec.c: Add missing handlers for locale \d

regexec.c had some code to handle \d under locales, but not everywhere.

regcomp.sym: Correct DIGITL, NDIGITL entries

These were missing that they were simple (matching exactly 1 character)
and have 0 regnode arguments

regcomp.c: Add locale for \d

The DIGITL and NDIGITL regnodes were not being generated; instead
regular DIGIT and NDIGIT regnodes were even under locale.

This means no one has probably ever used Perl on a locale that changed
the digits.

toke.c: Fix EBCDIC problem

Commit 356979f4a7d780fd67a92a9ca6c8659bd12e7168 failed to include two
instances in toke.c that needed the same treatment, i.e., converting
properly from I8 to native.

toke.c: highlight problematic-mentioning comment

uni/fold.t: Prevent [] from being optimized out

This test hasn't been testing bracketed char classes because a single
character class gets optimized out.

regcomp.c: Revert to using regcomp.sym order

Now that the new nodes are grouped properly, we can use the fact that
the named backreferences all come after all the numbered backreferences,
as had been there before.

regcomp.sym: Re-order for better grouping

The recently added regnodes are moved to their respective equivalence
classes, and the named backreferences are moved to just after the
numbered backreferences

regcomp.sym: Remove misleading comments

Yves informed me that in spite of the comments giving precise node
numbers, those numbers can change, so new nodes can be mixed in with
their kin.  Remove those comments

Avoid setting PL_cv_has_eval unnecessarily

Quoting op.c:

/* /$x/ may cause an eval, since $x might be qr/(?{..})/  */

But the (?{..})’s compilation is only ever reached in the scope of
‘use re 'eval'’, so we can avoid setting PL_cv_has_eval (and the
slight overhead that entails) when that pragma is off.

[perl #66104] Bugs in extended regexp features

More precisely: Make run-time (?{...}) inherit pragmata.

This commit makes Perl_sv_compile_2op_is_broken (nice name!) copy the
hints from PL_curcop if invoked during run time. Usually they are
inherited from the code that is currently being compiled (which works
for $foo =~ /(?{...})/), but the code currently being compiled is not
the enclosing scope at run time ($bar = '(?{...})'; $foo =~ $bar),
hence the need for copying in a similar manner to pp_entereval.

Theoretically this code should also have to avoid copying a statement
label, but goto inside a regexp eval does not currently work, so I
cannot prove or disprove that yet.

The docs for SvRX and SvRXOK still refered to magic and the code snippet
was wrong.

Don't advertise PERL_HASH_SEED_EXPLICIT and NO_HASH_SEED

These (commented out) options had been advertised/documented in the
Makefile to make it easy to produce binary compatible builds of
Perl 5.8.x after the hash key randomization feature was introduced.
There is no reason to point out these customization option in
a prominent space.

Always build with crypt() support on Windows

The sources did support to drop win32/fcrypt.c from a Perl distribution
and build without a crypt() implementation to account for the paranoia
around distribution of encryption technology.  However sources and binaries
have been released for at least 10 years with all the code in place, so
there is not much point in making it configurable.

Uncomment and fix up tests at the end of Storable's blessed.t

These tests have never been "live", having been added, commented out, in 2004
as part of 754c00caaffe2ec6. The trick to make them work is to delete from the
symbol table, rather than merely attempting to undefine the subroutine.

Update ExtUtils-CBuilder to CPAN version 0.2800

  [DELTA]

  0.2800 - Mon Dec  6 16:05:46 EST 2010

    - No changes from 0.27_07

  0.27_07 - Wed Sep 29 21:48:55 EDT 2010

    Fixed:

    - Fixed t/02-link.t on perl < 5.8

  0.27_06 - Mon Sep 27 15:29:54 EDT 2010

    Fixed:

    - Preserves exit status on VMS [Craig Berry]

    - Fix Win32 split_like_shell escaping [Christian Walde]

  0.27_05 - Wed Jul 28 15:29:59 EDT 2010

    Fixed:

    - Tests no longer fail if user has set the CC environment variable

  0.27_04 - Mon Jul 26 22:41:43 EDT 2010

    Added:
    - handle c compiler and c++ compiler separately
      (adds requirement for IPC::Cmd) [Jens Rehsack]

    Others:
    - rely on File::Temp::tempfile and File::Spec::tmpdir to
      get unique file name for checking for compiler
      [Jens Rehsack]

    - Code base modernization and substantial code coverage improvments
      [Jim Keenan]

Perldelta for 22a3069

As contributed by Zefram on irc, with some pod markup added.

Get rid of PERL_MSVCRT_READFIX

The code works around a bug in very old versions of MSVCRT.dll.
The issue has been fixed a long time ago by Microsoft, so anyone
who has installed a Windows Service Pack in the last 10 years
or so won't be affected by the problem.

Remove dead Windows code

stopgap hack for $@ as unwinding reason indicator

Set $@ early in a die as well as late, so that it continues to function
as an unreliable indicator of whether unwinding in progress is due to
an exception.  This is a stopgap arrangement, until the unwinding process
can be introspected properly.

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

Clarify this is the Perl 5 interpreter for manpage and META

regcomp.c: Fix longjmp-related warnings

This patch should get rid of the compiler warnings recently introduced.

Another way to handle the pm_flags warning is to declare it to be
volatile, but not all compilers that perl uses apparently have that, so
I chose a longer way of introducting a new variable that isn't changed
within the jumpable area.  The others were fixed by not initializing
them before the jumpable area.

To-do tests for abnormal exiting from (?{})

Hideous to-do tests for (?{}) scoping issues

Un-TODO and un-SKIP the [perl #22354] tests

We can run these now that the regexp engine is reëntrant.

Emit warning for use re "/ul"

This was an omission on my part.

This should perhaps be an error, but I am just following what
‘use re’ already does with ‘use re "whatever"’.

regexec.c: Clean up initialized vars in switch

Commit cf5c578b88887f19f7fba519bbc12296 attempted to clean up d7ef4b73,
but did not get it quite right.  This fixes it, after consultation with
the author.

regcomp.c: small efficiency, portability fix

The code had hard-coded into it the ascii platform values for possible
start bytes.  There are macros that do that portably with no branches

regcomp.c: small efficiency improvement

The inline function repeats the test removed here.

regcomp.c: small efficiency gain

The 7-bit test operations always fail on non-ascii characters, therefore
it isn't needed to test each such character individually.  The loops
that do that and then set a bit for each character can therefore stop at
127 instead of 255 (the bitmaps are initialized to all zeros).  For
EBCDIC, the same applies, except that we have to map those 7-bits
characters to the 8-bit EBCDIC range.  This creates an extra array
lookup for each ebcdic character, but half as many times through the
loop.

For the complement of the 7-bit operations, we know that they will all
be set for the non-ascii characters.  Therefore, we don't need to test,
we can just unconditionally set those bits.  It would not be a good idea
to just do a memset on that range under /i, as each character that gets
chosen may have its fold added as well and that has to be looked up
individually.

regecomp, regexec: Use mnemonic character names

This patch replaces hex ordinals by macros containing the character
names, for clarity and portability to EBCDIC.

utf8.h: Need to make sure macro result fits in byte

The UTF8_TWO_BYTE_HI_nocast() macro has an error in it, in that the
START_MARK is larger than a byte, and only the last 8 bits of it are
relevant.  This hasn't caused a problem because the macro hasn't been
called directly, but from other macros that make sure the result gets
cast to a U8.

utf8.h: Add whitespace for readability

regcomp.c: Move code out of longjump area

This code should be done before the setjump to avoid the longjump
clobbering it.

[perl #80220] order of hash keys wrong in perlfunc.pod

Add Peter Holzer to AUTHORS

perldelta for Storable saving a code reference as UTF-8

Up Storable version to 2.25

Up Storable minor version

The change for storing UTF-8 coderefs necessitates this change.

Make Storable correctly store coderefs with UTF-8 flag

make empty string regexp stringify to the same thing regardless of unicode flags

make the jump point a little more obvious in a comment

Remove the Makefile command to touch perlmain.c after running makedepend.

Jerry D. Hedden was right to question whether this was now superfluous, because
as best I can tell it was *always* superfluous. Perl 5.000 shipped with a
Makefile rule to build perlmain.c using a shell script, with a dependency for
perlmain.c on makefile. [Lowercase makefile, now abstracted as $(FIRSTMAKEFILE).
This is generated by makedepend from Makefile, by calculating the dependency
rules to append to it.] The rule to generate perlmain.c had a prerequisite on
$(FIRSTMAKEFILE), and in turn $(FIRSTMAKEFILE) has a dependency on $(c), which
included perlmain.c. Hence there was a circular dependency, and the "solution"
to avoiding repeated rebuilds was to touch perlmain.c (if it exists), after
running makedepend.

As best I can tell there is no *actual* dependency for the correct generation
of perlmain.c on the contents of $(FIRSTMAKEFILE), as the relevant variables
are the same in both Makefile and $(FIRSTMAKEFILE). Hence this command should
have been removed, along with the dependency, years ago. However, no-one
spotted this until Jerry observed that fbcaf61123069fe4 changed the build
rules, and removed the dependency, but not the touch command.

[perl #71286] fallback/nomethod failures

This fixes two bugs related to overload and fallback on binary ops.

First, if *either* of the args has a 'nomethod', this will now be used;
previously the RH nomethod was ignored if the LH arg had fallback value
of undef or 1.

Second, if neither arg has a 'nomethod', then the fallback to the built-in
op will now only occur if *both* args have fallback => 1; previously it
would do so if the *RHS* had fallback => 1. Clearly the old behaviour was
wrong, but there were two ways to fix this: (a) *both* args have fallback
=> 1; (b) *either* arg has fallback=> 1. It could be argued either way,
but the the choice of 'both' was that classes that hadn't set 'fallback =>
1' were implicitly implying that their objects aren't suitable for
fallback, regardless of the presence of conversion methods.

This commit doesn't really need a delta entry

Remove lots of unused, VMS-specific variables.

Brought to you by -Duser_c_flags=/WARN=(ENABLE=LEVEL5,INFORMATIONAL=ALL)

Fix K&R-style function declaration.

Remove unused include file from vms/vms.c.

Document the explicit * in tie *$handle

perldelta for the tie $handle changes

Deprecate tie $handle without *

Fix up tests to work around tie $handle

Revert "[perl #77496] tied gets scalars and globs confused"

This reverts commit 8752206e276cffe588c0932b5a9f2331640e8447.

Revert "[perl #77688] tie $scalar can tie a handle"

This reverts commit 9aa8b00892d81bb5e94565d3cb9841dd57b7b9cf.

Revert "Make untie check the FAKE flag on globs"

This reverts commit 830748013f81bcc28d145baf4024efd1b6537704.