This is a live mirror of the Perl 5 development currently hosted at
9 years agoregcomp.c: Add capability for stricter parsing of []
Karl Williamson [Thu, 10 Jan 2013 23:06:53 +0000 (16:06 -0700)]
regcomp.c: Add capability for stricter parsing of []

This adds the capability, currently unused, of forbidding certain things
in [bracketed] character classes.  Included are things that warn bug
still compile, such as false ranges, [\d-\w], and unrecognized escapes.

Also forbidden are potentially ambiguous cases where \x (without braces)
isn't followed by exactly two hex digits, or \000 where the number of
octal digits isn't precisely three.

9 years agoregcomp.c: Add capability for strict [:posix:]
Karl Williamson [Thu, 10 Jan 2013 22:47:24 +0000 (15:47 -0700)]
regcomp.c: Add capability for strict [:posix:]

This adds a parameter to regpposixcc() to enforce stricter rules on the
posix class syntax.  It is currently unused

9 years agoregcomp.c: Add function to skip pattern white space
Karl Williamson [Thu, 10 Jan 2013 22:42:36 +0000 (15:42 -0700)]
regcomp.c: Add function to skip pattern white space

The plan is to eventually convert all of regcomp to use this for white
space ignoring under /x, but this will be used for now in just the new
syntax for (?[ ]), coming in a few commits.  Until then, this function
is unused.

9 years agoregcomp.c: Comments, white space
Karl Williamson [Thu, 10 Jan 2013 22:36:22 +0000 (15:36 -0700)]
regcomp.c: Comments, white space

This also reverses the order of 2 macro calls in 2 places which will be
useful for future commits.

9 years agoregcomp.c: Add parameter to regclass()
Karl Williamson [Thu, 10 Jan 2013 22:15:21 +0000 (15:15 -0700)]
regcomp.c: Add parameter to regclass()

This parameter silences warnings for non-portable characters.  It
currently is always FALSE, meaning that warnings are given.

9 years agoregcomp.c: Add parameter to regclass()
Karl Williamson [Thu, 10 Jan 2013 22:03:39 +0000 (15:03 -0700)]
regcomp.c: Add parameter to regclass()

This parameter allows the caller to specify whether multi-character
folds should be allowed or not.  In general it should, and in the case
where this commit says it shouldn't, they never are returned anyway from
Unicode properties.

This capability will be put to real use by future commits

9 years agoregcomp.c: Add a vFAIL4 macro
Karl Williamson [Thu, 10 Jan 2013 19:40:44 +0000 (12:40 -0700)]
regcomp.c: Add a vFAIL4 macro

This one is missing.

9 years agoregcomp.c: Remove unnecessary variable
Karl Williamson [Thu, 10 Jan 2013 15:58:04 +0000 (08:58 -0700)]
regcomp.c: Remove unnecessary variable

This variable is used just in the statement following it.  Its value
doesn't need to be saved, can just combine the statements

9 years agoregcharclass.h: Add macro for finding pattern white space
Karl Williamson [Mon, 7 Jan 2013 15:50:56 +0000 (08:50 -0700)]
regcharclass.h: Add macro for finding pattern white space

This Unicode property will be used in future commits

9 years agogrok_bslash_[ox]: Add param to silence non-portable warnings
Karl Williamson [Mon, 7 Jan 2013 15:23:58 +0000 (08:23 -0700)]
grok_bslash_[ox]: Add param to silence non-portable warnings

If a hex or octal number is too big to fit in a 32 bit word, grok_oct
and grok_hex by default output a warning that it is a non-portable
value.  This new parameter to the grok_bslash functions can cause them
to shut up those warnings.  This is currently unused, but will be needed
in future commits.

9 years agodquote_static.c: refactor common code to initialization
Karl Williamson [Mon, 7 Jan 2013 15:18:54 +0000 (08:18 -0700)]
dquote_static.c: refactor common code to initialization

This flag bit is set in both branches of the code; it might as well be
initialized instead.

9 years agoregcomp.c: Save a test by moving a line of code
Karl Williamson [Mon, 7 Jan 2013 06:10:16 +0000 (23:10 -0700)]
regcomp.c: Save a test by moving a line of code

The condition for executing this moved code are met by the else clause a
few lines below.

9 years agoregcomp.c: Omit unnecessary 'else's
Karl Williamson [Mon, 7 Jan 2013 05:56:37 +0000 (22:56 -0700)]
regcomp.c: Omit unnecessary 'else's

