RT #73520: POSIX::strftime memory leak

Tested before/after with valgrind.

Die with $@ instead of empty message

Remove extra/useless $@ check after eval { require PadWalker } (which is still checked)

Promote eval( "require ..." ) to eval { require ... }

Promote eval { require( ... )} || die to mere require( ... )

Remove indirect object notation from debugger

Document that @{$main::{'_<'.$filename}} lines are dualvar to (COP*).

In t/harness, clear PERL5LIB, PERLLIB, PERL5OPT as t/TEST does.

In fact, as t/harness requires t/TEST, simply get t/TEST to do it for
t/harness too.

Silence compiler warning

Signed-off-by: David Golden <dagolden@cpan.org>

PATCH: t/lib/common.pl localized $/ setting; allow input 'no_plan'

# New Ticket Created by  karl williamson
# Please include the string:  [perl #76398]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=76398 >

See attached patches

>From 275169a8df5851331669da2dd791483379c3fc5e Mon Sep 17 00:00:00 2001
From: Karl Williamson <khw@khw-desktop.(none)>
Date: Tue, 6 Jul 2010 09:20:42 -0600
Subject: [PATCH] t/lib/common.pl: localize changing $/

$/ changes should not affect callers.

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

Fix typo

Reorganize dist/threads-shared

Move dist/threads-shared/shared.pm to dist/threads-shared/lib/threads/shared.pm
and remove its Makefile.PL.

Get rid of PERL_DECL_PROT

It was added for PERL_OBJECT support in commit 0cb9638,
which has been removed again with commit acfe0ab.

Reorganize dist/threads

Move threads.pm to lib/threads.pm and remove Makefile.PL

cautionary note about "no VERSION"

Signed-off-by: David Golden <dagolden@cpan.org>

Change of address for Bo Lindbergh

Signed-off-by: David Golden <dagolden@cpan.org>

Add Bo Lindburgh alternate email to checkAUTHORS.pl

