This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
perl5.git
11 years agoMake cpan/CGI/t/http.t (hopefully) compatible with VMS
Father Chrysostomos [Thu, 14 Oct 2010 21:04:55 +0000 (14:04 -0700)]
Make cpan/CGI/t/http.t (hopefully) compatible with VMS

I will submit a similar patch upstream in a few days.

11 years agoperldelta: Missing a C from C<>
Father Chrysostomos [Thu, 14 Oct 2010 20:45:13 +0000 (13:45 -0700)]
perldelta: Missing a C from C<>

11 years agoConvert modules in ext/ to pass minimal arguments to XSLoader::load().
Nicholas Clark [Thu, 14 Oct 2010 20:44:08 +0000 (21:44 +0100)]
Convert modules in ext/ to pass minimal arguments to XSLoader::load().

11 years agoXSLoader::load() with no arguments can use caller to find a default package.
Nicholas Clark [Thu, 14 Oct 2010 20:30:16 +0000 (21:30 +0100)]
XSLoader::load() with no arguments can use caller to find a default package.

In the case of dynamic linking, it's already using caller to get a filename, so
this isn't usually any extra work.

11 years agoperldelta: delete $+{...} was in the wrong section
Father Chrysostomos [Thu, 14 Oct 2010 20:37:18 +0000 (13:37 -0700)]
perldelta: delete $+{...} was in the wrong section

11 years agoperldelta up to 97a3247
Father Chrysostomos [Thu, 14 Oct 2010 20:34:15 +0000 (13:34 -0700)]
perldelta up to 97a3247

11 years agoCorrect threads version
Father Chrysostomos [Thu, 14 Oct 2010 20:02:40 +0000 (13:02 -0700)]
Correct threads version

11 years agoUpdate .gitignore for the Tie::Hash::NamedCapture move
Florian Ragwitz [Thu, 14 Oct 2010 16:55:52 +0000 (18:55 +0200)]
Update .gitignore for the Tie::Hash::NamedCapture move

11 years agoChange the perl version mentioned in threads.pm concerning dirhandles
Father Chrysostomos [Thu, 14 Oct 2010 16:33:01 +0000 (09:33 -0700)]
Change the perl version mentioned in threads.pm concerning dirhandles
as this may change before 5.14.

11 years agoRemove the code for file exclusions from Maintainers.pm - nothing needs it now.
Nicholas Clark [Thu, 14 Oct 2010 14:30:47 +0000 (15:30 +0100)]
Remove the code for file exclusions from Maintainers.pm - nothing needs it now.

11 years agoSince c440907b, Maintainers.pl doesn't need exclusions for XSLoader's files.
Nicholas Clark [Thu, 14 Oct 2010 13:39:15 +0000 (14:39 +0100)]
Since c440907b, Maintainers.pl doesn't need exclusions for XSLoader's files.

Previously, XSLoader and DynaLoader shared a directory (ext/DynaLoader).
The ownership lists for core files was expressed as ext/DynaLoader, with
specific exclusions for XSLoader's files. These are now superfluous.

11 years agoMove remaining Tie::Hash::NamedCapture XS code to NamedCapture.xs
Nicholas Clark [Thu, 14 Oct 2010 14:34:03 +0000 (15:34 +0100)]
Move remaining Tie::Hash::NamedCapture XS code to NamedCapture.xs

Now all the support code for %+ and %- is contained in the module in ext/

11 years agoMove Tie::Hash::NamedCapture::{FIRST,NEXT}KEY to NamedCapture.xs
Nicholas Clark [Thu, 14 Oct 2010 13:09:15 +0000 (14:09 +0100)]
Move Tie::Hash::NamedCapture::{FIRST,NEXT}KEY to NamedCapture.xs

11 years agoConvert lib/Tie/Hash/NamedCapture.pm to an XS module in ext/
Nicholas Clark [Thu, 14 Oct 2010 12:29:22 +0000 (13:29 +0100)]
Convert lib/Tie/Hash/NamedCapture.pm to an XS module in ext/

Initially move only Tie::Hash::NamedCapture::flags from universal.c to it.

11 years agoMerge XS_Tie_Hash_NamedCapture_{FIRSTK,NEXTK} into S_named_capture_iter_common.
Nicholas Clark [Thu, 14 Oct 2010 10:41:50 +0000 (11:41 +0100)]
Merge XS_Tie_Hash_NamedCapture_{FIRSTK,NEXTK} into S_named_capture_iter_common.

