2 years agoFIXME WIP Feeble attempt to improve failure modes smueller/sort
Steffen Mueller [Thu, 24 Nov 2011 17:50:35 +0000]
FIXME WIP Feeble attempt to improve failure modes

2 years agoMore brutal sort optimization tests
Steffen Mueller [Mon, 21 Nov 2011 07:04:57 +0000]
More brutal sort optimization tests

2 years agoSome tests for the sort deref optimization
Steffen Mueller [Fri, 18 Nov 2011 07:08:11 +0000]
Some tests for the sort deref optimization

Specifically in the context of overloading.

2 years agoFIXME this can't go into blead as is
Steffen Mueller [Tue, 15 Nov 2011 17:51:15 +0000]
FIXME this can't go into blead as is

Optimization to make the Schwartzian transform faster

This patch consists of two conceptually separate but technically
entwined changes. In a nutshell, the idea is to detect the use of an
array dereference for sorting on precomputed data. This means sort
invocations like the following:

  sort { $a->[0] <=> $b->[0] }

and

  sort { $a->[0] cmp $b->[0] }

as well as the reversed cases.

The two major changes are:

1) Detection of a basic Schwartzian transform-style sort blocks
and removal of the corresponding bits of OP tree. Sets a new flag on
the sort OP which will indicate that this sort needs to be performed
by one of the special-case optimized C functions in pp_sort.

Right now, we only match array dereferencing using the constant 0.
Technically, it is trivial to match any other constant, and feasible to
match the use of a variable. But I could not think of a good way to pass
the array index to pp_sort.

2) Implementation of special-purpose sort functions that do
array-dereferencing. This change implements the above sort functions for
all of numeric, integer, and string sorts, as well as overloaded values
in the arrays and locale-string sorts.

It supports overload magic inner SVs (see tests added in next
commit). I believe it also handles the locale cmp case in the same way
as the normal optimized sort functions do, but since I don't properly
understand locales, the tests don't cover that.

The net effect of this is a consistent speed-up by over 2x on my
machine when doing a Schwartzian transform or similar technique
involving sorting on the first element of an array. I tried a variety
of array sizes, as well as inclusion of the two map stages of the
idiom.

2 years agoRename variable for clarity
Steffen Mueller [Mon, 14 Nov 2011 19:50:41 +0000]
Rename variable for clarity

2 years agonarrower localisation of PL_compcv around eval
Zefram [Sat, 19 Nov 2011 16:00:32 +0000]
narrower localisation of PL_compcv around eval

PL_compcv used to be localised around the entire string eval process,
and hence at runtime of the evaled code would refer to the evaled code
rather than code of a surrounding compilation.  This interfered with the
ability of string-evaled code in a BEGIN block to affect the surrounding
compilation, in a similar way to the localisation of $^H and %^H that
was fixed in f45b078d20.

Similar to the fix there, this change moves the localisation of PL_compcv
inside the new evalcomp scope.  A couple of things were relying on
PL_compcv to find the running code when in a string-eval scope; they now
need to find it from cx->blk_eval.cv, which was already being populated.

2 years agosv.c: More consistent use of spaces after dots
Father Chrysostomos [Sat, 19 Nov 2011 14:59:59 +0000]
sv.c: More consistent use of spaces after dots

in apidocs, plus other tweaks

2 years agoCorrect sv_catsv_flags docs
Father Chrysostomos [Sat, 19 Nov 2011 14:39:48 +0000]
Correct sv_catsv_flags docs

Get-magic is only called on ssv.  SV_SMAGIC is accepted, too.

2 years agoCorrect sv_catpv_flags docs
Father Chrysostomos [Sat, 19 Nov 2011 14:36:39 +0000]
Correct sv_catpv_flags docs

SV_SMAGIC, not SV_GMAGIC.

2 years agoCorrect sv_catpvn_flags docs
Father Chrysostomos [Sat, 19 Nov 2011 14:35:23 +0000]
Correct sv_catpvn_flags docs

It respects SV_SMAGIC flag, not SV_GMAGIC (which it ignores).

2 years agoAvoid a redundant copy in pp_glob
Father Chrysostomos [Sat, 19 Nov 2011 08:30:14 +0000]
Avoid a redundant copy in pp_glob

Most of the time, the argument to glob() won’t be tied, so we don’t
need to copy it.  So only copy it if it is gmagical.