Code for allowing uppercase X/B in hexadecimal/binary numbers (#76296).

Signed-off-by: David Golden <dagolden@cpan.org>

Tests for allowing uppercase X/B in hexadecimal/binary numbers (#76296).

Signed-off-by: David Golden <dagolden@cpan.org>

Clean up new Locale-Maketest test for use in core

Locale::Maketext external cache support

This patch with tests provides RO support for lexicon hashes in
Locale::Maketext.  This allows you to have GDBM language files owned by
root which can be accessed by non-root, but not altered.

If your lexicon is a tied hash the simple act of caching the compiled
value can be fatal.

For example a GDBM_File GDBM_READER tied hash will die with something
like:

  gdbm store returned -1, errno 2, key "..." at ...

All you need to do is turn on caching outside of the lexicon hash itself
like so:

  sub init {
      my ($lh) = @_;
      ...
      $lh->{'use_external_lex_cache'} = 1;
      ...
  }

And then instead of storing the compiled value in the lexicon hash it
will store it in $lh->{'_external_lex_cache'}

I've verified that blead is the authoritative location for
Locale::Maketext source.

Signed-off-by: David Golden <dagolden@cpan.org>

replace spaces with tabs in MANIFEST

Don't set strict for 'no 6;'

Commit faee19b51573e81abe8811f1256a1d27777d6d04 was incomplete
and only stopped features from being enabled under 'no'.

This patch merges all 'use N.NN' type logic into a single if clause
and then checks version numbers in separate if statements within it.

Ignore generated files from Time-HiRes

Ensure new arg to watchdog() in t/test.pl gets initialized

added a "list" command to list open tickets that might have patches

we can't depend on patches having filenames. Thanks git!

First pass at a tool to help porters apply RT patches from the
commandline

In Perl_lex_start(), use newSVpvn_flags() to reduce source and object size.

In pp_regcomp and pp_entereval, use newSVpvn_flags() to simplify code.

Speed up viacode

Capturing parentheses greatly slow down regexes, at least here.
On my machine, viacode took 27 seconds for the 22K Unicode names without
capturing parens; 45s with.

charnames.t update because of rebase

Use of t/lib/common.pl caused some glitches; some behaviors of the
underlying is() functions changed, so revised .t to work under this
scheme.

Correct perldelta for these changes

Add vi hint for non-std format of charnames.pm

More charnames pod updates

charnames: check for use bytes in vianame; efficiency

When vianame returns a chr, it now verifies that it is legal under 'use
bytes'.  Update .t

An instance of taking of a substr of a huge string is needed only in an
error leg.  Move it to that leg for performance.

And make the message a subroutine so will be identical whenever raised.

Clean up charnames pod, including new changes

This patch brings the charnames pod up-to-date, and rewords it to
hopefully be more clear.

Clean up viacode, accept large aliases

This changes viacode to accept aliases that the user has defined beyond
the Unicode range.

Extend \N{} enhancements to vianame()

This patch refactors charnames so that vianame and \N call the same
common subroutine so that they have as identical behavior as possible.

Bump version; some pod cleanup

charnames: add CORE:: to hex()

Other programs do this; I don't know why just hex() needs to be
protected from user override, but I'm just copying prior art.

Abandon plans to change viacode's return of unassigned

The BUGS section of the charnames pod said that it was a bug to return
undef for unassigned characters, whereas the real Unicode name is the
empty string.  demerphq noted that undef stringifies to the empty
string, so we are in fact in compliance with the standard.  This
clarifies the pod wording, removing the text from the BUGS section.

Allow defining custom charnames to ordinals

This adds the ability of a user to create a custom alias that maps to a
numeric ordinal value, instead of an official Unicode name.

The number of hashes went up so that is better to refer to them by a
name than a number, so I renamed them.

Also, viacode will return any defined user's alias for an otherwise
unamed code point.

This change is principally so that private use characters can be named
so it is more convenient to use them in Perl.

Add clarifying comments to charnames.t

Reword feedback request.

It's not clear to me what should be done about the problem of vianame
being bipolar.

Add a number of abbrs and variants to \N{}

This patch adds the standard abbreviations for the control characters
(such as ACK, BEL, etc) to the repertoire that \N{} knows about.  It
also adds a few common variants of their full names, and the old names
for the 4 controls that Unicode has chosen not to have any names at all
for.

The patch also adds all the abbreviations that Unicode lists in 5.2 for
longer characters, such as NBSP, SHY, LRE, ...

To preserve complete backward compatibilty for these and future changes,
user-defined aliases are now checked first, before these are.

As a performance enhancement, these aliases are mapped to their actual
code values instead of their full names which then had to be looked up
in the large table.  Now that is avoided, and the table is not loaded
at all until a name is encountered that is not one of these aliases.

The pod and .t are updated.

Remove a release_manager_guide step since it's now been automated away

Update Makefile to regen META.yml as it regens other files in regen_all

Make the META.yml regen code use regen_lib.pl per the request in release_mananger_guide,

Missing AUTHORS will now generate failing tests. Removing the manual
checkAUTHORS step from the release engineering guide.

Make checkAUTHORS a test to further reduce the release burden

Further refactoring of checkAUTHORS

refactoring checkAUTHORS in advance of testifcation

Bring AUTHORS up to date

Fix Digest-MD5 tests to run properly under core

Update Digest-MD5 to CPAN version 2.40

  [DELTA]

  2010-07-03   Gisle Aas <gisle@ActiveState.com>

    Release 2.40

    Marc Pignat (1):
      Safer alignment test [RT#35823]

    Robin Barker (1):
      consting in new ext/

    Gisle Aas (1):
      Remove the MacOS branch of this test

remove trailing space in cmpVERSION

Make Test::Harness source_handler.t use the build perl (%PERL_CORE%) on Win32.

Add Matt Johnson to AUTHORS.

docs: updates for git-based cmpVERSION.pl

Update the release_managers_guide.pod with the new command usage,
and remove the todo item

Update Porting/cmpVERSION.pl to use git

Update Porting/cmpVERSION.pl to take a directory and a git tag,
rather than two directories, as suggested in perltodo.

Make previous test fix conditional on the platform being Windows

Force t/io/openpid.t to use the alarm() watchdog strategy.

* The default watchdog strategy on Win32 doesn't work because the watchdog
is executed via subshell ("cmd /c perl -e '...'"). When the test finishes
and tries to kill the watchdog all it manages to do is whack cmd.exe and
the watchdog still fires, potentially killing an innocent process.

* Trying to use the fork() strategy instead causes openpid.t's test #9's
"ok" to be lost occasionally.  The message itself is generated by the 4th
child of the test and should go directly to stdout, but sometimes the
output vanishes for unexplained reasons.  This doesn't appear to happen
without the watchdog enabled.

* Using the alarm() watchdog strategy seems to not cause any badness.

add some comments to pp_concat

make it clearer what type of concat each code branch handles

make_patchnum.pl: handle not on a branch properly

avoid multiple FETCH/stringify on filetest ops

some of the filetest operators could call mg_get and/or overload fallback
stringify multiple times

add my_[l]stat_flags(); make my_[l]stat() mathoms

my_stat() and my_lstat() call get magic on the stack arg, so create _flags()
variants that allow us to control this. (I can't just change the signature
or the mg_get() behaviour since my_[l]stat() are listed as being in the
public API, even though they're undocumented.)

PL_amagic_generation doesn't show overload loaded

PL_amagic_generation is non-zero even without the presence of
'use overload', so don't bother using it as a short-cut test of
whether we can skip AMAGIC processing

fix bad indentation in pp_regcomp

avoid extra FETCHes on overloaded qr stringify

/$tied/ called FETCH too many times if the FETCH returned an overloaded
object with no qr method, but with stringify fallback

overload.t: clarify concat #FETCH expected

It turns out that the number of FETCHes for the fallback ($tied_ovld . foo)
just needed explaining, not fixing.

remove double stringify-overload from $ovld .= foo

There was a piece of code in pp_concat who's job it was to determine the
UT8ness of the LHS, and it did it in a heavy-handed way to cope with the
special case of a regexp (which is an RV pointing to REGEXP which might
be UTF8)

eval: handle taint of overloaded/tied arg

string eval would check its arg for taint before processing magic, overload
etc. If the magic or overload returned a tainted value, it wouldn't
be detected. Fixes part of #75716.

eval $overloaded can crash

Perl_lex_start() assumes that the SV passed to it is a well-behaved
string that it can do PVX() stuff to. If it's actually a ref to an
overloaded object, it can crash and burn. Fixed by creating a stringified
copy of the SV if necessary.

fix tainting and overload

Sometimes when an overload method returned a tainted value, that
taintedness got lost. This fixes #75716: overload removes tainting.

It also considerably expands the tied series of tests in overload.t.
It now taints the return value, and checks for correct taintedness.
It also tests against two overload packages: the new one only has fallback
methods, which affects the return path for the tainted value.

It now also compares the expected (non-tied, non-overload) expression
value against a overloaded version of that expression in addition to
versions where a tied var returned an overloaded object; e.g. in these
expressions:
    1: 1 + $plain_value
    2: 1 + $overloaded_var
    3: 1 + $tied_scalar_that_returns_overloaded_value
    4: 1 + $tied_array_whose element_0_holds_an_overloaded_value[0]
then the value of expression 1 is compared against each of 2,3,4, whereas
before it was only compared against 3,4.

taint-enable lib/overload.t

Stick a -T at the top of lib/overload.t in preparation for adding some
taint tests later. This causes some of the current tests to fail, since
the FETCH count has changed: so we fix those up too. They change because
under taint, code like ${$x} is compiled as enter/gvsv/leave/rv2sv rather
than gvsv/rv2sv (see Perl_scope), and the leave creates a mortal copy of
the tied value, avoiding any further fetches.

Exporter version bump

After being modified by 9b86bb5cacfb65a283299b005919058f8a61701a.

Fix RT #74472 Exporter.pm blocks Signal handling

Exporter.pm overrides SIG{__WARN__}. This means It's not possible to
write a test to trap warning events for this error. Since all the line
seems to do is make the warning more verbose, I suggest it not be done
if someone has defined a custom signal. At the moment Test::NoWarnings
cannot trap these warnings because of this.

Skip suid File::Copy tests on a nosuid partition

These tests were being skipped on OpenBSD, but nosuid partitions can
exist on other systems too. Now it just checks if it can create a suid
directory, if not the tests are skipped.

Perl builds without errors in a nosuid /tmp with this patch.

Adjust 'make test.valgrind' to account for cpan/dist/ext separation.

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

Plug more memory leaks in vms.c.

In Perl_rename() we were allocating the same thing twice.

In vms_fid_to_name() (used by realpath() and realname()), we had
another flagrant omission of freeing local storage.

Correct the version of the included Test-Harness in Maintainers.pl

Update CPANPLUS to CPAN version 0.9005

  [DELTA]

  Changes for 0.9005      Tue Jun 29 22:42:25 2010
  ================================================
  * Corrected spelling mistakes in POD, pointed out by H.Merijn Brand
  * Amend SQLite source engine to set SYNCHRONOUS pragma to OFF
  * Make sure that Selfupdate checks if CPANPLUS::Dist::Build is
    installed or not and update it if it is. Spotted by Ilmari
  * Apply a patch from Peter Ludikovsky RT #55782 that adds indexes
    to Source::SQLite

  Changes for 0.9004      Sat May  8 22:21:04 2010
  ================================================
  * Applied a patch from Schwern RT #53133 "test failure occurs if
    the build directory is symlinked"
  * Explicitly set the location of cpanp-run-perl when under PERL_CORE
    in t/inc/conf.pl
  * Make sure that we find cpanp-run-perl in ../../utils when PERL_CORE
  * Apply a patch from Barbie [RT #56768], that fixes regex in
    RELEVANT_TEST_RESULT
  * Fixed a typo in the POD for cpan2dist, pointed out by Babar
    (Olivier Raginel) on irc.
  * Add a BEGIN {} block to cpanp-run-perl this resolves RT #55964
    and RT #57106

  Changes for 0.9003      Mon Mar 15 13:51:12 2010
  ================================================
  * Resolve warnings when using blead perl, reported by Apocalypse RT #55501
  * Applied patch from Apocalypse RT #55541 that changes the
    behviour when a prereq on a core-only module is detected. We
    raise a warning now, but will proceed with the installation.
    This is how CPAN.pm handles it.
  * Removed one duff mirror and an out-of-date mirror from
    our default mirror list. Replaced with fast-sync mirrors.

  Changes for 0.9002      Fri Mar 12 13:11:00 2010
  ================================================
  * Added support for CPAN Testers 2.0 (David Golden)

  Changes for 0.9001      Thu Dec 24 10:21:11 2009
  ================================================
  * RT #52988 Regression in Internals::Utils, reported by CRAKRJACK
  * Fix a regression with dist_type not being propagated

Mark UPSTREAM as 'cpan' for Digest-SHA at Mark Shelor's request.

  Date: Thu, 01 Jul 2010 16:51:02 -0700
  Reply-To: mshelor@cpan.org
  User-Agent: Thunderbird 2.0.0.23 (X11/20090817)
  To: "Chris 'BinGOs' Williams" <chris@bingosnet.co.uk>
  Subject: Re: [Fwd: [rt.cpan.org #58997] correct URL for SHA1 NIST test vectors]

  >Chris 'BinGOs' Williams wrote:

  >Speaking of which, the UPSTREAM flag is set to 'undef' for Digest-SHA
  >
  >How should I mark Digest-SHA?

  I keep very close tabs on Digest-SHA, and would like to assure that the
  current version on CPAN (which I maintain) is the canonical one.
  Whenever blead suggests changes to Digest-SHA, I try to respond immediately.

  Cheers, Mark

Fix nasty memory leak in vms.c:Perl_flex_stat_int().

Thanks to Mark Berryman in

  Message-id: <52BFD39B-C4BA-4350-AA1F-C2E7F128EF21@theberrymans.com>

for the report and the fix.

t/lib/common.pl can use caller to infer the name of the pragma under test.

This avoids needing to use a variable in package main to pass in information.
Also, remove an unnecessary $ENV{PERL5LIB} assignment cargo-culted into most
users of t/lib/common,pl, and remove the BEGIN block as the code it contained
doesn't need to run at BEGIN time.

Fix to MANIFEST in base

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

Fix pthread include error for Time::Piece

Oh, um, lovely. Did I really want to look at that code? I can only assume
that in *BSD the API for _strptime() is fixed. But it's static for us, so
it doesn't need to be fixed.

Does the appended patch fix it?

Nicholas Clark

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

add cherrymaint and git-deltatool to MANIFEST

Convert zlib src from K&R functions declarations to ANSI C.

K&R declarations aren't valid C++, hence C++ builds choke. zlib defaults to
K&R (I infer for maximum portability), but helpfully the zlib distribution
provides a zlib2ansi script to convert the declarations.

Convert subs.t to use t/lib/common.pl

Convert charnames.t to use t/lib/common.pl

Convert strict.t to use t/lib/common.pl

Convert gmtime.t, localtime.t and overloading.t to Test::More.

Convert DirHandle's tests to Test::More.

Remove a vestigial reference to MacOS.

Add additional notes regarding srand and forking

perldoc -f srand states that typical use requires no srand() to be
called.  This is true with the exception of forking where you may not
want the same seed across various child processes (i.e. mod_perl).
This patch simply adds a note reminding the reader of this fact and
more specifically states that srand should only be called once per
*process* (instead of the previous language of once per *script*).

Signed-off-by: Brian Phillips <bphillips@cpan.org>

Convert Fcntl and PerlIO::encoding's tests to Test::More.

Remove one vestigial mention of MacOS.

Convert POSIX's test to Test::More.

Fix MANIFEST to include new Test-Harness files added in 6d3136644eb73f7a0727d341c90937f4234835bf

Convert Devel::Peek's test to Test::More.

The flag change from 0x0 to 0x4 is because Test::More sets autoflush on STDOUT.