11 years agoMerge XS_Tie_Hash_NamedCapture_STORE into S_named_capture_common.
Nicholas Clark [Thu, 14 Oct 2010 10:17:35 +0000 (11:17 +0100)]
Merge XS_Tie_Hash_NamedCapture_STORE into S_named_capture_common.

11 years agoMerge XS_Tie_Hash_NamedCapture_CLEAR into S_named_capture_common.
Nicholas Clark [Thu, 14 Oct 2010 10:04:51 +0000 (11:04 +0100)]
Merge XS_Tie_Hash_NamedCapture_CLEAR into S_named_capture_common.

11 years agoMerge XS_Tie_Hash_NamedCapture_DELETE into S_named_capture_common.
Nicholas Clark [Thu, 14 Oct 2010 09:44:15 +0000 (10:44 +0100)]
Merge XS_Tie_Hash_NamedCapture_DELETE into S_named_capture_common.

11 years agoMerge XS_Tie_Hash_NamedCapture_SCALAR into S_named_capture_common.
Nicholas Clark [Thu, 14 Oct 2010 09:24:45 +0000 (10:24 +0100)]
Merge XS_Tie_Hash_NamedCapture_SCALAR into S_named_capture_common.

11 years agoMerge XS_Tie_Hash_NamedCapture_{FETCH,EXISTS} into S_named_capture_common.
Nicholas Clark [Thu, 14 Oct 2010 09:08:08 +0000 (10:08 +0100)]
Merge XS_Tie_Hash_NamedCapture_{FETCH,EXISTS} into S_named_capture_common.

11 years agoExpand CALLREG_NAMED_BUFF* macros in XS_Tie_Hash_NamedCapture_*
Nicholas Clark [Thu, 14 Oct 2010 08:47:57 +0000 (09:47 +0100)]
Expand CALLREG_NAMED_BUFF* macros in XS_Tie_Hash_NamedCapture_*

This reveals even more similarity between the routines' bodies.

11 years agoFix x2p and utils dependencies for static builds
Andy Dougherty [Wed, 13 Oct 2010 17:27:12 +0000 (13:27 -0400)]
Fix x2p and utils dependencies for static builds

The x2p and utils .PL extractions use the Cwd module, which is not
available in miniperl in a static build.  (For a dynamic build, miniperl
only loads the pure perl portion of Cwd, but building the whole extension
ensures that the pure perl part is correctly built and available.  It might
be cleaner just to always require the full perl.)

11 years agoRemove or update pod references to regexp reëntrance.
Father Chrysostomos [Thu, 14 Oct 2010 06:41:38 +0000 (23:41 -0700)]
Remove or update pod references to regexp reëntrance.

11 years agowarnings.pl -> regen/warnings.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:29:57 +0000 (23:29 -0700)]
warnings.pl -> regen/warnings.pl

11 years agoregcomp.pl -> regen/regcomp.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:28:48 +0000 (23:28 -0700)]
regcomp.pl -> regen/regcomp.pl

11 years agoreentr.pl -> regen/reentr.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:26:14 +0000 (23:26 -0700)]
reentr.pl -> regen/reentr.pl

11 years agooverload.pl -> regen/overload.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:24:22 +0000 (23:24 -0700)]
overload.pl -> regen/overload.pl

11 years agoopcode.pl -> regen/opcode.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:18:36 +0000 (23:18 -0700)]
opcode.pl -> regen/opcode.pl

11 years agokeywords.pl -> regen/keywords.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:22:56 +0000 (23:22 -0700)]
keywords.pl -> regen/keywords.pl

11 years agoembed.pl -> regen/embed.pl
Father Chrysostomos [Thu, 14 Oct 2010 06:15:06 +0000 (23:15 -0700)]
embed.pl -> regen/embed.pl

so newcomers can find it more easily

11 years agoaf00134636ffe4172cbffeaed3bbad802e58d8a0 broke regen_perly.
Zefram [Thu, 14 Oct 2010 05:34:15 +0000 (22:34 -0700)]
af00134636ffe4172cbffeaed3bbad802e58d8a0 broke regen_perly.

