perl.git
6 years agotweak the way we mention if.pm
Ricardo Signes [Fri, 18 Jan 2013 03:21:12 +0000 (22:21 -0500)] 
tweak the way we mention if.pm

It should be clear even to readers of the documentation in text
format that "if" is a module and not the "if" flow control.

6 years agoAdded asserts() to check the arguments to S_copy_little_tm_to_big_TM.
Andy Dougherty [Thu, 17 Jan 2013 15:34:33 +0000 (10:34 -0500)] 
Added asserts() to check the arguments to S_copy_little_tm_to_big_TM.

The original version just zeroed dest if src == NULL, but that code path
was never used.  (gcc -Os inlined the function and optimized the test
away anyway.)

6 years agoremove an useless null check in S_copy_little_tm_to_big_TM
Daniel Dragan [Thu, 17 Jan 2013 07:52:45 +0000 (02:52 -0500)] 
remove an useless null check in S_copy_little_tm_to_big_TM

All callers of S_copy_little_tm_to_big_TM pass a true variable src.
Checking for null is pointless. On Win32 32 bit VC 2003 -O1 -GL,
perl517.dll's .text section went from 0xCO13F to 0xC012F bytes long. It can
be argued that the compiler should have figured this out on its own, but VC
for whatever reason didn't. Also pretty the indenting and align the
assignments. The null check blames to commit 806a119aef .

6 years agoutf8.c: Reword a warning message
Karl Williamson [Wed, 16 Jan 2013 22:05:04 +0000 (15:05 -0700)] 
utf8.c: Reword a warning message

This follows the suggestion by Aristotle Pagaltzis.

6 years agoperldiag: Move an out-of-alphabetical_order entry
Karl Williamson [Wed, 16 Jan 2013 22:03:51 +0000 (15:03 -0700)] 
perldiag: Move an out-of-alphabetical_order entry

6 years agoUpdate File-Path to CPAN version 2.09
Chris 'BinGOs' Williams [Wed, 16 Jan 2013 21:57:40 +0000 (21:57 +0000)] 
Update File-Path to CPAN version 2.09

  [DELTA]

  2.09 2013-01-13 21:09:12 UTC
    - merge two bugfixes from blead
      see https://github.com/dland/File-Path/pull/1

6 years agoUpdate Test to CPAN version 1.26
Chris 'BinGOs' Williams [Wed, 16 Jan 2013 20:38:54 +0000 (20:38 +0000)] 
Update Test to CPAN version 1.26

  [DELTA]

  1.26    2013-01-16  Jesse R Vincent <jesse@bestpractical.com>
    * Bump version to a stable number. No functional changes

6 years agoutf8.h: Add comments
Karl Williamson [Wed, 16 Jan 2013 19:49:50 +0000 (12:49 -0700)] 
utf8.h: Add comments

This also reorders one #define to be closer to a related one.

6 years agoAvoid wraparound when casting unsigned size_t to signed ssize_t.
Andy Dougherty [Wed, 16 Jan 2013 17:30:43 +0000 (12:30 -0500)] 
Avoid wraparound when casting unsigned size_t to signed ssize_t.

Practically, this only affects a perl compiled with 64-bit IVs on a 32-bit
system.  In that instance a value of count >= 2**31 would turn negative
when cast to (ssize_t).

6 years agoPATCH: [perl 116411]: code comment for commit 518a5310cc "Silence a MSVC++-specific...
bulk88 (via RT) [Wed, 16 Jan 2013 05:54:30 +0000 (21:54 -0800)] 
PATCH: [perl 116411]: code comment for commit 518a5310cc "Silence a MSVC++-specific warning"

There is no written investigation to google up for
the record. I don't want to forget that the #ifdef is benign and
accidentally reinvestigate it in the future. .text section of
perl517.dll was 0xC013F before and after the commit. No change.

-----------------------------------------------------------------

6 years agoin testing EISDIR string, check known value first
Ricardo Signes [Wed, 16 Jan 2013 13:56:57 +0000 (08:56 -0500)] 
in testing EISDIR string, check known value first

It was not enough to ensure the English value, as some platforms
use a different string entirely.  Rather than goof around with
figuring them out, just get the known value by making an EISDIR
and stringifying it, then compare to that.

