perl.git
3 years agoAdd new release to perlhist v5.23.9
Abigail [Sun, 20 Mar 2016 13:53:00 +0000 (14:53 +0100)] 
Add new release to perlhist

3 years agoChanged modules section for perldelta.pod
Abigail [Sun, 20 Mar 2016 13:25:31 +0000 (14:25 +0100)] 
Changed modules section for perldelta.pod

3 years agoAdded Acknowledgements
Abigail [Sun, 20 Mar 2016 12:45:33 +0000 (13:45 +0100)] 
Added Acknowledgements

3 years agoUpdate Module::CoreList for 5.23.9
Abigail [Sun, 20 Mar 2016 12:25:37 +0000 (13:25 +0100)] 
Update Module::CoreList for 5.23.9

3 years agoMention (relative) path to Module::CoreList's Changes file.
Abigail [Sun, 20 Mar 2016 12:21:42 +0000 (13:21 +0100)] 
Mention (relative) path to Module::CoreList's Changes file.

So we can easily copy-and-paste it.

3 years agoImprison at_tindex_nomg()
Karl Williamson [Sun, 20 Mar 2016 04:48:54 +0000 (22:48 -0600)] 
Imprison at_tindex_nomg()

This name is controversial.  Use #ifdef's to keep it from being exposed
outside of the places that already use it.  And don't publicize that it
exists.

3 years agoregcomp.c: Fix -Dr output.
Karl Williamson [Sun, 20 Mar 2016 04:02:32 +0000 (22:02 -0600)] 
regcomp.c: Fix -Dr output.

I got things slightly wroing in
b77aba85f6ec3b2a2341077b14f39261c5753cea.  It did not properly handle
the case for inversion /d of complemented classes, such as \W.  Two
examples being [^\W0-9] and [^\S\wa].

3 years agoDocument the changes for perl 5.23.9
Abigail [Sat, 19 Mar 2016 23:29:26 +0000 (00:29 +0100)] 
Document the changes for perl 5.23.9

We still need to take care of the automated parts (new/changed/delete module,
and the Acknowledgements.

3 years agoconfigure.com update for newlocale, freelocale, and uselocale.
Craig A. Berry [Sat, 19 Mar 2016 23:39:52 +0000 (18:39 -0500)] 
configure.com update for newlocale, freelocale, and uselocale.

Follow-up to ecb44b8e4ad52.

3 years agoSort perldiag
Father Chrysostomos [Sat, 19 Mar 2016 20:30:39 +0000 (13:30 -0700)] 
Sort perldiag

It’s nice that people leave things for me to clean up! :-)

3 years agoregcomp.c, perl.c: Use 'VOL' not 'volatile'
Karl Williamson [Sat, 19 Mar 2016 15:10:41 +0000 (09:10 -0600)] 
regcomp.c, perl.c: Use 'VOL' not 'volatile'

I presume that VOL is so perl can at least compile on compilers that lack
'volatile'.  (It was added by Larry in 1991 by 352d5a3ab; there were
probably a lot more compilers like that back then.)  But VOL is used
consistently in the core, with these being the two exceptions.

3 years agoregen
H.Merijn Brand [Sat, 19 Mar 2016 14:58:17 +0000 (15:58 +0100)] 
regen

3 years agoAdd Configure probes for newlocale, freelocale, and uselocale
H.Merijn Brand [Sat, 19 Mar 2016 14:01:58 +0000 (15:01 +0100)] 
Add Configure probes for newlocale, freelocale, and uselocale

3 years agoUpdate version to 0.9916
John Peacock [Sat, 19 Mar 2016 01:43:52 +0000 (21:43 -0400)] 
Update version to 0.9916

3 years agoUpgrade Win32API::File from version 0.1202 to 0.1203
Steve Hay [Sat, 19 Mar 2016 01:28:40 +0000 (01:28 +0000)] 
Upgrade Win32API::File from version 0.1202 to 0.1203

Fixes perl #125303.

(Includes a regen for the moved Myconst2perl.pm.)

3 years ago[MERGE] dtrace linking fixups
David Mitchell [Sat, 19 Mar 2016 00:01:51 +0000 (00:01 +0000)] 
[MERGE] dtrace linking fixups

3 years agofix up dtrace compile/link for Solaris
David Mitchell [Tue, 8 Mar 2016 11:31:22 +0000 (11:31 +0000)] 
fix up dtrace compile/link for Solaris