11 years agoperldelta up to 825563b9
Father Chrysostomos [Thu, 14 Oct 2010 05:31:59 +0000 (22:31 -0700)]
perldelta up to 825563b9

11 years agoperldelta up to 7ec2c349
Father Chrysostomos [Thu, 14 Oct 2010 01:15:07 +0000 (18:15 -0700)]
perldelta up to 7ec2c349

11 years agoperldelta up to 07ffcb738
Father Chrysostomos [Thu, 14 Oct 2010 01:01:47 +0000 (18:01 -0700)]
perldelta up to 07ffcb738

11 years agoperldelta up to 91332126
Father Chrysostomos [Wed, 13 Oct 2010 23:31:28 +0000 (16:31 -0700)]
perldelta up to 91332126

11 years agoperldelta up to 2748c77610
Father Chrysostomos [Wed, 13 Oct 2010 21:53:09 +0000 (14:53 -0700)]
perldelta up to 2748c77610

11 years agoAdd tests for the usage messages for Tie::Hash::NamedCapture::*
Nicholas Clark [Wed, 13 Oct 2010 20:04:47 +0000 (21:04 +0100)]
Add tests for the usage messages for Tie::Hash::NamedCapture::*

11 years agoIn reg_nc_tie.t, use like() to get better diagnostics on failure.
Nicholas Clark [Wed, 13 Oct 2010 17:58:06 +0000 (18:58 +0100)]
In reg_nc_tie.t, use like() to get better diagnostics on failure.

11 years agoAdd perl5122delta
Florian Ragwitz [Wed, 13 Oct 2010 15:35:03 +0000 (17:35 +0200)]
Add perl5122delta

11 years agoStop cpan/CGI/t/http.t from failing
Father Chrysostomos [Wed, 13 Oct 2010 16:45:08 +0000 (09:45 -0700)]
Stop cpan/CGI/t/http.t from failing

A similar patch has been applied upstream. This will be overwritten
when the next CGI.pm is merged into blead, but at least we can get
the tests passing for now.

This does not affect any installed files.

11 years agoIn S_save_scalar_at, remove oldtainted, unneeded since 0cbee0a449cc4e11.
Nicholas Clark [Wed, 13 Oct 2010 16:07:34 +0000 (17:07 +0100)]
In S_save_scalar_at, remove oldtainted, unneeded since 0cbee0a449cc4e11.

0cbee0a449cc4e11 removed the call to mg_get(), and hence any possibility of
calling code with the side effect of changing PL_tainted.

11 years agoConsistent stack handling for XS_re_regnames_*
Nicholas Clark [Wed, 13 Oct 2010 15:16:11 +0000 (16:16 +0100)]
Consistent stack handling for XS_re_regnames_*

This may also fix bugs for the (untested) cases where the called routine
returns NULL, and the calling routine attempted XSRETURN_UNDEF.

11 years agoConsistent usage messages for XS_Tie_Hash_NamedCapture_*
Nicholas Clark [Wed, 13 Oct 2010 13:37:16 +0000 (14:37 +0100)]
Consistent usage messages for XS_Tie_Hash_NamedCapture_*

The $flags argument is actually the object, so should not be mentioned.

11 years agoIn XS_Tie_Hash_NamedCapture_{CLEAR,STORE}, free any returned value.
Nicholas Clark [Wed, 13 Oct 2010 13:12:27 +0000 (14:12 +0100)]
In XS_Tie_Hash_NamedCapture_{CLEAR,STORE}, free any returned value.

The calling convention for CALLREG_NAMED_BUFF_*() is to return NULL, or a
reference to a scalar. For CLEAR and STORE we return no values, so if we're
erroneously passed a reference, we should free it to stop it being leaked.

11 years agoConsistent stack handling for XS_Tie_Hash_NamedCapture_*
Nicholas Clark [Wed, 13 Oct 2010 12:56:38 +0000 (13:56 +0100)]
Consistent stack handling for XS_Tie_Hash_NamedCapture_*

This may also fix bugs for the (untested) cases where CALLREG_NAMED_BUFF_*
returned NULL, and the calling routine attempted XSRETURN_UNDEF.

It's fine to call sv_2mortal() on the immortals returned by
CALLREG_NAMED_BUFF_EXISTS()