2 years agoperldelta for var names in @a=~// warning
Father Chrysostomos [Sat, 19 Nov 2011 08:20:04 +0000]
perldelta for var names in @a=~// warning

2 years agoMention variable names in @a =~ // warnings
Father Chrysostomos [Sat, 19 Nov 2011 08:17:44 +0000]
Mention variable names in @a =~ // warnings

2 years agoperldelta for y///r implicit $_ uninit warning
Father Chrysostomos [Sat, 19 Nov 2011 08:03:00 +0000]
perldelta for y///r implicit $_ uninit warning

2 years agoMention implicit $_ in y///r uninit warning
Father Chrysostomos [Sat, 19 Nov 2011 07:57:41 +0000]
Mention implicit $_ in y///r uninit warning

This brings it into conformity with y without the /r.

2 years agoMerge two adjacent #ifdefs in embed.fnc
Father Chrysostomos [Sat, 19 Nov 2011 07:22:17 +0000]
Merge two adjacent #ifdefs in embed.fnc

d08d57ef added a new #ifdef right after and #endif with the same con-
dition.  There is no need for the separation.

2 years agoMention the variable name in the new length warnings
Father Chrysostomos [Sat, 19 Nov 2011 06:03:04 +0000]
Mention the variable name in the new length warnings

2 years agoDon’t create an AV for unfiltered code
Father Chrysostomos [Sat, 19 Nov 2011 04:27:15 +0000]
Don’t create an AV for unfiltered code

When there are no source filters, we don’t need to create an AV to
hold them.  The code for adding a filter already does a null check.

2 years agoTweaks to find_uninit_var docs
Father Chrysostomos [Sat, 19 Nov 2011 01:57:11 +0000]
Tweaks to find_uninit_var docs

• Typos
• Double space after fullstop, as that is how nroff renders
  a fullstop at the end of a line in mid-paragraph.

2 years agoperldelta for new length() warning
Father Chrysostomos [Sat, 19 Nov 2011 01:49:35 +0000]
perldelta for new length() warning

2 years agoFix pp_goto crash with orphaned GV
Father Chrysostomos [Sat, 19 Nov 2011 00:38:49 +0000]
Fix pp_goto crash with orphaned GV

a7999c089 inadvertently made pp_goto crash if the GV had no
stash pointer.

2 years agoutil.c:get_db_sub: correct comment
Father Chrysostomos [Sat, 19 Nov 2011 00:33:58 +0000]
util.c:get_db_sub: correct comment

This comment was copied from pp_goto by commit 005a8a35, but was not
adjusted to make sense in its new surroundings.

2 years agoperldelta for GV scalar localisation
Father Chrysostomos [Sat, 19 Nov 2011 00:18:46 +0000]
perldelta for GV scalar localisation

2 years agoCopy magic during localisation even for GVs
Father Chrysostomos [Sat, 19 Nov 2011 00:17:22 +0000]
Copy magic during localisation even for GVs

The magic-copying is skipped for GVs.  This logic goes back to perl
5.000 (patch a0d0e21e).  I think it has always been wrong.

2 years agoTest length() warning with pad vars
Father Chrysostomos [Thu, 17 Nov 2011 21:03:39 +0000]
Test length() warning with pad vars

2 years agoperldiag: Rewrap ‘length() used on %s’
Father Chrysostomos [Thu, 17 Nov 2011 20:43:15 +0000]
perldiag: Rewrap ‘length() used on %s’

splain output looks better in an 80-column terminal this way.

2 years agoThrow a helpful warning when someone tries length(@array) or length(%hash)
Matthew Horsfall (alh) [Thu, 17 Nov 2011 04:06:33 +0000]
Throw a helpful warning when someone tries length(@array) or length(%hash)

2 years agoTweak the cBOOL() macro to avoid problems with the AIX compiler.
Nicholas Clark [Tue, 15 Nov 2011 18:44:18 +0000]
Tweak the cBOOL() macro to avoid problems with the AIX compiler.
(cherry picked from commit 0cebf65582f924952bfee1472749d442d51e43e6)

2 years agoFix threaded build
Father Chrysostomos [Fri, 18 Nov 2011 23:55:08 +0000]
Fix threaded build

Commit a7999c0893, you are at fault!

2 years agoperldelta: fix for $DB::sub with bad subname
Father Chrysostomos [Fri, 18 Nov 2011 20:42:45 +0000]
perldelta: fix for $DB::sub with bad subname