[perl #127543]

On some platforms, the use of dtrace / SystemTap requires generating an
extra .o file from a list of .o files before linking. For example,

    cc -o foo a.o b.o c.o

has to be replaced with

    dtrace -G -s dtrace.d -o dtrace.o  a.o b.o c.o # creates dtrace.o
    cc -o foo                dtrace.o  a.o b.o c.o

On Solaris in particular, "dtrace -G" modifies the *.o files that it's
passed as well as creating dtrace.o, and all the new/updated .o files need
to be linked together at the same time from the same single use of "dtrace
-G".

This complicates matters when building all of miniperl, libperl and perl,
and the reason for this commit is that once a dtrace probe made its way
into an inline static function via the recent context work, Solaris
stopped building under -Dusedtrace -Duseshrplib.

The fix that seems to work under both Solaris and Linux, for all
4 permutations of -Dusedtrace +/- -Duseshrplib, is (approx):

    # compile all the *.o's, then:

    # build miniperl:

    $ dtrace ...  -o dtrace_mini.o  a.o b.o c.o perlminimain.o
    $ cc -o miniperl dtrace_mini.o  a.o b.o c.o perlminimain.o

    # build libperl.a or .so:

    $ dtrace ...    -o dtrace_perllib.o  a.o b.o c.o
    $ ar rcu libperl.a dtrace_perllib.o  a.o b.o c.o

    # build perl:

    $ dtrace ...  -o dtrace_main.o  perlmain.o
    $ cc -o perl     dtrace_main.o  -lperl

This is has only recently arisen because we switched from PUSHSUB()
etc macros to S_cx_pushsub() etc inline functions, which contain
probes. Since the inline static functions, and hence the probes, are now
included in every source file, and since Solaris isn't smart enough to
remove inline static fns that aren't used in a particular compilation
unit, the probes end up getting used in every source file (at least where
PERL_CORE is true).

It also required fixing up XS-APItest's Makefile.PL, since one object
file is compiled using PERL_CORE.

3 years agoMakefile.SH: rationalise object list variables
David Mitchell [Mon, 7 Mar 2016 11:55:11 +0000 (11:55 +0000)] 
Makefile.SH: rationalise object list variables

There are a number of variables in the generated Makefile, such as
'obj' and 'minindt_obj', that enumerate various subsets of the object
files that need to be compiled and linked.

Rename and reorganise these vars slightly, to make the next commit
simpler. In particular it now splits the object files in into 3 sets:
common (av.o etc), those used just by miniperl (opmini.o etc)
and those used just by perl (op.o etc).

Should be no functional changes.

The changes to os2/Makefile.SHs have been done blind. Does anyone still
use OS2?

3 years agorename and function-ise dtrace macros
David Mitchell [Mon, 15 Feb 2016 13:48:24 +0000 (13:48 +0000)] 
rename and function-ise dtrace macros

This commit:

1. Renames the various dtrace probe macros into a consistent and
self-documenting pattern, e.g.

ENTRY_PROBE  => PERL_DTRACE_PROBE_ENTRY
RETURN_PROBE => PERL_DTRACE_PROBE_RETURN

Since they're supposed to be defined only under PERL_CORE, this shouldn't
break anything that's not being naughty.

2. Implement the main body of these macros using a real function.

They were formerly defined along the lines of

    if (PERL_SUB_ENTRY_ENABLED())
        PERL_SUB_ENTRY(...);

The PERL_SUB_ENTRY() part is a macro generated by the dtrace system, which
for example on linux expands to a large bunch of assembly directives.
Replace the direct macro with a function wrapper, e.g.

    if (PERL_SUB_ENTRY_ENABLED())
        Perl_dtrace_probe_call(aTHX_ cv, TRUE);

This reduces to once the number of times the macro is expanded.

The new functions also take simpler args and then process the values they
need using intermediate temporary vars to avoid huge macro expansions.

For example

    ENTRY_PROBE(CvNAMED(cv)
                    ? HEK_KEY(CvNAME_HEK(cv))
                    : GvENAME(CvGV(cv)),
                CopFILE((const COP *)CvSTART(cv)),
                CopLINE((const COP *)CvSTART(cv)),
                CopSTASHPV((const COP *)CvSTART(cv)));

is now

    PERL_DTRACE_PROBE_ENTRY(cv);

This reduces the executable size by 1K on -O2 -Dusedtrace builds,
and by 45K on -DDEBUGGING -Dusedtrace builds.

3 years agoConfigure: silence 'dtrace -G' probe
David Mitchell [Fri, 18 Mar 2016 23:29:44 +0000 (23:29 +0000)] 
Configure: silence 'dtrace -G' probe

When built with -Dusedtrace, Configure checks to see whether 'dtrace -G'
is supported, by running it. If it fails, it may spew error messages
to stderr, so use >/dev/null 2>&1.

3 years agoregex sets: fix Solaris optimiser bug (2nd attempt)
David Mitchell [Fri, 18 Mar 2016 22:36:40 +0000 (22:36 +0000)] 
regex sets: fix Solaris optimiser bug (2nd attempt)

[perl #127455]

My first fix for this, (v5.23.7-586-gd9cb841) didn't actually fix it.
The diagnosis was correct, but somewhere during fixing and testing I went
wrong, and the optimiser bug was still being triggered without me spotting
it.

The new approach, based on a suggestion by Tony C, is simply to
declare the errant local var 'fence' to be volatile. This stops the
compiler from optimising away its value.

3 years agoregcomp.c: Avoid a memory leak
Karl Williamson [Mon, 7 Mar 2016 22:37:11 +0000 (15:37 -0700)] 
regcomp.c: Avoid a memory leak

I spotted this in code reading.  The chances of it happening are quite
small.  It could happen under tainting with a user-defined \p{}
property, and /i matching.

3 years agoregcomp.c: Comment enhancements
Karl Williamson [Mon, 7 Mar 2016 17:49:17 +0000 (10:49 -0700)] 
regcomp.c: Comment enhancements

The text of one of these comments was furnished to me by Yves Orton some
time ago.

3 years agoMake instr() a macro
Karl Williamson [Fri, 11 Mar 2016 23:22:49 +0000 (16:22 -0700)] 
Make instr() a macro

... thus avoiding a function call overhead

3 years agoperlapi: Further clarification for term "Latin1"
Karl Williamson [Fri, 18 Mar 2016 02:57:48 +0000 (20:57 -0600)] 
perlapi: Further clarification for term "Latin1"

This is a follow on for 14d32fa99f736009ef63a8b17d164cd8f6e967d9

3 years agohandy.h: Add synonyms for macros
Karl Williamson [Fri, 11 Mar 2016 03:34:17 +0000 (20:34 -0700)] 
handy.h: Add synonyms for macros

I realized that we have inconsistent naming conventions for the
isFOO_uni() macros, like isALPHA_uni().  The "_uni" suffix elsewhere
refers to official Unicode code point numbers; whereas, for these macros
it refers to the native code point numbers for code points below 256.
And elsewhere, there are in some cases a _uni() and a uvchr() which mean
different things.

This commit adds '_uvchr' suffix equivalents for these macros, while
dropping mention in the documentation of the '_uni' forms.  Thus code
following the new paradigm will not be confusing, while existing code
will function unchanged.

3 years agoCarp: Fix .t failure to compile in v5.6
Karl Williamson [Thu, 17 Mar 2016 22:45:47 +0000 (16:45 -0600)] 
Carp: Fix .t failure to compile in v5.6

This doesn't guarantee the tests pass in that release; just that it will
actually get to testing.

3 years agoVarious fixups for version.pm 0.9914
Leon Timmermans [Fri, 18 Mar 2016 00:44:41 +0000 (01:44 +0100)] 
Various fixups for version.pm 0.9914

3 years agoRemove no longer valid tests
John Peacock [Thu, 18 Feb 2016 14:23:08 +0000 (08:23 -0600)] 
Remove no longer valid tests

3 years agoImport version.pm 0.9914 from CPAN
John Peacock [Wed, 17 Feb 2016 03:34:52 +0000 (21:34 -0600)] 
Import version.pm 0.9914 from CPAN

3 years agoperldelta: document exception thrown when -i output close fails
Ricardo Signes [Thu, 17 Mar 2016 23:49:28 +0000 (19:49 -0400)] 
perldelta: document exception thrown when -i output close fails

3 years agocroak on failure to close an in-place edit output file
Tony Cook [Thu, 5 Nov 2015 04:06:00 +0000 (15:06 +1100)] 
croak on failure to close an in-place edit output file

3 years agoreport an error and fail if we can't flush stdout
Tony Cook [Wed, 4 Nov 2015 05:52:37 +0000 (16:52 +1100)] 
report an error and fail if we can't flush stdout

3 years agoperldelta for 22e30cb2a3, 6156385671 and a28fff5120
Steve Hay [Thu, 17 Mar 2016 23:28:29 +0000 (23:28 +0000)] 
perldelta for 22e30cb2a36156385671 and a28fff5120

3 years agolocale.c: Add comment
Karl Williamson [Thu, 17 Mar 2016 22:11:04 +0000 (16:11 -0600)] 
locale.c: Add comment

3 years agoregen/mk_invlists.pl: Revamp so works on earlier Unicodes
Karl Williamson [Tue, 15 Mar 2016 02:49:00 +0000 (20:49 -0600)] 
regen/mk_invlists.pl: Revamp so works on earlier Unicodes

The code that generates the tables for the \b{foo} handling (in
regexec.c) did not correctly work when compiled on an earlier Unicode.
This fixes things up to do that, consolidating some common code into a
common function and making the generated hdr file look nice, with the
tables taking fewer columns of screen space

3 years agomktables: Use correct structure to look up data
Karl Williamson [Thu, 17 Mar 2016 21:25:09 +0000 (15:25 -0600)] 
mktables: Use correct structure to look up data

There are two types of tables in mktables: Map tables map code points
to the values a property have for those code points; and match tables
which are booleans, give "does a code point match a given property
value?".  There are different data structures to encapsulate each.  This
code was using the wrong structure to look something up.  Usually this
failed, and a fall-back value was used instead.  When compiling an early
Unicode release, I discovered that there could be a conflict.

3 years agomktables: Fix bug with early Unicode versions
Karl Williamson [Thu, 17 Mar 2016 20:27:34 +0000 (14:27 -0600)] 
mktables: Fix bug with early Unicode versions

An array had 2 optional elements at the end.  I got confused about
handling them.  This change first deals with the final one, pops it and
saves it separately if found.  Then only one optional element needs to
be dealt with in the course of the code.

This only gets executed for very early Unicode versions

3 years agomktables: Unicode 1.5 only had 2**16 code points
Karl Williamson [Tue, 15 Mar 2016 22:19:30 +0000 (16:19 -0600)] 
mktables: Unicode 1.5 only had 2**16 code points

Therefore, we shouldn't add any above that.

3 years agoThe public_html directory on dromedary is working again.
Abigail [Thu, 17 Mar 2016 11:30:32 +0000 (12:30 +0100)] 
The public_html directory on dromedary is working again.

So, I removed the references that it wasn't working in December 2015.

3 years agoAdd contributing info to Carp.
Shlomi Fish [Thu, 10 Mar 2016 19:32:41 +0000 (21:32 +0200)] 
Add contributing info to Carp.

It lacked a link to the VCS repository so I had to ask rjbs about it on
the #p5p IRC channel. And one should always fix a problem twice:

* http://www.joelonsoftware.com/articles/customerservice.html .

3 years agoharmonize S_dump_exec_pos()'s last arg type
David Mitchell [Mon, 14 Mar 2016 19:11:08 +0000 (19:11 +0000)] 
harmonize S_dump_exec_pos()'s last arg type

embed.fnc declared it as "U32 depth", while it was defined as "const U32
depth".

3 years agoavoid generating an empty statement outside a function on non-clang
Tony Cook [Tue, 15 Mar 2016 00:26:26 +0000 (11:26 +1100)] 
avoid generating an empty statement outside a function on non-clang

On non-clang compilers, the code:

CLANG_DIAG_RESTORE;

outside a function generated a simple empty statement (or empty
declaration), which is invalid syntax outside a function in C,
causing a warning from the solaris studio compiler.

3 years agofix "bad match" issue reported in perl #127705
Yves Orton [Mon, 14 Mar 2016 22:30:02 +0000 (23:30 +0100)] 
fix "bad match" issue reported in perl #127705

In 24be310237a0f8f19cfdb71de1b068b4ce9572a0 I reworked how
we stored the close_paren info in the regexp match state
structure. Unfortunately I missed a subtle aspect of the
logic which meant that in certain cases we were relying
on close_paren being true to avoid comparing it against
a false ARG value for things like CURLYX, which meant that
sometimes we would exit an stack frame prematurely. This
patch fixes that logic and makes it more clear (via macros)
what is going on.

3 years agofix perl #127705, incorrect restoration of state during EVAL/GOSUB
Yves Orton [Mon, 14 Mar 2016 19:03:22 +0000 (20:03 +0100)] 
fix perl #127705, incorrect restoration of state during EVAL/GOSUB

We were not restoring the cur_curlyx property properly during our funky
stack traversal when we really need to. This was a longstanding bug in this
code which my patches appear to have "tickled", although I am not sure why.

3 years agoadd consistency with other union members
Yves Orton [Mon, 14 Mar 2016 18:58:33 +0000 (19:58 +0100)] 
add consistency with other union members

In most cases the curlyx member is the first thing after the yes state
member, but eval was reversed. While debugging perl #127705 I switched
them to see what would happen, which changed the bug, and ultimately
revealed the cause of the problem. So I am going to leave them in the
"consistent" order.

3 years agofixup broken diagnostic
Yves Orton [Mon, 14 Mar 2016 17:10:19 +0000 (18:10 +0100)] 
fixup broken diagnostic

we shouldnt show this in normal re debug mode, just when debugging the stack,
also the macro wasnt using the STR argument like it should

3 years agoamigaos4: avoid PerlIO_findFILE() in popen/plcose
Andy Broad [Mon, 14 Mar 2016 21:43:30 +0000 (17:43 -0400)] 
amigaos4: avoid PerlIO_findFILE() in popen/plcose

Merges amigaos_popen / amigaos_pclose with the amigaos specific
version of the Perl_my_popen / Perl_my_pclose functions and uses PerlIO
directly for the perl facing end of the PIPE:s thus avoid the issues
of PerlIO_findFILE() completely.

Also fixes a couple of warnings.

3 years agoConfigure: silence some try.c warnings
David Mitchell [Mon, 14 Mar 2016 16:49:05 +0000 (16:49 +0000)] 
Configure: silence some try.c warnings

On one of the try.c compilations, redirect stderr to /dev/null, since
the code can legitimately warn without there being a problem.

The try.c in question is probing for what symbols the compiler supports.
The clang extensions __has_include and __has_include_next are designed
only to be used in .h files, so they warn if used from try.c

3 years agos/ar rcu/ar rc/ during linking
David Mitchell [Mon, 14 Mar 2016 15:41:46 +0000 (15:41 +0000)] 
s/ar rcu/ar rc/ during linking

The are a few places in Makefile.SH which do (approximately):

    rm $libfile
    ar rcu $libfile *.o

The 'u' in 'rcu' seems redundant since the old lib file is always deleted
just before being recreated; and more to the point, it generates warnings
on recent Linux builds:

    /bin/ar: `u' modifier ignored since `D' is the default (see `U')

This is because the 'u' modifier updates the archive, i.e. only replaces
the objects which are newer in an existing archive. On my Linux system,
ar by default operates in 'deterministic' mode, which means that it
doesn't add timestamps etc (so that repeated builds will give identical
binaries). In this mode 'u' can't work, hence the warning.