The 'if' part of this croaks, so there is no need to specify this as an

9 years agoCreate new warnings category experimental::regex_sets
Karl Williamson [Mon, 7 Jan 2013 05:50:54 +0000 (22:50 -0700)]
Create new warnings category experimental::regex_sets

This on-by-default warning will be raised upon use of the
soon-to-be-delivered regex sets feature.  The syntax used is currently
invalid, so no feature activation is necessary.

The purpose of this warning is to make sure that any user is properly
notified that this feature is experimental and subject to change

9 years agodquote_static.c: White-space only; no code changes
Karl Williamson [Mon, 7 Jan 2013 05:49:05 +0000 (22:49 -0700)]
dquote_static.c: White-space only; no code changes

This properly indents a newly-formed block

9 years agoAdd optional strict mode to grok_bslash_[xo]
Karl Williamson [Mon, 7 Jan 2013 05:28:33 +0000 (22:28 -0700)]
Add optional strict mode to grok_bslash_[xo]

This mode croaks on any iffy constructs that currently compile.  It is
not currently used; documentation of the error messages will be
delivered later.

9 years agoBetter error pos for grok_bslash_[xo]
Karl Williamson [Mon, 7 Jan 2013 05:03:09 +0000 (22:03 -0700)]
Better error pos for grok_bslash_[xo]

These functions advance the parse pointer for the caller.  The regex
code has the infrastructure to output a marker as to where the error
was.  This commit simply moves the parse pointer past all the legal
digits in the input, which are likely supposed to be part of the number,
which makes it likely that the missing right brace point is just past

9 years agoRevise calling sequences for grok_bslash_[xo]
Karl Williamson [Mon, 7 Jan 2013 03:25:24 +0000 (20:25 -0700)]
Revise calling sequences for grok_bslash_[xo]

By passing the address of the parse pointer, the functions can advance
it, eliminating a parameter to the function, and simplifying the code in
the caller.

9 years agoregcomp.c: Use a parameter to simplify some code
Karl Williamson [Mon, 31 Dec 2012 18:54:44 +0000 (11:54 -0700)]
regcomp.c: Use a parameter to simplify some code

When parsing \p{} outside of a bracketed character class, code in
regcomp.c has pretended it is a bracketed character class by changing
and restoring the parsing pointers, and then calling the charclass
handler.  This code can be simplified by instead passing a flag to the
handler meaning to just parse one item.  The faking is simpler there,
with no restoring necessary.  Also we can eliminate the duplicate
handling of special cases.

Future commits will make more extensive use of this mechanism.

9 years agoregexp.t: Add a period in test name skip reason
Karl Williamson [Fri, 11 Jan 2013 18:19:20 +0000 (11:19 -0700)]
regexp.t: Add a period in test name skip reason

This is easier to read.

9 years agolib/warnings/regcomp: Add note about better place for these
Karl Williamson [Thu, 10 Jan 2013 02:00:54 +0000 (19:00 -0700)]
lib/warnings/regcomp: Add note about better place for these

reg_mesg.t has some more infrastructure, so it is probably easier
to add new tests there.

9 years agoembed.fnc: Fix flags for _invlist_dump
Karl Williamson [Thu, 10 Jan 2013 21:54:34 +0000 (14:54 -0700)]
embed.fnc: Fix flags for _invlist_dump

This debugging function is normally #ifdef'd out, but should it e
enabled, the flags were wrong.

9 years agopod/perlcommunity.pod: mention and Stack Overflow
Aaron Crane [Mon, 7 Jan 2013 13:29:34 +0000 (13:29 +0000)]
pod/perlcommunity.pod: mention and Stack Overflow

And mention that is now read-only.

9 years agoMake perlbug recommend perlcommunity.pod over Usenet
Aaron Crane [Mon, 7 Jan 2013 13:28:41 +0000 (13:28 +0000)]
Make perlbug recommend perlcommunity.pod over Usenet

Suggested by Dave Mitchell.

9 years agoexpand comment in Perl_pad_tidy
David Mitchell [Fri, 11 Jan 2013 15:04:47 +0000 (15:04 +0000)]
expand comment in Perl_pad_tidy

make it clear that /$var/ is only eval-able in the presence of
'use re eval'; then generally tidy and reformat the comment.

9 years agoUpdate File-Fetch to CPAN version 0.38
Chris 'BinGOs' Williams [Fri, 11 Jan 2013 08:13:59 +0000 (08:13 +0000)]
Update File-Fetch to CPAN version 0.38


  Changes for 0.38        Thu Jan 10 20:52:53 2013
  * Add support for an optional tempdir_root
    parameter (Kent Fredric)