6 years agoRevert "Upgrade Socket to CPAN version 2.008"
Ricardo Signes [Wed, 16 Jan 2013 15:05:47 +0000 (10:05 -0500)] 
Revert "Upgrade Socket to CPAN version 2.008"

This reverts commit 90ae46a10a094e68135385e525c78962d6572da3.

Socket 2.008 fails to build on Linux in core, and possibly other
platforms as well.

6 years agoUpgrade Socket to CPAN version 2.008
Ricardo Signes [Wed, 16 Jan 2013 14:36:47 +0000 (09:36 -0500)] 
Upgrade Socket to CPAN version 2.008

2012/12/27
2.008   CHANGES:
         * Fix uninitialised memory read (RT82119)

2012/12/16
2.007   CHANGES:
         * Test %Config keys for definedness, not mere existence (RT79854)
         * Fix missing argument in sprintf in Socket.xs (from perl.git
           5d6dfea82e1c4b6, RT82007)

6 years agoperlfunc.pod: fix incorrect short description of `length`
Aaron Crane [Wed, 16 Jan 2013 11:47:19 +0000 (11:47 +0000)] 
perlfunc.pod: fix incorrect short description of `length`

Thanks to ranguard for pointing out the report at
https://github.com/perlorg/perlweb/issues/65

6 years agoUpdate Test-Harness to CPAN version 3.26
Chris 'BinGOs' Williams [Wed, 16 Jan 2013 08:21:49 +0000 (08:21 +0000)] 
Update Test-Harness to CPAN version 3.26

  [DELTA]

  3.26    2013-01-16
        - Renamed env.opts.t to env_opts.t (for VMS)
        - Skipped some TAP::Formatter::HTML tests due to this bug: #82738

6 years agoRemove the Rhapsody port.
Nicholas Clark [Thu, 3 Jan 2013 15:32:26 +0000 (16:32 +0100)] 
Remove the Rhapsody port.

Rhapsody was an Apple OS that later evolved into Darwin and Mac OS X. It was
initially only released to developers, but later became Mac OS X Server, with
releases in 1999 and 2000. It was obsoleted by Mac OS X 10.0, released in
March 2001.

6 years agoNew PathTools CPAN release 3.40
Steffen Mueller [Wed, 16 Jan 2013 06:40:59 +0000 (07:40 +0100)] 
New PathTools CPAN release 3.40

6 years agoCwd/File::Spec version bump to 3.40
Steffen Mueller [Wed, 16 Jan 2013 06:40:23 +0000 (07:40 +0100)] 
Cwd/File::Spec version bump to 3.40

6 years agoavoid having to worry whether the test runs in a locale
Ricardo Signes [Tue, 15 Jan 2013 23:59:20 +0000 (18:59 -0500)] 
avoid having to worry whether the test runs in a locale

This is a time-honored tradition from such places as t/op.  Tony
Cook alerted me to failures caused by this test on machines smoking
in non-English locales.

6 years agoUpdate Digest-SHA to CPAN version 5.81
Chris 'BinGOs' Williams [Tue, 15 Jan 2013 10:32:29 +0000 (10:32 +0000)] 
Update Digest-SHA to CPAN version 5.81

  [DELTA]

  5.81  Mon Jan 14 05:17:08 MST 2013
        - corrected load subroutine (SHA.pm) to prevent double-free
              -- Bug #82655: Security issue - segfault
              -- thanks to Victor Efimov and Nicholas Clark
                 for technical expertise and suggestions

6 years agoSilence a MSVC++-specific warning
Steve Hay [Tue, 15 Jan 2013 09:04:20 +0000 (09:04 +0000)] 
Silence a MSVC++-specific warning

("function declared with __declspec(noreturn) has non-void return type" /
"function declared with __declspec(noreturn) has a return statement".)

6 years agohandy.h: Fix isIDCONT_utf8()
Karl Williamson [Mon, 14 Jan 2013 19:12:50 +0000 (12:12 -0700)] 
handy.h: Fix isIDCONT_utf8()

It was handling above-Latin1 code points as IDstarts instead of
continues.

6 years agoSilence a couple of warnings
Steve Hay [Mon, 14 Jan 2013 17:59:55 +0000 (17:59 +0000)] 
Silence a couple of warnings