So this commit just removes the 'u' flag.

3 years agoHOPBACKc(pos, off): silence compiler warning
David Mitchell [Mon, 14 Mar 2016 15:24:35 +0000 (15:24 +0000)] 
HOPBACKc(pos, off): silence compiler warning

This macro does (amongst other things)
    reghopmaybe3(..., -off, ...)

When passed an off that is unsigned (e.g. prog->gofs, which is STRLEN),
The MSWin32 smoker complains:

    ..\regexec.c(2939) : warning C4146: unary minus operator applied to
    unsigned type, result still unsigned

Chane the arg to (SSize_t)0-off and hope that integer conversion rules
make the expression legal.

3 years agoonly call Perl_re_printf() under -DDEBUGGING
Tony Cook [Mon, 14 Mar 2016 01:56:32 +0000 (12:56 +1100)] 
only call Perl_re_printf() under -DDEBUGGING

This caused link failures on Win32 with MSVC.

3 years agoTime-HiRes: recent changes
Jarkko Hietaniemi [Mon, 14 Mar 2016 00:20:23 +0000 (20:20 -0400)] 
Time-HiRes: recent changes

3 years agoTime-HiRes: version bump
Jarkko Hietaniemi [Mon, 14 Mar 2016 00:05:15 +0000 (20:05 -0400)] 
Time-HiRes: version bump