9 years agoDon't use a fixed iteration count in dist/threads-shared/t/stress.t
Nicholas Clark [Mon, 3 Dec 2012 20:12:33 +0000 (21:12 +0100)]
Don't use a fixed iteration count in dist/threads-shared/t/stress.t

Looping 500,000 times takes between 0.025s and 1s depending on hardware and
optimisation levels on machines I have access to. For a fixed iteration
count, on a particularly slow machine the timeout can fire before all
threads have had a realistic chance to complete, but dropping the iteration
count will cause fast machines to finish each thread too quickly.

So use an initial busy loop (single-thread) to estimate a suitable iteration
count to use for the per-thread test loop.

9 years agoTweak the test from 2721a2ca2e5e889d to work with PERL_UNICODE set.
Nicholas Clark [Fri, 11 Jan 2013 10:07:53 +0000 (11:07 +0100)]
Tweak the test from 2721a2ca2e5e889d to work with PERL_UNICODE set.

9 years agoregcomp.c: Don't write to freed memory
Karl Williamson [Thu, 10 Jan 2013 20:28:49 +0000 (13:28 -0700)]
regcomp.c: Don't write to freed memory

This commit moves code in two places that may have freed memory just
before writing to it, so that the writing happens before the freeing.

It seems not worth the extra test if we are going to free this memory
before actually doing the write.

This bug was found by valgrind

9 years agoUpdate Archive-Extract to CPAN version 0.62
Chris 'BinGOs' Williams [Thu, 10 Jan 2013 08:13:46 +0000 (08:13 +0000)]
Update Archive-Extract to CPAN version 0.62


  Changes for 0.62    Wed Jan  9 21:38:10 2013
  * OpenBSD's tar does not like some archives
    on CPAN, prefer gtar if it is available.
  * Correctly detect lack of /bin/tar (Smylers)

9 years agostop -MO=Concise -e'm?x?' segfaulting
David Mitchell [Wed, 9 Jan 2013 11:46:26 +0000 (11:46 +0000)]
stop -MO=Concise -e'm?x?' segfaulting

The op_pmreplstart field in a PMOP is actually a union, containing
op_pmreplstart for OP_SUBST, and op_pmstash/op_pmstashoff for OP_MATCH
with PMf_ONCE set.

The B pmreplstart method just blindly treated the value of this field as
an op pointer, even when it wasn't an OP_SUBST. Hence the segfault.
Make it instead return a null pointer when not OP_SUBST.

At the same time I improved the PMOP handling code in Concise.xs so that
it expects particular types for that and the op_pmreplrootu union based on
the op type (OP_SUBST or OP_PUSHRE) rather than trying to handle any field
value regardless of op type.

9 years agoportability to Haiku-OS for the cachepropagate-*.t tests
Tony Cook [Wed, 9 Jan 2013 04:45:09 +0000 (15:45 +1100)]
portability to Haiku-OS for the cachepropagate-*.t tests

- defines but doesn't implement SO_TYPE
- doesn't support datagram unix sockets

9 years agot/re/re_tests: Remove duplicate test
Karl Williamson [Wed, 9 Jan 2013 04:27:33 +0000 (21:27 -0700)]
t/re/re_tests: Remove duplicate test

This is identical to the test two lines above.

9 years agot/re/re_tests: Remove unused variable initializations
Karl Williamson [Wed, 9 Jan 2013 04:22:16 +0000 (21:22 -0700)]
t/re/re_tests: Remove unused variable initializations

These lines all fail to compile, so matching doesn't happen, so it
doesn't matter at all what the target string to be matched against
is set to.  It is misleading to put apparently meaningful stuff in that

9 years agoregcomp.c: Fix a panic
Karl Williamson [Wed, 9 Jan 2013 04:12:14 +0000 (21:12 -0700)]
regcomp.c: Fix a panic

I haven't figured out a way to get this to panic with existing code
(hence no tests), but chanes I'm working on triggered it.

An inversion list has elements that indicate a range that matches code
points in the list; and elements for ranges that do not match code
points in the list.  This code was assuming that the final range in the
inversion list was in the list; it might not be, and so an extra test is

9 years agoregcomp.c: Avoid recalculating a value
Karl Williamson [Wed, 9 Jan 2013 04:06:54 +0000 (21:06 -0700)]
regcomp.c: Avoid recalculating a value