11 years agoXS_Tie_Hash_NamedCapture_DELETE should return a value.
Nicholas Clark [Wed, 13 Oct 2010 10:09:02 +0000 (11:09 +0100)]
XS_Tie_Hash_NamedCapture_DELETE should return a value.

The core's regexp implementation will always croak, but other pluggable engines
may have read/write semantics, and hence return from their delete call.

Note that STORE and EXISTS are called in void context, hence their return stack
is discarded.

11 years agoperlfaq4: use given() in number/float/integer answer
brian d foy [Wed, 13 Oct 2010 07:40:49 +0000 (02:40 -0500)]
perlfaq4: use given() in number/float/integer answer

How do I determine whether a scalar is a number/whole/integer/float?

David Canzi also adjusted some of the regexes. The
real number and decimal number tests are really the
same thing, and we don't have to capture in the float
case.

11 years agoFix typos in comments
Karl Williamson [Tue, 12 Oct 2010 22:24:59 +0000 (16:24 -0600)]
Fix typos in comments

11 years agoperldelta entries for op-building functions and LINKLIST
Zefram [Wed, 13 Oct 2010 05:18:46 +0000 (22:18 -0700)]
perldelta entries for op-building functions and LINKLIST

11 years agocharnames.t: Make sure code point aliasess are right
Karl Williamson [Thu, 7 Oct 2010 04:16:56 +0000 (22:16 -0600)]
charnames.t: Make sure code point aliasess are right

Some code points have two (possibly more names).  This makes sure that
all work.

11 years agocharnames.t: Extract common code to subroutine
Karl Williamson [Thu, 7 Oct 2010 04:12:35 +0000 (22:12 -0600)]
charnames.t: Extract common code to subroutine