("'initializing' : conversion from 'I32' to 'U8', possible loss of data"
and "formal parameter n different from declaration".)

6 years agoAdd warnings for "\08", /\017/
Karl Williamson [Sun, 13 Jan 2013 20:33:22 +0000 (13:33 -0700)] 
Add warnings for "\08", /\017/

This was discussed in thread
http://perl.markmail.org/thread/avtzvtpzemvg2ki2
but I never got around to this portion of the consensus, until now.

I did a cpan grep
http://grep.cpan.me/?q=%28^|[^\\]%29\\[0-7]{1%2C2}[8-9]&page=1

and eyeballing the results, saw three cases where this warning might
show up; one of which was for EBCDIC.  The others looked to be false
positives, such as in .css files.

6 years agobump threads::shared version 1.43
Jerry D. Hedden [Fri, 11 Jan 2013 15:53:42 +0000 (10:53 -0500)] 
bump threads::shared version 1.43

To sync with the forthcoming CPAN release.

6 years agoregcomp.c: Silence compiler warning
Karl Williamson [Mon, 14 Jan 2013 00:22:34 +0000 (17:22 -0700)] 
regcomp.c: Silence compiler warning

Some compilers say this can be used uninitialized. I don't believe them,
but silencing them will save effort in the long run.

6 years agoMake Data::Dumper XS ignore Freezer return value
Aaron Crane [Sun, 13 Jan 2013 10:08:02 +0000 (10:08 +0000)] 
Make Data::Dumper XS ignore Freezer return value

Fixes RT #116364

6 years agoregcomp.c: Stop potential leaks
Karl Williamson [Sun, 13 Jan 2013 20:31:38 +0000 (13:31 -0700)] 
regcomp.c: Stop potential leaks

These two ref counts were not at the same level as their corresponding
dec.

6 years agoregcomp.c: Add comments; align ternary operator for clarity
Karl Williamson [Sun, 13 Jan 2013 20:28:09 +0000 (13:28 -0700)] 
regcomp.c: Add comments; align ternary operator for clarity

6 years agoreg_mesg.t: Add, revise test names
Karl Williamson [Sun, 13 Jan 2013 18:46:27 +0000 (11:46 -0700)] 
reg_mesg.t: Add, revise test names

6 years agoembed.fnc: Clarify that varargs suppresses embed.h
Karl Williamson [Sun, 13 Jan 2013 18:11:20 +0000 (11:11 -0700)] 
embed.fnc: Clarify that varargs suppresses embed.h

Macro don't have variable numbers of args, hence the entry in embed.h is
suppressed.

6 years agobetter POD for unpackstring
Daniel Dragan [Sun, 13 Jan 2013 02:02:33 +0000 (21:02 -0500)] 
better POD for unpackstring

Someone asked me how to use unpackstring() since they couldn't figure it
out from the docs, so I added some things.

6 years agoCreate deprecated fncs to replace to-be-removed macros
Karl Williamson [Sun, 30 Dec 2012 03:18:09 +0000 (20:18 -0700)] 
Create deprecated fncs to replace to-be-removed macros

These macros should not be used as they are prone to misuse.  There are
no occurrences of them in CPAN.  The single use of either of them in
core has recently been removed (commit
8d40577bdbdfa85ed3293f84bf26a313b1b92f55), because it was a misuse.