This value was calculated and saved in 'len' a couple of lines above.

9 years agoregexp.t: Skip tests that are supposed to
Karl Williamson [Wed, 9 Jan 2013 03:56:44 +0000 (20:56 -0700)]
regexp.t: Skip tests that are supposed to

This reorders some if elsif ... blocks so that skip is tested for and
done before actually trying the test.  This only affected tests which
were supposed to generate compiler errors.

9 years agoUpdate IO-Compress to CPAN version 2.060
Chris 'BinGOs' Williams [Tue, 8 Jan 2013 10:29:59 +0000 (10:29 +0000)]
Update IO-Compress to CPAN version 2.060


  2.060 7 January 2013

      * Updated POD
        RT# 82138: Example code not clear - gunzip() takes filenames!

      * IO::Compress::Base
        Remove the flush call when opening a filehandle.

9 years agoUpdate Compress-Raw-Bzip2 to CPAN version 2.060
Chris 'BinGOs' Williams [Tue, 8 Jan 2013 10:27:17 +0000 (10:27 +0000)]
Update Compress-Raw-Bzip2 to CPAN version 2.060


  2.060 7 January 2013

      * No Changes

9 years agoUpdate Compress-Raw-Zlib to CPAN version 2.060
Chris 'BinGOs' Williams [Tue, 8 Jan 2013 10:25:11 +0000 (10:25 +0000)]
Update Compress-Raw-Zlib to CPAN version 2.060


  2.060 7 January 2013

      * Mention SimpleZip in POD

9 years agoUpdate Term-ANSIColor to CPAN version 4.02
Chris 'BinGOs' Williams [Tue, 8 Jan 2013 10:23:00 +0000 (10:23 +0000)]
Update Term-ANSIColor to CPAN version 4.02


  2013-01-07  Russ Allbery  <>

       * Version 4.02 released.

       * Makefile.PL: Add the minimum Perl version to the package

       * t/basic256.t: List the tag first in the import list for
       compatibility with the Exporter from Perl 5.6.2.  Thanks to David
       Cantrell for the testing and debugging.

9 years agoskip appropriately when XS::APItest isn't available
Tony Cook [Tue, 8 Jan 2013 10:50:51 +0000 (21:50 +1100)]
skip appropriately when XS::APItest isn't available

XS::APItest isn't available under -Uusedl

9 years agoadd a dependency for dynamic extensions on $(LIBPERL)
Tony Cook [Tue, 8 Jan 2013 10:38:41 +0000 (21:38 +1100)]
add a dependency for dynamic extensions on $(LIBPERL)

This is necessary on some platforms (Haiku-OS, Cygwin and possibly
others) since EU::MM adds (with their appropriate
extensions) as a dependency to the link step.

Added as a dependency on all platforms because it causes no practical
harm (fewer choices for parallel making) on the platforms where it isn't

9 years agoregexec.c: Cast to silence compiler warnings
Karl Williamson [Mon, 7 Jan 2013 06:01:33 +0000 (23:01 -0700)]
regexec.c: Cast to silence compiler warnings

9 years agoregexec.c: Cast to silence compiler warning
Karl Williamson [Mon, 7 Jan 2013 06:00:15 +0000 (23:00 -0700)]
regexec.c: Cast to silence compiler warning

9 years agoconstant-1.24 has been released to CPAN, update to reflect this
Chris 'BinGOs' Williams [Mon, 7 Jan 2013 15:10:42 +0000 (15:10 +0000)]
constant-1.24 has been released to CPAN, update to reflect this

9 years agoForgot to update for podlators update
Chris 'BinGOs' Williams [Mon, 7 Jan 2013 15:09:22 +0000 (15:09 +0000)]
Forgot to update for podlators update

9 years agotoke.c: Silence compiler uninitialized warning
Karl Williamson [Mon, 7 Jan 2013 04:51:12 +0000 (21:51 -0700)]
toke.c: Silence compiler uninitialized warning

I believe that there really is not a path through this that yields an
uninitialized value for this variable, but initialize it to UV_MAX in
case there is, that value will hopefully stand out more than '0' would.

9 years agoregcomp.c: Silence compiler warning
Karl Williamson [Mon, 7 Jan 2013 03:01:28 +0000 (20:01 -0700)]
regcomp.c: Silence compiler warning

One of our smoke compilers is not smart enough to know that the
remainder when dividing by two can only be a 0 or a 1.