2 years agoUpdate several test cases to not generate failures on VOS.
Paul Green [Thu, 10 Nov 2011 22:34:19 +0000]
Update several test cases to not generate failures on VOS.

2 years agoMake sure $DB::sub is callable
Father Chrysostomos [Fri, 18 Nov 2011 17:08:32 +0000]
Make sure $DB::sub is callable

When DB::sub is about to be called (to handle a subroutine call under
the debugger), $DB::sub is set to the name of the subroutine or a ref-
erence to it.

Sometimes $DB::sub is set to the name when the subroutine is not call-
able under that name.  That should not happen.

This logic in util.c:Perl_get_db_sub decides whether a reference
should be used:

if ( svp && ((CvFLAGS(cv) & (CVf_ANON | CVf_CLONED))
     || strEQ(GvNAME(gv), "END")
     || ((GvCV(gv) != cv) && /* Could be imported, and old sub redefined. */
 !( (SvTYPE(*svp) == SVt_PVGV)
    && (GvCV((const GV *)*svp) == cv)
    && (gv = (GV *)*svp)
  )
)
)) {
    /* Use GV from the stack as a fallback. */

(That comment about using the GV from the stack as a fallback applies
to the assignment to gv, but was mistakenly divorced from it in commit
3de9ffa12.)

This logic (introduced in 71be2cbc7 [inseparable changes from
perl5.003_13 to perl5.003_14] and integrated into blead in 491527d02)
tries to find a GV that points to the CV, trying the CV’s own GV
first, and falling back to what is on the stack.  But it does not
account for GVs that are not found under their names, which can hap-
pen when a glob is copied and the original is undefined ($foo = *bar;
undef *bar; &$foo) or when a stash element or package is deleted, such
as via Symbol::delete_package.

If the subroutine is not locatable under its own name or the name
under which it was called (the name of the GV argument to entersub),
then a reference should be passed.  Otherwise a name that can access
the sub should be passed.

So this commit adds more (no, not more!) conditions to make sure the
gv is actually reachable under its name before using a string.

Since, for effiency, those conditions do not perform an actual symbol
lookup, but simply look inside the GV’s stash, we can no longer rely
on gv_efullname (or even gv_fullname), as the stash may have been
moved around, but use HvENAME and construct the GV name ourselves.

2 years agoRemove redundant check in pp_hot.c:pp_entersub
Father Chrysostomos [Fri, 18 Nov 2011 00:38:42 +0000]
Remove redundant check in pp_hot.c:pp_entersub

PVGVs are always globs now.

2 years agoUpdate IO-Compress to CPAN version 2.042
Chris 'BinGOs' Williams [Fri, 18 Nov 2011 20:11:08 +0000]
Update IO-Compress to CPAN version 2.042

  [DELTA]

  2.042 17 November 2011

      * IO::Compress::Zip
        - Added exUnixN option to allow creation of the "ux" extra field.
          This allows 32-bit UID/GID to be stored.
        - In one-shot mode use exUnixN rather than exUnix2 for the UID/GID.

      * IO::Compress::Zlib::Extra::parseExtraField
        - Fixed bad test for length of ID field
          [RT# 72329 & #72505]

2 years agoUpdate Compress-Raw-Zlib to CPAN version 2.042
Chris 'BinGOs' Williams [Fri, 18 Nov 2011 20:06:18 +0000]
Update Compress-Raw-Zlib to CPAN version 2.042

  [DELTA]

  2.042 17 November 2011

      * No Changes

2 years agoUpdate Compress-Raw-Bzip2 to CPAN version 2.042
Chris 'BinGOs' Williams [Fri, 18 Nov 2011 19:59:16 +0000]
Update Compress-Raw-Bzip2 to CPAN version 2.042

  [DELTA]

  2.042 17 November 2011

      * No Changes

2 years agoSeparate the build and non-build functionality of pod/buildtoc
Nicholas Clark [Fri, 18 Nov 2011 10:22:58 +0000]
Separate the build and non-build functionality of pod/buildtoc

pod/buildtoc now now only generates pod/perltoc.pod
All the other file (re)generation code is now in Porting/pod_rules.pl
Porting/new-perldelta.pl automates adding a new perldelta

This should simplify the release manager's work.

2 years agoNote the changes to pod/buildtoc in the perldelta.
Nicholas Clark [Tue, 15 Nov 2011 12:17:41 +0000]
Note the changes to pod/buildtoc in the perldelta.

Principally to note that no-one should notice the difference, but give a clue
as to what might be wrong if they do.

2 years agoIn buildtoc, refactor to de-duplicate code and variables.
Nicholas Clark [Thu, 10 Nov 2011 14:41:47 +0000]
In buildtoc, refactor to de-duplicate code and variables.

Converting the globals %Pragmata and %Modules to keys in a single hash
reduces the code size.

2 years agoRefactor pod_rules.pl and buildtoc to use new functions in pod_lib.pl
Nicholas Clark [Thu, 10 Nov 2011 10:52:34 +0000]
Refactor pod_rules.pl and buildtoc to use new functions in pod_lib.pl

This reduces code duplication.

2 years agoPorting/pod_rules.pl should not warn about files that it is rebuilding.
Nicholas Clark [Thu, 10 Nov 2011 10:18:06 +0000]
Porting/pod_rules.pl should not warn about files that it is rebuilding.

Previously it would generate warnings about inconsistencies in files, even if
it was about to rewrite the file to be consistent.

2 years agoAdd Porting/new-perldelta.pl, which automates adding a new perldelta.
Nicholas Clark [Wed, 9 Nov 2011 20:50:56 +0000]
Add Porting/new-perldelta.pl, which automates adding a new perldelta.

Strictly, "mostly automates", as it doesn't run `git add`, or test that the
build is clean. However, it simplifies a gnarly step in the release manager's
guide, so it's progress.

2 years agoFlag to get_pod_metadata() whether generated files are expected.
Nicholas Clark [Wed, 9 Nov 2011 16:25:57 +0000]
Flag to get_pod_metadata() whether generated files are expected.

This permits Porting/pod_rules.pl to be run from a clean checkout without
warning, which simplifies updating perl's version.

2 years agoRefactor pod_lib.pl's get_pod_metadata() to remove a temporary array.
Nicholas Clark [Wed, 9 Nov 2011 12:40:59 +0000]
Refactor pod_lib.pl's get_pod_metadata() to remove a temporary array.

The intermediate array @cpanpods can be avoided by generating the lookup
hashes directly.

Rename %cpanpods_short to %cpanpods_leaf and $shortname to $leafname, as this
makes it clearer that each deals with leafnames.

2 years agoRename t/porting/buildtoc.t to pod_rules.t to reflect the previous change.
Nicholas Clark [Wed, 9 Nov 2011 12:52:29 +0000]
Rename t/porting/buildtoc.t to pod_rules.t to reflect the previous change.

As it now calls Porting/pod_rules.pl, not pod/buildtoc, it makes little sense
to keep the previous name.

2 years agoCreate Porting/pod_rules.pl, with all non perltoc.pod code from buildtoc.
Nicholas Clark [Tue, 8 Nov 2011 14:11:46 +0000]
Create Porting/pod_rules.pl, with all non perltoc.pod code from buildtoc.

pod/buildtoc now only generates pod/perltoc.pod, and only takes a --quiet
option. All the other file (re)generation code is now in Porting/pod_rules.pl.

This splits functionality which generates a new file at build time from
functionality for maintainers, which edits files in place. This permits the
unwinding of some of commit 02cc404a20a8a345, which achieved net
simplification in pod/buildtoc by merging code paths dealing with new files
(only pod/perltoc.pod) and existing files being edited.

pod_rules.pl has a --tap option, instead of buildtoc's non-standard --test
option, and defaults to --build-all if no arguments are passed. As buildtoc
now does one job, it no longer takes a --build-toc option, hence all the
Makefiles which call it need updating.

2 years agoIn pod_lib.pl's get_pod_metadata(), generate lookup hashes directly.
Nicholas Clark [Tue, 8 Nov 2011 08:45:24 +0000]
In pod_lib.pl's get_pod_metadata(), generate lookup hashes directly.

Previously the code was structured to build arrays for various categories of
files, then generate hashes from those arrays. However, the arrays had no
other purpose, and the values in the hashes were only needed for truth tests,
so it's better to generate the lookup hashes directly.

@disk_pods hasn't actually been used since commit d5e2eea989a69524 refactored
pod/buildtoc to no longer chdir to the pod/ directory.

2 years agoExtract from buildtoc the code that processes pod.lst, MANIFEST and perl.pod
Nicholas Clark [Tue, 8 Nov 2011 08:29:33 +0000]
Extract from buildtoc the code that processes pod.lst, MANIFEST and perl.pod

This will permit splitting pod/buildtoc into two - one script used during
the build process to build pod/perltoc.pod, and used by maintainers to
regenerate sections of various Makefiles.

2 years agoIn buildtoc, refactor the code that reads pod.lst, MANIFEST and perl.pod
Nicholas Clark [Tue, 8 Nov 2011 08:09:02 +0000]
In buildtoc, refactor the code that reads pod.lst, MANIFEST and perl.pod

Move the code into a function, get_pod_metadata(), which returns a reference
to a hash of metadata. This replaces a swathe of global variables.

2 years agobuildtoc --showfiles should report the correct pathnames.
Nicholas Clark [Tue, 8 Nov 2011 13:39:41 +0000]
buildtoc --showfiles should report the correct pathnames.

The output of --showfiles has been garbled since commit d5e2eea989a69524
refactored pod/buildtoc to no longer chdir to the pod/ directory.

2 years agobuildtoc --showfiles should only list the files to regenerated.
Nicholas Clark [Tue, 8 Nov 2011 13:29:08 +0000]
buildtoc --showfiles should only list the files to regenerated.

This fixes a bug caused by the refactoring of commit 9dce16cd064e70e6,
which caused --showfiles to always list list all known files, irrespective
of which had been specified with command line flags.

2 years agoIn buildtoc, don't use File::Spec when initialising %BuildFiles.
Nicholas Clark [Tue, 8 Nov 2011 10:57:53 +0000]
In buildtoc, don't use File::Spec when initialising %BuildFiles.

%BuildFiles is initialised from the leafnames of %Build, which is
initialised from %Targets, which always contains pathnames in Unix
format. Hence using File::Spec->splitpath() is actually erroneous, as it
will process pathnames assuming the format for the current platform.

2 years agoIn buildtoc, refactor the open ... or my_die pattern into a function.
Nicholas Clark [Mon, 7 Nov 2011 20:36:55 +0000]
In buildtoc, refactor the open ... or my_die pattern into a function.

This eliminates several variables used for filenames, including $masterpodfile,
which was declared as a global.

2 years agoIn buildtoc, make %Readmepods lexical, and $delta_target global.
Nicholas Clark [Mon, 7 Nov 2011 20:23:39 +0000]
In buildtoc, make %Readmepods lexical, and $delta_target global.

%Readmepods is only used within the code to read pod.lst, pod/perlpod.pod
and MANIFEST, not by the generation code.
$delta_target is set by the reading code, but used in the generation code.

This makes the use of global and lexical variables more consistent.

2 years agoIn buildtoc, eliminate abs_from_top() as it now runs from the top level.
Nicholas Clark [Mon, 7 Nov 2011 12:21:12 +0000]
In buildtoc, eliminate abs_from_top() as it now runs from the top level.

There's also no need to convert to native path specs, as no paths are passed
to external commands, and the perl runtime on all platforms understands
unambiguously Unix-like paths.

2 years agoIn pod/buildtoc, chdir to the top level of the source tree before running.
Nicholas Clark [Mon, 7 Nov 2011 12:03:09 +0000]
In pod/buildtoc, chdir to the top level of the source tree before running.

Originally buildtoc called chdir to ensure it ran from the pod/ directory.
However, this caused problems when the environment is set to honour UTF-8
locales, and the regex engine (attempts to) demand-load UTF-8 swashes,
because on *nix and VMS buildtoc is invoked with a relative path in @INC.
d5e2eea989a69524 fixed this by avoiding the chdir.

However, an equally valid solution is to run from the top level of the source
tree, because the actual use cases only have relative paths in @INC when
invoked from the top level, with absolute paths in @INC for the use cases
that invoke perltoc from other directories. Hence, it's always safe to change
to the top level of the source tree. This permits future simplification and
refactoring.

2 years agoLocalise $/ properly in UCD.pm
Father Chrysostomos [Thu, 17 Nov 2011 23:53:18 +0000]
Localise $/ properly in UCD.pm

Localisation of $/ in the module’s main CV is undone once the module
finishes loading, and hence has no effect on its function after that.
Each function that uses readline has to localise $/ itself.

2 years agoDetect whether ccache is available to bisect.
Paul Johnson [Thu, 17 Nov 2011 20:20:15 +0000]
Detect whether ccache is available to bisect.

ccache will save time during a bisect but it may not always be availble.
Teach bisect-runner.pl to only use it if it is available.

2 years agoturn off buggy optimisations in one HP compiler
Zefram [Thu, 17 Nov 2011 19:53:04 +0000]
turn off buggy optimisations in one HP compiler

Fixes [perl #103668].

2 years agoRe-deprecate $[
Father Chrysostomos [Thu, 17 Nov 2011 17:59:09 +0000]
Re-deprecate $[

This was something e1dccc0d3 removed that b82b06b8 failed to restore.

2 years agoarybase.xs: Don’t use arybase::[
Father Chrysostomos [Thu, 17 Nov 2011 17:38:20 +0000]
arybase.xs: Don’t use arybase::[

arybase was changing $[=1 to $arybase::[ = 1 to avoid any run-time
effects.  In case this module is ever dual-lifed, we ought to avoid
punctuation variables even outside the main stash.  See bug #76138.

This is a classic example of paranoia.

2 years agoarybase.xs: Check that $[ is in the main stash
Father Chrysostomos [Thu, 17 Nov 2011 17:35:53 +0000]
arybase.xs: Check that $[ is in the main stash

Check that $[ is in the main stash before doing anything magical with
$[=1.  This can only be in another stash if some other XS module has
overridden the check functions to change $[ to $foo::[.  But robust-
ness is good, right?

2 years agoAdd can_isa_ok.t to MANIFEST
Father Chrysostomos [Thu, 17 Nov 2011 17:40:49 +0000]
Add can_isa_ok.t to MANIFEST

2 years agoAdd class_ok() and object_ok() to t/test.pl.
Michael G. Schwern [Wed, 16 Nov 2011 01:39:07 +0000]
Add class_ok() and object_ok() to t/test.pl.

Change every existing instance of isa_ok() to use object_ok().  This is safe because
before this point, t/test.pl's isa_ok() only worked on objects.  lib/dbmt_common.pl is
the last hold out because it uses Test::More.

These are like isa_ok() but they also check if it's a class or an object.
This lets the core tests defend against outlandish bugs while allowing
t/test.pl to retain feature parity with Test::More.

2 years agoPatch t/test.pl so isa_ok() works with objects.
Michael G. Schwern [Wed, 16 Nov 2011 01:14:52 +0000]
Patch t/test.pl so isa_ok() works with objects.

This mirrors can_ok() and Test::More.

Also add some tests for isa_ok() and can_ok().

2 years agoSun cc and CC do not support gcc attributes.
Reini Urban [Tue, 15 Nov 2011 22:14:28 +0000]
Sun cc and CC do not support gcc attributes.

Similar code as in linux.sh

2 years agoDetect SunOS CC workshop c++ compiler
Reini Urban [Tue, 15 Nov 2011 22:11:02 +0000]
Detect SunOS CC workshop c++ compiler

Solaris 11 SunStudio 12.2. ships without cc, so CC must be used.
Detect it and set ccversion correctly

2 years agoperldelta for #70151
Father Chrysostomos [Thu, 17 Nov 2011 16:39:06 +0000]
perldelta for #70151

2 years agoDocument S_doeval’s calling convention
Father Chrysostomos [Thu, 17 Nov 2011 02:21:16 +0000]
Document S_doeval’s calling convention

This should make it easier for future maintainers, who now will not
have to figure out what I figured out.

2 years ago[perl #70151] eval localises %^H at runtime
Father Chrysostomos [Thu, 17 Nov 2011 02:18:23 +0000]
[perl #70151] eval localises %^H at runtime

It doesn’t any more.

Now the hints are localised in a separate inner scope surrounding the
call to yyparse.  This meant moving hint-handling code from pp_require
and pp_entereval into S_doeval.

Some tests in t/comp/hints.t were testing for the buggy behaviour, so
they have been adjusted.

Basically, this fixes

    sub import {
        eval "strict->import"
    }

which should work the same way as

    sub import {
        strict->import
    }

but was not working because %^H and $^H were being localised to
the eval at its run time, not just its compilation.  So the values
assigned to %^H and $^H at the eval’s run time would simply be lost.

2 years agoperldelta for PL_curcop localisation
Father Chrysostomos [Thu, 17 Nov 2011 16:36:43 +0000]
perldelta for PL_curcop localisation

2 years agoLocalise PL_curcop for BEGIN blocks
Father Chrysostomos [Thu, 17 Nov 2011 16:34:31 +0000]
Localise PL_curcop for BEGIN blocks

Usually when a BEGIN block exits it has to set PL_curcop to
&PL_compiling, so that subsequent compiled code in the surrounding
scope will have the right warning hints during compilation.

If an XS function creates a BEGIN block via newXS or newATTRSUB, how-
ever, the assumption that compilation will resume as soon as the block
exits is false.

This can be demonstrated with this code, which warns about CHECK and
INIT blocks created too late when it shouldn’t due to ‘no warnings’:

use warnings;
eval q|
  BEGIN{
    no warnings;
    package XS::APItest; require XSLoader; XSLoader::load()
  }
|;

In every case where it is correct for BEGIN to set PL_curcop to
&PL_compiling when it exits it is actually just restoring it to its
previous value, so localisation is the right fix.

2 years agoCombine two blocks in pp_ctl.c:S_doeval
Father Chrysostomos [Wed, 16 Nov 2011 02:06:10 +0000]
Combine two blocks in pp_ctl.c:S_doeval

As of commit e4a21daa4e9 which removed the intervening code, there
have been two adjacent if blocks in S_doeval with the same condition.
They can be merged into one, to avoid confusing people (like me) try-
ing to understand the code.

2 years agoeval.t: add bug number
Father Chrysostomos [Tue, 15 Nov 2011 13:54:14 +0000]
eval.t: add bug number

2 years agoeval.t: typos
Father Chrysostomos [Tue, 15 Nov 2011 13:49:28 +0000]
eval.t: typos

2 years agoAdd --check-shebang to bisect-runner.pl, to avoid mistakes with #! lines
Nicholas Clark [Thu, 17 Nov 2011 16:59:22 +0000]
Add --check-shebang to bisect-runner.pl, to avoid mistakes with #! lines

This allows bisect.pl to instruct bisect-runner.pl to check that the user's
testcase is not a script with a #! line which will run an installed perl.
This rapidly catches the potential error of specifying a test case that will
not actually be tested with the bisect-build perl. Without this, the user
may end up with an unexpected failure to bisect, and little clue as to what
the actual problem is.

2 years agoNote in Porting/bisect.pl that its documentation is in bisect-runner.pl
Nicholas Clark [Thu, 17 Nov 2011 15:15:06 +0000]
Note in Porting/bisect.pl that its documentation is in bisect-runner.pl

It may not be obvious to anyone skimming for Pod that they need to look
elsewhere to find it.

2 years agoFix fatal bug in bisect-runner.pl for some 5.004 revisions.
Nicholas Clark [Thu, 17 Nov 2011 16:42:59 +0000]
Fix fatal bug in bisect-runner.pl for some 5.004 revisions.

Commit afc98f896ef0b0e2 tweaked bisect-runner.pl to patch a bug in perl.c
introduced in commit 2a92aaa05aa1acbf and fixed in commit 8490252049bf42d3.
However, it didn't check properly whether it should apply the patch at all,
hence caused bisect-runner.pl to abort for some other revisions. Now fixed.

2 years agoUnused variable in in sv_eq
Brian Fraser [Thu, 17 Nov 2011 15:06:53 +0000]
Unused variable in in sv_eq

2 years agoFix misprint in doc of UNIVERSAL
Perlover [Thu, 17 Nov 2011 02:46:41 +0000]
Fix misprint in doc of UNIVERSAL

2 years agoavoid some test-time warnings
Zefram [Wed, 16 Nov 2011 21:31:41 +0000]
avoid some test-time warnings

2 years ago\x-ify non-ASCII chars in C string literal
Zefram [Wed, 16 Nov 2011 20:39:42 +0000]
\x-ify non-ASCII chars in C string literal

2 years agoexpunge gratuitous Unicode punctuation in comments
Zefram [Wed, 16 Nov 2011 20:32:56 +0000]
expunge gratuitous Unicode punctuation in comments

2 years ago$Config{locincpath} might be empty
H.Merijn Brand [Tue, 15 Nov 2011 16:07:56 +0000]
$Config{locincpath} might be empty

This causes h2xs.t to fail on systems where that is the case

2 years agoperldelta additions
Karl Williamson [Tue, 15 Nov 2011 15:29:50 +0000]
perldelta additions

2 years agofeature.pm: Avoid repetition; save memory
Father Chrysostomos [Tue, 15 Nov 2011 12:58:50 +0000]
feature.pm: Avoid repetition; save memory

2 years agoAdd 5.16 feature bundle
Father Chrysostomos [Tue, 15 Nov 2011 12:52:25 +0000]
Add 5.16 feature bundle

It needs to be added some time before the major version
bump, so now is as good a time as any.

2 years agoperldelta up to 7c5c3d9b
Father Chrysostomos [Tue, 15 Nov 2011 06:47:12 +0000]
perldelta up to 7c5c3d9b

2 years agoMove a pp_entereval comment where it belongs.
Father Chrysostomos [Mon, 7 Nov 2011 01:51:25 +0000]
Move a pp_entereval comment where it belongs.

Commit f9bddea7d2 divorced it from the code it was describing (dele-
tion of the FILEGV on eval exit).  That code was subsequently repeated
in various places by commit 78da7625.  The comment is now above the
first instance.

2 years agoInternals::SvREFCNT() now treats reference counts as unsigned
Tony Cook [Mon, 14 Nov 2011 08:48:54 +0000]
Internals::SvREFCNT() now treats reference counts as unsigned

Previously setting a large (negative in 32-bit signed) reference count
would be returned as a positive number on 64-bit builds and negative
on 32-bit builds.

2 years ago[rt #103222] make Internals::SvREFCNT set/get consistent
Tony Cook [Mon, 14 Nov 2011 08:30:17 +0000]
[rt #103222] make Internals::SvREFCNT set/get consistent

2 years agomatch the definition of S_mro_gather_and_rename to its declaration
Tony Cook [Mon, 14 Nov 2011 07:50:52 +0000]
match the definition of S_mro_gather_and_rename to its declaration

Based on the warning from:

http://www.nntp.perl.org/group/perl.daily-build.reports/2011/11/msg108741.html

which I haven't been able to produce in any other compiler.

2 years agoIn Perl_lex_start(), don't read the byte before SvPVX().
Nicholas Clark [Sun, 13 Nov 2011 18:07:09 +0000]
In Perl_lex_start(), don't read the byte before SvPVX().

If len is 0, we shouldn't be reading from len - 1, as it's one before the
start of the buffer, and hence an out of bounds read.

Fixes a bug inadvertently added by commit 0abcdfa4c5da571f, restoring the
previous behaviour for the len == 0 case.

2 years agoUpdate string copying in vms/vms.c
Craig A. Berry [Sun, 13 Nov 2011 13:21:05 +0000]
Update string copying in vms/vms.c

In several places strncpy was being used to copy a string of known
length with null handling done separately; those cases have been
converted to use memcpy.  Most uses of strncpy and strcpy have
been converted to my_strlcpy and most uses of strcat have been
converted to my_strlcat.

2 years agoUpdate CGI to CPAN version 3.58
Chris 'BinGOs' Williams [Sat, 12 Nov 2011 23:26:25 +0000]
Update CGI to CPAN version 3.58

  [DELTA]

  Version 3.58 Nov 11th, 2011

    [DOCUMENTATION]
    - Clarify that using query_string() only has defined behavior when using the GET method. (RT#60813)

2 years agomktables: nits in comment and pod
Karl Williamson [Sat, 12 Nov 2011 17:42:23 +0000]
mktables: nits in comment and pod

2 years agomktables: Remove support for deprecated properties
Karl Williamson [Sat, 12 Nov 2011 17:34:43 +0000]
mktables: Remove support for deprecated properties

As proposed and agreed some months ago, certain deprecated since 5.12
Unicode properties that Perl should never have exposed are removed.
These are Unicode-internal properties that are proper subsets of the
properties that should be used instead; and are used by Unicode for
stability reasons in constructing those supersets.

2 years agoperlunicode: Update reference to Unicode standard
Karl Williamson [Sat, 12 Nov 2011 16:01:00 +0000]
perlunicode: Update reference to Unicode standard

2 years agoutf8.c: typo in comment
Karl Williamson [Sat, 12 Nov 2011 15:46:41 +0000]
utf8.c: typo in comment

2 years agopp.c: Make sure variable is initialized
Karl Williamson [Sat, 12 Nov 2011 15:45:40 +0000]
pp.c: Make sure variable is initialized

A compiler generated a warning about this.  It is the degenerate case
with an empty input, so isn't really a problem, but silence the warning