Instead code should use isIDFIRST_lazy_if or isWORDCHAR_lazy_if
(isALNUM_lazy_if is also available, but can be confused with the Posix
alnum, which it doesn't mean).

6 years agoporting/args_assert.t: Needs to look in inline.h too
Karl Williamson [Sun, 30 Dec 2012 05:02:49 +0000 (22:02 -0700)] 
porting/args_assert.t: Needs to look in inline.h too

inline.h is a special header file that contains C functions, and hence
perhaps PERL_ARGS_ASSERTS.

6 years agoexpect a different error for `perl dir` on Win32
Ricardo Signes [Sat, 12 Jan 2013 12:12:34 +0000 (07:12 -0500)] 
expect a different error for `perl dir` on Win32

We've known that this is how Win32 behaves, as it was documented in
the ticket for which this is a fix.  I don't think it's worth the
bother of ensuring we get EISDIR, as long as we don't just exit
silently!

6 years agoperldelta for #61362, croak on `perl directory`
Ricardo Signes [Mon, 7 Jan 2013 20:00:08 +0000 (15:00 -0500)] 
perldelta for #61362, croak on `perl directory`

6 years agoAdd a test to detect error when attempting to syntax-check a directory.
James E Keenan [Sun, 6 Jan 2013 03:25:29 +0000 (22:25 -0500)] 
Add a test to detect error when attempting to syntax-check a directory.

For: RT #61362

6 years agocroak on an attempt to run a directory as a script
Ricardo Signes [Sun, 6 Jan 2013 01:30:48 +0000 (20:30 -0500)] 
croak on an attempt to run a directory as a script

How many times have I meant to run "perl -I lib myprog" but instead
run "perl lib myprog" only to exit 0 because that's what perl does
when you try to run a directory as a script (at least on unix)?  Many.

perl should croak when instructed to execute a directory.
[perl #61362] suggests it already does so on Win32.  Now it does it
everywhere.  Tests not yet written.

6 years agoSimplify ExtUtils::Install::_can_write_dir on VMS.
Craig A. Berry [Sat, 12 Jan 2013 22:30:17 +0000 (16:30 -0600)] 
Simplify ExtUtils::Install::_can_write_dir on VMS.

This reverts the code (but not test) portions of 3d55b451d9544fb.

The old solution of using catdir rather than a combination of
catdir and catpath works with and without extended filespecs
enabled, so there's no reason to maintain two versions, and thus
no reason to have all the boilerplate feature checking code at
the beginning of the module.

6 years agoinclude SvREADONLY() in SvIMMORTAL() test
David Mitchell [Sat, 12 Jan 2013 10:26:15 +0000 (10:26 +0000)] 
include SvREADONLY() in SvIMMORTAL() test

SvIMMORTAL() is currently defined as

      ((sv)==&PL_sv_undef || (sv)==&PL_sv_yes
    || (sv)==&PL_sv_no || (sv)==&PL_sv_placeholder)

Which is relatively slow. Some places do this:

    if (SvREADONLY(sv) && SvIMMORTAL(sv)) ...

which quickly rejects most times.

This commit simply moves the SvREADONLY test into the SvIMMORTAL macro
so that *all* uses benefit from this speedup.

6 years agoperlapi: Clarify av_pop
Karl Williamson [Sat, 12 Jan 2013 15:48:01 +0000 (08:48 -0700)] 
perlapi: Clarify av_pop

This notes that the caller now has control of a reference count of the
returned SV.

Wording mostly suggested by Paul Evans

6 years agoAllow slop on a few locale tests
Karl Williamson [Fri, 11 Jan 2013 21:29:29 +0000 (14:29 -0700)] 
Allow slop on a few locale tests

Four recently introduced tests in locale.t fail for two locales of all
the ones that get tested in our smoke farm.  I investigated the failures
and it looks to me like the problem in each case is that the locale
definition is defective.

The tests were added because of finding and fixing a bug in Perl, so I
don't want to remove them.  Instead these 4 tests will be marked as TODO
if at least 95% of locales pass on any given machine.

This works for our current smokers.

6 years agoperlre: fix typo
Aaron Crane [Sat, 12 Jan 2013 15:34:43 +0000 (15:34 +0000)] 
perlre: fix typo

6 years agoIn S_regatom(), set flags to 0 at the start.
Nicholas Clark [Sat, 12 Jan 2013 10:45:26 +0000 (11:45 +0100)] 
In S_regatom(), set flags to 0 at the start.

Without this, flags will be read uninitialised for the execution path that
ends up with "Internal urp". (Although it's not clear that that error message
is ever reachable.)

The bug was introduced in perl 5.000, which added (?#) comments and (?m)
style embedding flags. It's not present in alpha 9.

6 years agoMerge branch 'sets' into blead
Karl Williamson [Fri, 11 Jan 2013 18:51:18 +0000 (11:51 -0700)] 
Merge branch 'sets' into blead

This branch adds the experimental regular expression (?[ ]) feature.
Details are in the individual commit messages.

6 years agoAdd back-compat (?[ ]) tests
Karl Williamson [Fri, 11 Jan 2013 18:17:32 +0000 (11:17 -0700)] 
Add back-compat (?[ ]) tests

This adds testing of (?[ ]), using the same tests, t/re/re_tests<
as are used by many of the regular expression .t files.  Basically, it
converts the [bracketed] character classes in these tests to the new
syntax and verifies that they work there.

Some tests won't work in one or the other, and the capability to skip
depending on the .t is added

6 years agoreg_mesg.t: Test messages from (?[ ])
Karl Williamson [Fri, 11 Jan 2013 00:14:40 +0000 (17:14 -0700)] 
reg_mesg.t: Test messages from (?[ ])

6 years agoNew regex experimental feature: (?[ ])
Karl Williamson [Fri, 11 Jan 2013 00:06:04 +0000 (17:06 -0700)] 
New regex experimental feature: (?[ ])

This is a fancier [bracketed] character class which allows set
operations, such as intersection and subtraction.  The entry in perlre
for this commit details its operation.

Besides extending regular expressions to handle this functionality,
recommended by Unicode, the intent here is to do three things:

1) Intersection has been simulated by regexes using zero-width
   look-around assertions, which are non-obvious.  This allows replacing
   those with a more powerful and clearer syntax; the compiled regexes
   are smaller and faster.  Everything is known at compile time.
2) Set operations have also been simulated by using user-defined Unicode
   properties.  These are globals, have security implications,
   restricted names, and d don't allow as complex expressions as this
   new feature.
3) I hope that this feature will come to be viewed as a "better"
   bracketed character class.  I took advantage of the fact that there
   is no embedded base to have to be compatibile with to forbid certain
   iffy practices with the existing ones, while remaining mostly
   backwards compatible.  The main difference is that /x is always
   enabled, so white space can be pretty much freely used with these,
   but to specify a match on white space, it must be escaped.  Things
   that should have been illegal are, such as \x{}, and \x{abcdefghi}.
   Things that look like a posix specifier but don't quite meet the
   rules now give an error instead of silently compiling. e.g., [:digit]
   is an error instead of the union of the characters that compose it.
   I may have omitted things; perhaps it should be an error to have the
   same letter occur twice, adjacent.  Since this is experimental, we
   can make such changes based on field feed back.