9 years agoregcomp.c: Silence compiler warning by using a cast
Karl Williamson [Mon, 7 Jan 2013 02:51:16 +0000 (19:51 -0700)]
regcomp.c: Silence compiler warning by using a cast

9 years agoregcomp.c: Comments only, white-space ; no code changes
Karl Williamson [Mon, 7 Jan 2013 02:35:00 +0000 (19:35 -0700)]
regcomp.c: Comments only, white-space ; no code changes

9 years agoregcomp.c: Silence compiler warning
Karl Williamson [Mon, 7 Jan 2013 02:32:01 +0000 (19:32 -0700)]
regcomp.c: Silence compiler warning

One compiler doesn't like this, so cast to bool

9 years agopp_ctl.c: Silence compiler warning.
Karl Williamson [Mon, 7 Jan 2013 02:24:13 +0000 (19:24 -0700)]
pp_ctl.c: Silence compiler warning.

A compiler on one of our smoke platforms wants empty braces here.

9 years agoext/XS-APItest: Remove unused variable decl.
Karl Williamson [Mon, 7 Jan 2013 02:19:43 +0000 (19:19 -0700)]
ext/XS-APItest: Remove unused variable decl.

This is to stop a compiler warning

9 years agoembed.fnc: Properly declare fcn inline
Karl Williamson [Mon, 7 Jan 2013 02:17:42 +0000 (19:17 -0700)]
embed.fnc: Properly declare fcn inline

This function is specified as inline in the source code, but not in the
prototypes; only one compiler seems to have noticed.

9 years agoperlunicode: Refactor verbatim lines into =item ones
Karl Williamson [Fri, 4 Jan 2013 04:45:34 +0000 (21:45 -0700)]
perlunicode: Refactor verbatim lines into =item ones

This is so that they are able to contain links; both versions render
uglily in different ways.

9 years agoperlunicode: Correct false statement
Karl Williamson [Fri, 4 Jan 2013 04:15:55 +0000 (21:15 -0700)]
perlunicode: Correct false statement

This text is just plain wrong.  It mixes up which is the block property
and which is the script

9 years agoregcomp.c: White-space only; no code changes
Karl Williamson [Tue, 1 Jan 2013 04:55:40 +0000 (21:55 -0700)]
regcomp.c: White-space only; no code changes

This outdents a block to the proper alignment, and reflows the comments

9 years agoMake Pod::Html::_unixify unescape dots on VMS.
Craig A. Berry [Sun, 6 Jan 2013 22:27:50 +0000 (16:27 -0600)]
Make Pod::Html::_unixify unescape dots on VMS.

Dots in directory names must be escaped in native syntax, but those
escapes confuse various things if left in during a conversion to
Unix syntax.

9 years agoPod::Html test portability redux.
Craig A. Berry [Fri, 4 Jan 2013 23:41:17 +0000 (17:41 -0600)]
Pod::Html test portability redux.

The flurry of activity to get these tests passing in the lead-up to
5.16.0 swept several issues under the rug.  In particular, some
of the tests had failures on VMS that became invisible once we
started skipping them wherever dots are not allowed in directory
names (which is true by default on VMS now, but is likely to change
very shortly).

The facts behind my changes, in no particular order:

File::Spec->rootdir() returns, on VMS, a a special string that will
not be found in any actual pathname, so taking the length of it and
stripping that much off the front of an actual pathname produces

File::Spec->rootdir(), on VMS, returns a string that contains a volume
specification, so simply prepending a volume to it produces garbage.
To portably produce a complete path, use catpath().

Windows appears not to care which direction the slashes are leaning,
and VMS can handle pure-Unix paths as long as they aren't pasted onto
components of native paths, so it's reasonbly safe to obtain directory
components with splitdir, and join them together with forward slashes.

The tests pass a value to --podpath that is not absolute, but nor
is it relative in the usual sense.  It's apparently intended to be
relative to the filesystem root, which is an inherently non-portable
concept as neither Windows nor VMS has such a thing.  The best that
can be done (without a complete rethink of the tests and perhaps the
module) is to make it relative to the top-level directory on the
current volume.

This gets all tests passing on Windows, VMS, and OS X.

9 years agoadd a LEAVE for the ENTER in LOAD_UTF8_CHARCLASS
Tony Cook [Sun, 6 Jan 2013 11:25:51 +0000 (22:25 +1100)]

This appears to fix the regression introduced in c7304fe2c