11 years ago[perl #78362] Make mro_package_moved check for recursion
Father Chrysostomos [Wed, 13 Oct 2010 05:07:17 +0000 (22:07 -0700)]
[perl #78362] Make mro_package_moved check for recursion

The existence of main::main::... caused mro_package_moved to break
Text::Template, and probably Acme::Meta as well.

11 years agoANSI C-ify the Perl_mro_isa_changed_in return mathoms.c
Tony Cook [Tue, 12 Oct 2010 23:12:35 +0000 (10:12 +1100)]
ANSI C-ify the Perl_mro_isa_changed_in return mathoms.c

A return statement with an expression shall not appear in a
function whose return type is void.

See http://source.test-smoke.org/tsdb?mode=report&rid=86779&top=86781
for an example build failure.

11 years agomktables: Change name of option to -annotate
Karl Williamson [Mon, 11 Oct 2010 19:04:02 +0000 (13:04 -0600)]
mktables: Change name of option to -annotate

This name is more descriptive of what this option has come to be.  The
previous option name has been released only in a few of the 5.13 series
Perls, and is for internal mktables development only, so backward
compatibility shouldn't be an issue.

11 years agomktables: Change variable name
Karl Williamson [Mon, 11 Oct 2010 19:00:59 +0000 (13:00 -0600)]
mktables: Change variable name

More than just names are output now, $output_names is renamed $annotate

11 years agomktables: Extend -output_names option
Karl Williamson [Mon, 11 Oct 2010 18:49:58 +0000 (12:49 -0600)]
mktables: Extend -output_names option

This option is not used for production, but is useful for someone
running mktables by hand who wants to compare Unicode versions, and
perhaps for debugging mktables.  It causes the code points in the output
tables to have comments added that give information about that code
point, so it isn't necessary to look up the hex numbers.

This patch is a significant expansion of the previous capability, and
adds the character's representation, and text for the non-named
characters.

I'm finding it useful in moving to Unicode 6.0.

11 years agomktables: Add arg to write() to say if in utf8
Karl Williamson [Mon, 11 Oct 2010 18:10:28 +0000 (12:10 -0600)]
mktables: Add arg to write() to say if in utf8

This will be used for annotations.  Currently no file is written in
utf8, so the calls all have 0 for the parameter.

11 years agomktables: Move method to different class
Karl Williamson [Mon, 11 Oct 2010 17:20:25 +0000 (11:20 -0600)]
mktables: Move method to different class

matches_identically_to() was a Table method, but it really applies to
the Range_List contained in the table.  Moving it to that class instead
will help with later code that wants to compare a Range_List that isn't
in a table.

11 years agomktables: Remove extraneous error message
Karl Williamson [Mon, 11 Oct 2010 16:47:04 +0000 (10:47 -0600)]
mktables: Remove extraneous error message

When I created matches_identically_to() originally, I just copied from
another subroutine, changing a few things.  I shouldn't have copied this
message, as it was only applicable to the original routine.

11 years agomktables: Change method name for clarity
Karl Williamson [Mon, 11 Oct 2010 16:33:06 +0000 (10:33 -0600)]
mktables: Change method name for clarity

is_equivalent_to() doesn't mean what it says.  Instead it means if the
two objects have been set equivalent to each other.

11 years agomktables: Finish the refactoring out of a pass.
Karl Williamson [Mon, 11 Oct 2010 03:59:25 +0000 (21:59 -0600)]
mktables: Finish the refactoring out of a pass.

Commit e1a3bfbd2f5194edc05010f0b5e5a568b1fdf508
omitted some code that was in my working version.  My git skills aren't
good enough to go back and add that in without zapping everything else.

11 years agomktables: Remove unshift onto large array
Karl Williamson [Mon, 11 Oct 2010 00:14:08 +0000 (18:14 -0600)]
mktables: Remove unshift onto large array

This changes the parameters to write() so it can accept more than one
array ref, thus eliminating the need for an unshift onto a large array.

11 years agomktables: Rmv obsolete field
Karl Williamson [Sun, 10 Oct 2010 22:36:06 +0000 (16:36 -0600)]
mktables: Rmv obsolete field

has_specials was used to avoid an extra pass for tables that didn't
need it.  That pass has been eliminated, so this code is useless.

11 years agomktables: refactor to eliminate a pass over tables
Karl Williamson [Sun, 10 Oct 2010 22:22:44 +0000 (16:22 -0600)]
mktables: refactor to eliminate a pass over tables

This patch is mainly for performance, but in eliminating a pass which
modified the tables, it allows for later changes to not have to rely on
things happening in a certain order.

Previously, tables that had special ranges in them, such as mappings to
multiple characters and character names that are algorithmically
derivable had a separate pass to process those ranges ahead of the main
table, as the information about these is output at the beginning of the
file.

What this patch does is to add a call-back for the processing of
the main body to call when it finds a special range that it doesn't
handle.  That call-back just adds the information to various
temporaries, depending on what the range is.

After the processing of all the ranges, those temporaries hold all the
information needed to output the specials data.  This is processed and
the text is unshifted to the beginning of the output.

pre_body() used to process all this.  But it has been split up.  The
code that does need to go at the beginning is moved to the write()
routine for the sub-class, which processes it before calling the super
class's write.  The code that figured out about the special ranges is
moved to the new call-back handle_special_range().  The remaining code
remains in pre_body(), but it is now called later in the process.

11 years agomktables: Add # XXX so can find experimental code
Karl Williamson [Sun, 10 Oct 2010 21:01:28 +0000 (15:01 -0600)]
mktables: Add # XXX so can find experimental code

11 years agomktables: Move 'format' to base class
Karl Williamson [Sun, 10 Oct 2010 15:58:24 +0000 (09:58 -0600)]
mktables: Move 'format' to base class

This patch is in preparation for adding more complete annotations to the
output tables.  Previously, only Map tables had a listed format.  Match
tables also have a format (empty), but that was understood.  But the new
annotation code will need to refer to that format, so I moved the field
into the base class common to both types of tables, and added the
EMPTY_FORMAT for use by Match tables.

11 years agomktables: Change format name for Decomposition.pl
Karl Williamson [Sun, 10 Oct 2010 15:41:44 +0000 (09:41 -0600)]
mktables: Change format name for Decomposition.pl

This table which is used only by Normalize.pm has a special format that
is used in no other table.  In preparation for adding annotations to it,
make a special format signifier for it.  This affects only one line at
the top that indicates to the user the format of the table.

The table has comments in it to say it is volatile and no one should use
this table besides Normalize.pm, which currently doesn't look at that
signifier.

11 years agomktables: change variable name for clarity
Karl Williamson [Sun, 10 Oct 2010 15:36:45 +0000 (09:36 -0600)]
mktables: change variable name for clarity

I found that 'youngest' was confusing

11 years agomktables: if modify during run, regen tables
Karl Williamson [Sun, 10 Oct 2010 15:28:09 +0000 (09:28 -0600)]
mktables: if modify during run, regen tables

This bug has bitten me enough times that I decided to fix it.  It would
only be applicable to someone who is developing mktables, and making
changes to it while the program is running.  Due to the complicated
nature of the dependencies of mktables, it basically functions as its
own make command, knowing when to rebuild or not based on the
modification times of itself and its input and output files.
Previously, it did not account for changes to itself made while running,
so it would think that it wouldn't have to rebuild, even though it did.

I solve this problem by keeping track of the time the process started,
and storing it in mktables.lst.  If something has changed since then,
then the files need to be rebuilt.

11 years agomktables: rewrite 'if' statement for clarity
Karl Williamson [Sun, 10 Oct 2010 15:08:05 +0000 (09:08 -0600)]
mktables: rewrite 'if' statement for clarity

11 years agomktables: complement variable name for rebuilding
Karl Williamson [Sun, 10 Oct 2010 14:57:56 +0000 (08:57 -0600)]
mktables: complement variable name for rebuilding

This is an intermediate patch that changes the variable name as to
whether the files need rebuilding or not to something I understand
better.  This has long been confusing, as up until a year ago, there had
been a long-standing bug in the program that didn't get it right.

11 years agomktables: Add type_of() method to get range's type
Karl Williamson [Sat, 9 Oct 2010 23:58:11 +0000 (17:58 -0600)]
mktables: Add type_of() method to get range's type

This will be useful in outputting better annotations.

11 years agomktables: Convert value_of() to use containing_range()
Karl Williamson [Sat, 9 Oct 2010 23:42:48 +0000 (17:42 -0600)]
mktables: Convert value_of() to use containing_range()

11 years agomktables: Remove extraneous debug statement
Karl Williamson [Sat, 9 Oct 2010 23:40:45 +0000 (17:40 -0600)]
mktables: Remove extraneous debug statement

11 years agomktables: Add containing_range method
Karl Williamson [Sat, 9 Oct 2010 23:33:30 +0000 (17:33 -0600)]
mktables: Add containing_range method

This method will be used in multiple places

11 years agomktables: Clarify comment in generated tables
Karl Williamson [Sat, 9 Oct 2010 23:29:35 +0000 (17:29 -0600)]
mktables: Clarify comment in generated tables

Tables whose values are strings, aren't arbitrary strings, as for
example they can't contain '#' characters that start comments.  So,
don't say they are 'arbitrary'.

11 years agomktables: Clarify comment in generated tables
Karl Williamson [Sat, 9 Oct 2010 22:49:59 +0000 (16:49 -0600)]
mktables: Clarify comment in generated tables

An installation may choose to output more tables than by default.  The
comment in tables should make it clearer that these are not necessarily
present.

11 years agomktables: Save reference to two commonly used tables
Karl Williamson [Sat, 9 Oct 2010 21:28:58 +0000 (15:28 -0600)]
mktables: Save reference to two commonly used tables

These two tables can be used frequently, so save a reference instead of
looking it up each time

11 years agomktables: Improve message
Karl Williamson [Sat, 9 Oct 2010 20:59:15 +0000 (14:59 -0600)]
mktables: Improve message

11 years agomktables: Remove commented out obsolete code
Karl Williamson [Sat, 9 Oct 2010 20:56:46 +0000 (14:56 -0600)]
mktables: Remove commented out obsolete code

This functionality is now in charnames.

11 years agomktables: Can't generate simple casing tables
Karl Williamson [Sat, 9 Oct 2010 20:35:08 +0000 (14:35 -0600)]
mktables: Can't generate simple casing tables

The simple as opposed to the full casing tables weren't able to be
generated correctly because I was passing the name of a table instead of
its object.  Normally these tables are generated at all, but I turned
generation of them on temporarily for development purposes, and found
this bug.  So there is no test case that can be generated for this fix.

11 years agomktables: Use more descriptive property names
Karl Williamson [Sat, 9 Oct 2010 20:29:45 +0000 (14:29 -0600)]
mktables: Use more descriptive property names

11 years agomktables: Make generated files use hex code points
Karl Williamson [Sat, 9 Oct 2010 20:18:56 +0000 (14:18 -0600)]
mktables: Make generated files use hex code points

Certain generated files had some values converted from hex into decimal.
All the published values are in hex, so should use the more familiar
form for readability.

11 years agomktables: fix typos in comments
Karl Williamson [Sat, 9 Oct 2010 20:04:24 +0000 (14:04 -0600)]
mktables: fix typos in comments

11 years agoperldelta entry for cv_set_call_checker
Zefram [Tue, 12 Oct 2010 17:32:06 +0000 (10:32 -0700)]
perldelta entry for cv_set_call_checker

plus tweaks from Reini Urban and the committer

11 years agoAdd LINKLIST to the API.
Ben Morrow [Mon, 11 Oct 2010 05:19:44 +0000 (06:19 +0100)]
Add LINKLIST to the API.

Also rename the underlying function to op_linklist, to match the other
API op functions.

11 years agoAPIify op list constructors
Zefram [Mon, 4 Oct 2010 21:19:08 +0000 (22:19 +0100)]
APIify op list constructors

Put into the API op_append_elem, op_prepend_elem, and op_append_list.  All
renamed from op_-less internal names.  Parameter types for op_append_list
changed to match the rest of the op API and avoid some casting.

11 years agoReset isa caches on nonexistent substashes when stash trees are moved
Father Chrysostomos [Tue, 12 Oct 2010 17:13:58 +0000 (10:13 -0700)]
Reset isa caches on nonexistent substashes when stash trees are moved

This fixes the problem of isa cache linearisations’ and method caches’
not being reset on nonexistent packages when they are replaced with
real packages as a result of parent stashes’ being moved. This can
happen in cases like this:

  @left::ISA = 'outer::inner';
  @right::ISA = 'clone::inner';
  {package outer::inner}
  *clone:: = \%outer::;
  print "ok 1", "\n" if left->isa("clone::inner");
  print "ok 2", "\n" if right->isa("outer::inner");

This changes mro_package_moved’s parameter list as documented in the
diff for mro.c. See also the new comments in that function.

11 years agoIn APItest.xs, augment croak("fail") with the file name and line number.
Nicholas Clark [Tue, 12 Oct 2010 19:41:36 +0000 (20:41 +0100)]
In APItest.xs, augment croak("fail") with the file name and line number.

With this, it should be possible to determine which C condition failed without
needing to run the test under a debugger. A debugger may still be needed to
determine *why*.

11 years agoXS_Tie_Hash_NamedCapture_STORE should always croak on invalid parameters.
Nicholas Clark [Tue, 12 Oct 2010 17:03:45 +0000 (18:03 +0100)]
XS_Tie_Hash_NamedCapture_STORE should always croak on invalid parameters.

Previously, it would return undef instead of calling Perl_croak_no_modify() if
PL_localizing was true. However, that case can never be reached. PL_localizing
is set non-zero for

1: Perl_mg_localize and PL_save_scalar, for the duration of executing the local
2: Perl_leave_scope for the duration of unwinding the local

However, XS_Tie_Hash_NamedCapture_STORE can't be reached in either case, with
PL_curpm NULL (or otherwise invalid) || !SvROK(ST(0)).

Case 1 would be the call to save_helem_flags() in pp_helem. However, this is
only reached if preeminent is TRUE, which will only hold if hv_exists_ent()
has already returned TRUE, which will only be possible if PL_curpm and ST(0)
are valid.

Case 2 would be the case SAVEt_HELEM in Perl_leave_scope(). However, this
case is only reached as part of the unwinding from Case 1 above, so again
PL_curpm and ST(0) will be valid, for this dynamic scope.

This commit is the patch proposed in RT ##77610. It does not resolve all issues
in that ticket.

Currently C<local $1> is legal - it's a runtime no-op, which neither errors
*nor* resets $1 to undef. Clearly C<local $+{k}> is inconsistent with this,
(as it errors at scope exit for all cases, and additionally errors at local
time if $+{k} exists) but I consider it not worth fixing until we decide
whether C<local $1>'s current behaviour is a "bug" or a "feature".

11 years agoCorrect mistaken references to &PL_sv_true and &PL_sv false in perlguts
Andrew Rodland [Tue, 12 Oct 2010 12:27:23 +0000 (07:27 -0500)]
Correct mistaken references to &PL_sv_true and &PL_sv false in perlguts

Their correct names are &PL_sv_yes and &PL_sv_no.

11 years agoMemoize has CPAN as its upstream
Florian Ragwitz [Tue, 12 Oct 2010 09:43:17 +0000 (11:43 +0200)]
Memoize has CPAN as its upstream

11 years agoImprove documentation of \Q
Eric Brine [Tue, 12 Oct 2010 00:17:15 +0000 (17:17 -0700)]
Improve documentation of \Q

11 years agoCorrect prototype of mathom Perl_mro_isa_changed_in() for ithreads.
Nicholas Clark [Mon, 11 Oct 2010 19:41:35 +0000 (20:41 +0100)]
Correct prototype of mathom Perl_mro_isa_changed_in() for ithreads.

6f86b615fa775fad forgot the pTHX_

11 years agoAllow mro_isa_changed_in to be called on nonexistent packages
Father Chrysostomos [Mon, 11 Oct 2010 17:10:06 +0000 (10:10 -0700)]
Allow mro_isa_changed_in to be called on nonexistent packages

This is necessary for an upcoming bug fix.

(For this bug:

 @left::ISA = 'outer::inner';
 @right::ISA = 'clone::inner';
 *clone:: = \%outer::;
 print left->isa('clone::inner'),"\n";
 print right->isa('outer::inner'),"\n";

)

This commit actually replaces mro_isa_changed_in with
mro_isa_changed_in3. See the docs for it in the diff for mro.c.

11 years agoAvoid calling Gcovert() when stringifying 0.0 and -0.0
Nicholas Clark [Mon, 11 Oct 2010 12:12:04 +0000 (13:12 +0100)]
Avoid calling Gcovert() when stringifying 0.0 and -0.0

Previously we always called Gconvert(), then tested its string output, then
fixed that up if necessary. This way has fewer tests. We can also reduce the
memory allocation for the string output for the case of zero.

11 years agoInstall l1_char_class_tab.h on VMS.
Craig A. Berry [Mon, 11 Oct 2010 03:56:49 +0000 (22:56 -0500)]
Install l1_char_class_tab.h on VMS.

11 years agoRemove some excess cleverness from the Bhk macros.
Ben Morrow [Sun, 10 Oct 2010 22:10:15 +0000 (23:10 +0100)]
Remove some excess cleverness from the Bhk macros.

Allowing BhkENTRY(bhk, start) to look up the bhk_start member defeats
much of the point of having a bhk_ prefix in the first place: if a
member is added later called (say) 'bhk_die', any invocation of
BhkENTRY(bhk, die) will expand to BhkENTRY(bhk, Perl_die) because of the
API macros. Requiring BhkENTRY(bhk, bhk_start), while tedious, is much
safer.

11 years agoplugin mechanism to rewrite calls to a subroutine
Zefram [Sun, 3 Oct 2010 13:53:16 +0000 (14:53 +0100)]
plugin mechanism to rewrite calls to a subroutine

New magic type PERL_MAGIC_checkcall attaches a function to a CV, which
will be called as the second half of the op checker for an entersub
op calling that CV.  Default state, in the absence of this magic,
is to process the CV's prototype if it has one, or apply list context
to all the arguments if not.  New API functions cv_get_call_checker()
and cv_set_call_checker() provide a clean interface to this facility,
hiding the internal use of magic.

Expose in the API the new functions rv2cv_op_cv(),
ck_entersub_args_list(), ck_entersub_args_proto(), and
ck_entersub_args_proto_or_list(), which are meaningful segments of
standard entersub op checking and are likely to be useful in plugged-in
call checker functions.

Expose new API function op_contextualize(), which is a public interface
to the internal scalar()/list()/scalarvoid() functions.  This API is
likely to be required in most plugged-in call checker functions.

Incidentally add new function mg_free_type(), in the API, which will
remove magic of one type from an SV.  (mg_free() removes all magic,
and there isn't anything else more selective.)

11 years agomake sv_clear() non-recursive on RVs
David Mitchell [Sun, 10 Oct 2010 23:13:07 +0000 (00:13 +0100)]
make sv_clear() non-recursive on RVs

The previous two commits made it non-recursive on AVs. With that
machinery in place, it's now trivial to extend it to RVs too.
This means that now any depth nesting of AVs and RVs will be freed
in a single call to sv_clear().