3 years agoTime-HiRes: the mutex needs init
Jarkko Hietaniemi [Mon, 14 Mar 2016 00:01:17 +0000 (20:01 -0400)] 
Time-HiRes: the mutex needs init

3 years agoTime-HiRes: also hrt_ualarm_itimer() is unused
Jarkko Hietaniemi [Sun, 13 Mar 2016 23:55:56 +0000 (19:55 -0400)] 
Time-HiRes: also hrt_ualarm_itimer() is unused

3 years agoTime-HiRes: MUTEX_LOCK, not PERL_MUTEX_LOCK
Jarkko Hietaniemi [Sun, 13 Mar 2016 23:53:17 +0000 (19:53 -0400)] 
Time-HiRes: MUTEX_LOCK, not PERL_MUTEX_LOCK

3 years agoautouse: update tests to match new v1.11 CPAN release
Ricardo Signes [Sun, 13 Mar 2016 22:47:18 +0000 (18:47 -0400)] 
autouse: update tests to match new v1.11 CPAN release

3 years agoTime-HiRes: aim for CPAN release of 1.9731 today
Jarkko Hietaniemi [Sun, 13 Mar 2016 21:26:06 +0000 (17:26 -0400)] 
Time-HiRes: aim for CPAN release of 1.9731 today