9 years agot/op/not.t: Add descriptions to all tests.
James E Keenan [Sat, 29 Dec 2012 16:23:56 +0000 (11:23 -0500)]
t/op/not.t: Add descriptions to all tests.

Also, add note() before tests 4 and 5 explaining rationale for addition of
parentheses to second arguments.

9 years agoAdd parens around second argument to tests 4 and 5.
James E Keenan [Sat, 29 Dec 2012 16:09:34 +0000 (11:09 -0500)]
Add parens around second argument to tests 4 and 5.

If a description were to be added to these tests, in the absence of
parentheses the scalar prototype of CORE::not would enforce a scalar context
onto the balance of the statement, leading to apparently anomalous behavior,
viz., the descriptions would not be printed and test 5 would be reported to

9 years agot/op/oct.t: Add descriptions to tests lacking them.
James E Keenan [Fri, 28 Dec 2012 02:51:54 +0000 (21:51 -0500)]
t/op/oct.t: Add descriptions to tests lacking them.

9 years agot/op/or.t: Provide descriptions for tests lacking them.
James E Keenan [Thu, 27 Dec 2012 02:28:32 +0000 (21:28 -0500)]
t/op/or.t: Provide descriptions for tests lacking them.

9 years agot/op/pos.t: Provide descriptions for remaining tests lacking them.
James E Keenan [Thu, 27 Dec 2012 02:15:40 +0000 (21:15 -0500)]
t/op/pos.t: Provide descriptions for remaining tests lacking them.

9 years agot/op/pwent.t: Add descriptions for all tests.
James E Keenan [Wed, 26 Dec 2012 02:37:04 +0000 (21:37 -0500)]
t/op/pwent.t: Add descriptions for all tests.

9 years agot/op/pow.t: Add descriptions for tests lacking them.
James E Keenan [Wed, 26 Dec 2012 02:35:25 +0000 (21:35 -0500)]
t/op/pow.t: Add descriptions for tests lacking them.

Those which were still lacking descriptions were all testing that
exponentiation has precedence over negation.

9 years agot/op/fork.t: Add description for one special test.
James E Keenan [Wed, 26 Dec 2012 02:34:29 +0000 (21:34 -0500)]
t/op/fork.t: Add description for one special test.

9 years agocharnames: Deprecate character names with spacing issues
Karl Williamson [Fri, 4 Jan 2013 19:34:34 +0000 (12:34 -0700)]
charnames: Deprecate character names with spacing issues

A user-defined character name with trailing or multiple spaces in a row
is likely a typo, and hence likely won't match what the other uses of
it.  These names also won't work if we extend :loose to these.  This
now generates a warning.

9 years agocharnames: :alias alone implies :full
Karl Williamson [Fri, 4 Jan 2013 18:05:32 +0000 (11:05 -0700)]
charnames: :alias alone implies :full

The documentation says this is how it should behave, but only 1 of the
three paths in the code did it, and in fact there was a test to the

I'm only adding a test for one of the two fixed paths, as the other one
appears to require a weird file name.

9 years agocharnames: Enforce better file name rules
Karl Williamson [Fri, 4 Jan 2013 18:04:08 +0000 (11:04 -0700)]
charnames: Enforce better file name rules

The code says that identifier syntax is required for a file name, but
the code accepted any sequence of \w's.

9 years agocharnames: comment, error message clarifications
Karl Williamson [Fri, 4 Jan 2013 16:06:26 +0000 (09:06 -0700)]
charnames: comment, error message clarifications

9 years agocharnames pod: Typo, clarification
Karl Williamson [Tue, 1 Jan 2013 18:52:58 +0000 (11:52 -0700)]
charnames pod: Typo, clarification

9 years agoFix SvREFCNT_dec doc typo
Steven Schubiger [Fri, 4 Jan 2013 17:05:08 +0000 (18:05 +0100)]
Fix SvREFCNT_dec doc typo

9 years agomake m?$pat? match only once under ithreads
David Mitchell [Fri, 4 Jan 2013 16:10:53 +0000 (16:10 +0000)]
make m?$pat? match only once under ithreads