The intent is to keep this feature, since it is strongly recommended by
Unicode.  The exact syntax is subject to change, so is experimental.

6 years agoregcomp.c: White-space only
Karl Williamson [Thu, 10 Jan 2013 23:59:33 +0000 (16:59 -0700)] 
regcomp.c: White-space only

Indents to newly formed blocks

6 years agoregcomp.c: Add capability for regclass() to return inversion list
Karl Williamson [Thu, 10 Jan 2013 23:42:19 +0000 (16:42 -0700)] 
regcomp.c: Add capability for regclass() to return inversion list

This is currently unused, but will have regclass() return an inversion
list instead of a node.

6 years agoregcomp.c: Add capability for ignoring white space inside []
Karl Williamson [Thu, 10 Jan 2013 23:36:05 +0000 (16:36 -0700)] 
regcomp.c: Add capability for ignoring white space inside []

This is currently unused, but allows pattern white space to be used
inside [bracketed] character classes.  When this is on, literal white
space must be escaped by a preceding backslash to be counted.

6 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.

6 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

6 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.

6 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.

6 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.

6 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

6 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.

6 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

6 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

6 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.

6 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.

6 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.

6 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
'else'

6 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

6 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

6 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.

6 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
those.

6 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.

6 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.

6 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.

6 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.

6 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.

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

And mention that use.perl.org is now read-only.

6 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.

6 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.

6 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

  [DELTA]

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

6 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.

6 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.

6 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

6 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

  [DELTA]

  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)

6 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.

6 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

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

6 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.

6 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
string.

6 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
needed.

6 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.

6 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.

6 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

  [DELTA]

  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.

6 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

  [DELTA]

  2.060 7 January 2013

      * No Changes

6 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

  [DELTA]

  2.060 7 January 2013

      * Mention SimpleZip in POD

6 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

  [DELTA]

  2013-01-07  Russ Allbery  <rra@stanford.edu>

       * ANSIColor.pm: Version 4.02 released.

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

       * 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.

6 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

6 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 libperl.so/libperl.a (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
needed.

6 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

6 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

6 years agoconstant-1.24 has been released to CPAN, update Maintainers.pl 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 Maintainers.pl to reflect this

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

6 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.

6 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.