3 years agodont call re_printf() with %*s if you arent going to pass it in
Yves Orton [Sun, 13 Mar 2016 17:43:31 +0000 (18:43 +0100)] 
dont call re_printf() with %*s if you arent going to pass it in

3 years agoonce more for the win
Yves Orton [Sun, 13 Mar 2016 17:12:29 +0000 (18:12 +0100)] 
once more for the win

Apparently I have to pass in the context explicitly for this type of
sub, since we cant do a proper macro wrap for subs using ... (va_args/va_list)
so we have to do everything explicitly. Hopefully this fixes the smoke reports

3 years agolib/warnings.t: Skip an ASCII-centric test on EBCDIC
Karl Williamson [Sun, 13 Mar 2016 16:08:36 +0000 (10:08 -0600)] 
lib/warnings.t: Skip an ASCII-centric test on EBCDIC

This test is for a specific UTF-8 malformation, but UTF-EBCDIC is
different, so just skip it there.

3 years agofixup definitions and usage of new re debugging subs
Yves Orton [Sun, 13 Mar 2016 15:59:32 +0000 (16:59 +0100)] 
fixup definitions and usage of new re debugging subs

this should fix the smoke failures on threaded builds,
also it renames re_indentfo which was a terrible name in the first
place, and now what i have had to strip the Perl_prefixes from
these subs with a perl -i -pe, I took the opportunity to rename
it to re_exec_indent, which self documents much better.