[perl #115080]

m?...? is only supposed to match once, until reset. Normally this is done
by setting the PMf_USED flag on the PMOP. Under ithreads we can't modify
ops, so instead we indicate by setting the regex's SV to readonly. (This
is a bit of a hack: the flag should be associated with the PMOP, not the

This breaks with run-time regexes when the pattern gets recompiled; for

    for my $c (qw(a b c)) {
print "matched $c\n" if $c =~ m?^$c$?;


    matched a

on unthreaded, but

    matched a
    matched b
    matched c

on threaded.

The re_eval jumbo fix made this more noticeable by sometimes recompiling
even when the pattern text hasn't changed (to make closures work ok).

The quick fix is to propagate the readonlyness of the old re to the new
re. (The proper fix would be to store the flag state in a pad slot
associated with the PMOP).
Needless to say, I've gone for the quick fix.

9 years agoRegenerated customized.dat file due to pod2*.PL changes
Chris 'BinGOs' Williams [Fri, 4 Jan 2013 15:26:28 +0000 (15:26 +0000)]
Regenerated customized.dat file due to pod2*.PL changes

9 years agoUpdate podlators to CPAN version 2.5.0
Chris 'BinGOs' Williams [Fri, 4 Jan 2013 14:33:58 +0000 (14:33 +0000)]
Update podlators to CPAN version 2.5.0


2013-01-02  Russ Allbery  <>

  * VERSION: podlators 2.5.0 released.

  * lib/Pod/ (guesswork): Extend a small-caps section through
  the punctuation that commonly appears in license disclaimers so
  that small caps isn't turned on and off at the boundaries of every
  word, producing unreadable *roff.
  * t/man.t: Test the updated small-caps behavior.

  * lib/Pod/ (new): Support a new errors option with values
  die, stderr, pod, and none.  Convert the stderr option to the
  errors option with value stderr.  (#39007)
  (end_document): Die if errors was set to die and there were POD
  syntax errors.
  * lib/Pod/ (new): Likewise.
  (end_document): Likewise.
  * scripts/pod2man: Support a new --errors option that sets the
  underlying errors formatter option and default to dying on POD
  syntax errors.
  * scripts/pod2text: Likewise.
  * t/man-options.t: Add support for catching exceptions during
  formatting.  Test valid and invalid errors settings.
  * t/text-options.t: Likewise.

  * lib/Pod/ (cmd_l): Add nourls option to suppress the URL
  from L<> formatting codes that contain anchor text.  (#62210)
  * lib/Pod/ (cmd_l): Likewise.
  * scripts/pod2man: New --nourls option to set the corresponding
  option in Pod::Man.
  * scripts/pod2text: Likewise for Pod::Text.
  * t/man-options.t: Test new nourls option.
  * t/text-options.t: Likewise.

  * lib/Pod/ (outindex): Also collapse consecutive whitespace
  in index term text.
  * t/man.t: Test for handling of index terms containing whitespace.

2013-01-02  Kevin Ryde  <>

  * lib/Pod/ (outindex): Remove newlines from index term text
  since *roff quotes don't span lines.  (#82332)

2012-06-01  Russ Allbery  <>

  * VERSION: podlators 2.4.2 released.

  * t/text-encoding.t: Remove the test of a POD document without an
  encoding.  We previously tested that this interpreted the document
  as ISO 8859-1, but Pod::Simple behavior has changed so that the
  test started failing, plus Pod::Simple now warns about a missing
  =encoding.  (#77553)

9 years agoCollapse "if(a){b}else{b}" to "b"
Eric Brine [Wed, 5 Dec 2012 07:48:56 +0000 (02:48 -0500)]
Collapse "if(a){b}else{b}" to "b"

9 years agoRemove redundant NULL checks.
Eric Brine [Wed, 5 Dec 2012 06:30:25 +0000 (01:30 -0500)]
Remove redundant NULL checks.

It's safe to pass NULLs to SvREFCNT_dec.

9 years agoAdd support for building with VC++ 2012
Steve Hay [Thu, 3 Jan 2013 23:30:57 +0000 (23:30 +0000)]
Add support for building with VC++ 2012

No code changes seem to be required, so it's just a matter of noting that
we've tried it, it works, and we'll support it :-)

9 years agoS_has_runtime_code(): avoid buffer overrun
David Mitchell [Thu, 3 Jan 2013 14:17:25 +0000 (14:17 +0000)]
S_has_runtime_code(): avoid buffer overrun

This function looks for '(?{' style strings in a pattern. If the last char
in the pattern was '(', it could read a couple of bytes off the end of
the pattern. This is harmless from a logic and security viewpoint since
false positives are ok; but I'm still fixing it for correctness's sake.

9 years agoFix period at the end of perldiag.pod and remove entry from TODO in diag.t
Matthew Horsfall (alh) [Wed, 26 Dec 2012 15:30:02 +0000 (10:30 -0500)]
Fix period at the end of perldiag.pod and remove entry from TODO in diag.t

9 years agoEliminate redundant words.
James E Keenan [Thu, 3 Jan 2013 02:47:37 +0000 (21:47 -0500)]
Eliminate redundant words.

Based on observation by Joaquin Ferrero++.

For RT #116274

9 years agoDon't usenm on haiku
Tony Cook [Wed, 2 Jan 2013 22:36:27 +0000 (09:36 +1100)]
Don't usenm on haiku

This cam up from building perl with nm enabled and make test_harness
croaking with the complaint that times wasn't implemented.

nm /system/lib/ | grep times produces:

00095540 T __times
00095460 T __times_beos
0009b690 t itimerspec_to_bigtimes__FRC10itimerspecRxT1
00095540 T times@@LIBROOT_1_ALPHA4
00095460 T times@LIBROOT_BASE
000981c8 t timesub
000958b4 T utimes

of which the two times@... entries appear to be version symbols.

The system perl on Haiku-OS alpha4.1 is built with -Uusenm

9 years agoMerge Shlomi's latest debugger refactoring work
Tony Cook [Wed, 2 Jan 2013 03:23:22 +0000 (14:23 +1100)]
Merge Shlomi's latest debugger refactoring work

9 years agobump $VERSION for lib/
Tony Cook [Wed, 2 Jan 2013 00:47:08 +0000 (11:47 +1100)]
bump $VERSION for lib/

There's been a release since this change was submitted.

9 years agoMissing test count.
Shlomi Fish [Mon, 24 Dec 2012 22:21:44 +0000 (00:21 +0200)]
Missing test count.


9 years agoFix perl -d's 'w $my_lexical_variable'.
Shlomi Fish [Sun, 9 Dec 2012 21:58:38 +0000 (23:58 +0200)]
Fix perl -d's 'w $my_lexical_variable'.

This was done by reverting parts of the offending commit from the
git bisecting and adding a test. Thanks to Kevin Dawson for the report.

9 years agoDocument HistSize and HistFile.
Shlomi Fish [Sat, 1 Dec 2012 20:46:49 +0000 (22:46 +0200)]
Document HistSize and HistFile.

Two undocumented perldebug options.

9 years agoStart refactoring "sub restart".
Shlomi Fish [Sun, 18 Nov 2012 11:33:27 +0000 (13:33 +0200)]
Start refactoring "sub restart".

9 years agoExtract _dump_trace_calc_saved_single_arg.
Shlomi Fish [Sun, 18 Nov 2012 10:14:12 +0000 (12:14 +0200)]
Extract _dump_trace_calc_saved_single_arg.

9 years agoExtract _dump_trace_calc_save_args.
Shlomi Fish [Sun, 18 Nov 2012 09:58:29 +0000 (11:58 +0200)]
Extract _dump_trace_calc_save_args.

9 years agoAvoid pre-declarations.
Shlomi Fish [Sun, 18 Nov 2012 09:47:10 +0000 (11:47 +0200)]
Avoid pre-declarations.

9 years agoFixed the test - it passes now.
Shlomi Fish [Fri, 16 Nov 2012 07:57:55 +0000 (09:57 +0200)]
Fixed the test - it passes now.

Thanks to Father Chrysostomos for fixing the bug in blead, I was now
able to fix the test in the debugger.

9 years agoAdd a testcase for the bug.
Shlomi Fish [Thu, 15 Nov 2012 15:54:26 +0000 (17:54 +0200)]
Add a testcase for the bug.

It is caused by $obj spontaneously becoming undef for no good reason -
seems like a bug in the perl interpreter (or elsewhere). This was
noticed by Father C here:

9 years agoExtract _cmd_L_handle_postponed_breakpoints.
Shlomi Fish [Thu, 15 Nov 2012 09:08:13 +0000 (11:08 +0200)]
Extract _cmd_L_handle_postponed_breakpoints.

9 years agoAdd empty lines and an explicit return.
Shlomi Fish [Wed, 14 Nov 2012 17:15:50 +0000 (19:15 +0200)]
Add empty lines and an explicit return.

9 years agoRemove a no-longer needed comment.
Shlomi Fish [Wed, 14 Nov 2012 11:45:23 +0000 (13:45 +0200)]
Remove a no-longer needed comment.

9 years agoExtract a subroutine.
Shlomi Fish [Wed, 14 Nov 2012 11:38:37 +0000 (13:38 +0200)]
Extract a subroutine.