3 years ago[perl #126182] rework pattern GOSUB infinite recursion detection
Yves Orton [Sun, 13 Mar 2016 10:15:30 +0000 (11:15 +0100)] 
[perl #126182] rework pattern GOSUB infinite recursion detection

In ba6840fbf2fdde3e7f1bda1a26f46c901f36d5ec I tried to fix
[perl #126182] which is a bug about us failing to detect regex
left recursion in some cases.

There were two problems with that patch, both pointed out by
Zefram. The first is that I made left recursion a match fail,
instead of throwing an exception, this makes left-recursion match
sometimes, but at least sometimes in what is arguably the wrong
way. Zefram was able to convince me that dying is better than
matching incorrectly.

The second patch was that it ignored some subtleties in how
the backtracking stack works, which affected how the patch
restored the recurse_locinput[] data which is used to track
what position a GOSUB was entered from. This meant that in
various cases it would not be restored correctly, and we would
still infinite recurse. I believe that it works correctly now.

Thanks for Zefram for the feedback on the original patch.

3 years agoRework diagnostics in the regex engine
Yves Orton [Fri, 11 Mar 2016 09:36:25 +0000 (10:36 +0100)] 
Rework diagnostics in the regex engine

This introduces three new subs:

Perl_re_printf() which is a wrapper for

    PerlIO_printf( Perl_debug_log, ... ),

which cuts down on clutter in the code. Arguably this could be moved
to util.c and renamed something like PerlIO_debugf() and then we could
declutter all the statements that write to the Perl_debug_log
filehandle. But that is a bit too ambituous for me right now, so
I leave this as a regex engine only sub for now.

Perl_re_indentf() which is a wrapper for PerlIO_re_printf(),
which adds an indent argument and automatically indents the
line appropriately, and is used in regcomp.c for trace diagnostics
during compilation.

Perl_re_indentfo() which is similar to Perl_re_indentf() but
is used in regexec.c which adds a specific prefix to each indented
line to account for the fact that during execution we normally have
string position information on the left.

The end result of this patch is that a lot of clutter in the debugging
statements in the regex engine is reduced, exposing what is actually
going on. It should also now be easier to add new diagnostics which
"do the right thing".

Over time the debugging trace output in regexec has become
very cluttered and confusing. This patch cleans much of it up,
if something happens at a given recursion depth it is output
at the right depth, etc, and formats have been changed to not have
leading spaces so you can actually see the indentation properly.

3 years agoTime-HiRes: record recent changes
Jarkko Hietaniemi [Sun, 13 Mar 2016 00:57:24 +0000 (19:57 -0500)] 
Time-HiRes: record recent changes

3 years agoTime-HiRes: not_used is unused
Jarkko Hietaniemi [Sun, 13 Mar 2016 00:17:20 +0000 (19:17 -0500)] 
Time-HiRes: not_used is unused

3 years agoTime-HiRes: no mutex for you due to mistyped USE
Jarkko Hietaniemi [Sun, 13 Mar 2016 00:15:24 +0000 (19:15 -0500)] 
Time-HiRes: no mutex for you due to mistyped USE

3 years agoTime-HiRes: remove unused hrt_ualarm
Jarkko Hietaniemi [Sun, 13 Mar 2016 00:07:54 +0000 (19:07 -0500)] 
Time-HiRes: remove unused hrt_ualarm

Binary compat with 1.91 would mean roughly 2006-10, which means
either Perl 5.8.8 or 5.9.5.  Time to let go.  No hits in CPAN,
as observed by Tony Cook: http://grep.cpan.me/?q=hrt_ualarm

3 years agoop.c: cosmetic comment changes
Lukas Mai [Sat, 12 Mar 2016 22:54:31 +0000 (23:54 +0100)] 
op.c: cosmetic comment changes

3 years agot/re/uniprops.t: Remove wrong test cases
Karl Williamson [Sat, 12 Mar 2016 22:00:45 +0000 (15:00 -0700)] 
t/re/uniprops.t: Remove wrong test cases

mktables generates the file used in this test.  Unicode version 9
introduces a numeric value that is an order of magnitude closer to 0
than any previous version had.  This demonstrated a bug in mktables,
where it didn't consider the possibility of floating point numbers being
indistinguishably close to integers.  It did check for being too close
to the rational numbers used in Unicode, but omitted checking for
integers.  This adds that check, which in turn causes some wrong test
cases to not be generated for this .t.

This bug has not shown up in earlier Unicode versions, but is there
nonetheless, so I'm pushing this now instead of waiting.

3 years agoAPItest.xs silence compiler warning
David Mitchell [Sat, 12 Mar 2016 11:09:06 +0000 (11:09 +0000)] 
APItest.xs silence compiler warning

On Solaris:
    "APItest.xs", line 1519: warning: integer overflow detected: op "<<"

3 years agoregex sets: fix Solaris optimiser bug
David Mitchell [Sat, 12 Mar 2016 09:39:41 +0000 (09:39 +0000)] 
regex sets: fix Solaris optimiser bug

[perl #127455]

On Solaris with -DDEBUGGING, re/regex_sets.t was failing to compile.

This appears to be due to an optimiser bug.

The code in question looked like:

  handle_operand:
    top_index = av_tindex_nomg(stack);
    if (top_index - fence >= 0) {
        ...
    }

printf()ing the value of fence after the av_tindex_nomg() showed that its
value was corrupted (compared with its expected value based on a different
platform with the same debugging print). However, putting a another printf
prior to the av_tindex_nomg() call not only displayed the correct value,
but caused the later printf() to also display the correct value. It seems
that merely accessing fence prior to av_tindex_nomg() avoids the
corruption.

Simplifying the av_tindex_nomg(), the bad behaviour could be reduced to:

    if (!stack) { __assert( "" , "", 1); }

Putting a printf after this gave a corrupted fence; a printf before
made everything work.

So this workaround commit just makes sure that fence is accessed prior to
calling av_tindex_nomg().

3 years agoamigaos4: better popen() + pclose() implementation
Andy Broad [Sat, 12 Mar 2016 01:20:31 +0000 (20:20 -0500)] 
amigaos4: better popen() + pclose() implementation

popen(): handle better the case where the popened external
might exit before the child process manages to start.

pclose(): protect with a semaphore.

3 years agoFix various pod errors.
Karl Williamson [Fri, 11 Mar 2016 21:43:33 +0000 (14:43 -0700)] 
Fix various pod errors.

Mostly these are too long verbatim lines.

3 years agoinline.h: Fix comment
Karl Williamson [Fri, 11 Mar 2016 16:56:48 +0000 (09:56 -0700)] 
inline.h: Fix comment

3 years agoperlvar: Add a couple links
Karl Williamson [Fri, 11 Mar 2016 16:55:55 +0000 (09:55 -0700)] 
perlvar: Add a couple links

3 years agoperllocale: Nits, update for 5.24 changes
Karl Williamson [Fri, 11 Mar 2016 16:54:51 +0000 (09:54 -0700)] 
perllocale: Nits, update for 5.24 changes

3 years agoperlapi: Clarify Latin1 and ISO-8859-1
Karl Williamson [Fri, 11 Mar 2016 03:14:24 +0000 (20:14 -0700)] 
perlapi: Clarify Latin1 and ISO-8859-1

3 years agoExport Winsock error constants from POSIX.pm
Steve Hay [Tue, 16 Feb 2016 13:40:08 +0000 (13:40 +0000)] 
Export Winsock error constants from POSIX.pm

3 years agoExport Winsock error constants from Errno.pm
Steve Hay [Tue, 16 Feb 2016 08:50:08 +0000 (08:50 +0000)] 
Export Winsock error constants from Errno.pm

3 years agoSet $^E for socket errors on Windows
Steve Hay [Thu, 4 Feb 2016 08:51:21 +0000 (08:51 +0000)] 
Set $^E for socket errors on Windows

3 years agoBetter wording.
Jarkko Hietaniemi [Fri, 11 Mar 2016 11:57:56 +0000 (06:57 -0500)] 
Better wording.

3 years agogitignore for cscope index files.
Jarkko Hietaniemi [Fri, 11 Mar 2016 01:07:35 +0000 (20:07 -0500)] 
gitignore for cscope index files.

3 years agomake target for cscope indexing.
Jarkko Hietaniemi [Fri, 11 Mar 2016 01:05:32 +0000 (20:05 -0500)] 
make target for cscope indexing.

3 years agoCPAN release of autouse is now 1.10
Ricardo Signes [Fri, 11 Mar 2016 00:10:16 +0000 (19:10 -0500)] 
CPAN release of autouse is now 1.10

this has no changes to code from 1.08; it gets in the test changes
from blead and adds a prereq for Scalar::Util with isdual

3 years agoperlapi: Document ninstr() and rninstr()
Karl Williamson [Thu, 10 Mar 2016 22:52:34 +0000 (15:52 -0700)] 
perlapi: Document ninstr() and rninstr()

3 years agot/re/reg_mesg.t: Add some more tests
Karl Williamson [Thu, 10 Mar 2016 19:10:38 +0000 (12:10 -0700)] 
t/re/reg_mesg.t: Add some more tests

3 years agouse inequality when checking "tail" in trie compilation
Yves Orton [Wed, 9 Mar 2016 19:03:21 +0000 (20:03 +0100)] 
use inequality when checking "tail" in trie compilation

It is possible that a node inside an alternation points out
and past the end of the alternation, using equality on tail
means we don't notice. Its not clear that this is actually
required, but it has been an issue in other contexts so lets
just always use inequalities for this purpose.

3 years agoFix Perl #126206: handle NOTHING regops and EXACTFU_SS regops in make_trie() properly
Yves Orton [Wed, 9 Mar 2016 19:00:53 +0000 (20:00 +0100)] 
Fix Perl #126206: handle NOTHING regops and EXACTFU_SS regops in make_trie() properly

... and avoid dereffing non-EXACT nodes unnecessarily at the same
time. This fixes https://rt.perl.org/Ticket/Display.html?id=126206

3 years agoutf8.c: Add missing {}
Karl Williamson [Wed, 9 Mar 2016 01:39:46 +0000 (18:39 -0700)] 
utf8.c: Add missing {}

which caused an if's scope to be less than intended.

Spotted by Jarkko Hietaniemi using gcc 6 with the
'-Wmisleading-indentation' option

3 years agoperldelta for c2538af7458
Tony Cook [Wed, 9 Mar 2016 01:02:50 +0000 (12:02 +1100)] 
perldelta for c2538af7458

3 years ago[perl #122287] probe in Configure whether dtrace builds an object
Tony Cook [Wed, 9 Mar 2016 00:54:13 +0000 (11:54 +1100)] 
[perl #122287] probe in Configure whether dtrace builds an object

When building the object file, newer versions of dtrace (on Illumos
based systems at least) require an input object file that uses
at least one of the probes defined in the .d file.

The test in Makefile.SH didn't provide that definition so the test
would fail, and not build an object file, and fail to link later on,
on systems that *do* need the object file.

Moved the probe to Configure (where it probably belongs) and supplied
an object file that uses a probe.

Tested successfully on OmniOS (with the new dtrace), Solaris 11,
and darwin.

3 years agofix a skip count in cpan/IPC-SysV/t/ipcsysv.t
Tony Cook [Tue, 8 Mar 2016 23:43:57 +0000 (10:43 +1100)] 
fix a skip count in cpan/IPC-SysV/t/ipcsysv.t

reported upstream as https://rt.cpan.org/Ticket/Display.html?id=112827

3 years ago[perl #127533] only test semctl() if we have everything needed to use it
Tony Cook [Tue, 1 Mar 2016 00:35:21 +0000 (11:35 +1100)] 
[perl #127533] only test semctl() if we have everything needed to use it

In a FreeBSD jail, the semctl() entry point might exist, but can be
disabled by policy, when it is disabled, the Configure code that
tests for the different structures that can be supplied to semctl()
fail.

The code that implements semctl() for perl treats semctl() as
unimplemented if neither structure is available, so avoid testing
semctl() if the structures couldn't be detected.

3 years agoperlvar: Cross reference other pods
Karl Williamson [Tue, 8 Mar 2016 04:37:00 +0000 (21:37 -0700)] 
perlvar: Cross reference other pods

Suggested by "kes" in
http://nntp.perl.org/group/perl.perl5.porters/234447

3 years agoConvert to use av_tindex_nomg()
Karl Williamson [Mon, 7 Mar 2016 21:49:54 +0000 (14:49 -0700)] 
Convert to use av_tindex_nomg()

I looked at the code I'm familiar with, and converted the av_tindex
calls on arrays I was confident don't have magic to av_tindex_nomg().
This saves a little work each time.

3 years agoAdd av_tindex_nomg()
Karl Williamson [Mon, 7 Mar 2016 21:44:50 +0000 (14:44 -0700)] 
Add av_tindex_nomg()

This is like av_tindex, but doesn't handle magic.  I'm not documenting
it for now, in case it turns out this was not a good idea.

Inspired from an observation by Tony Cook.

3 years agoregcomp.c: Silence some compiler warnings
Karl Williamson [Tue, 8 Mar 2016 00:02:04 +0000 (17:02 -0700)] 
regcomp.c: Silence some compiler warnings

Some compilers wrongly think these variables can be used uninitialized.

3 years agoperlguts: Slight clarification
Karl Williamson [Mon, 7 Mar 2016 22:50:41 +0000 (15:50 -0700)] 
perlguts: Slight clarification