This is a live mirror of the Perl 5 development currently hosted at
7 years agoDon't add unnecessary linespace.
James E Keenan [Fri, 4 Jul 2014 00:55:49 +0000 (20:55 -0400)]
Don't add unnecessary linespace.

And don't add trailing whitespace ever!  (Committer's error.)

7 years agoSupply description for only test lacking one.
James E Keenan [Fri, 4 Jul 2014 00:38:09 +0000 (20:38 -0400)]
Supply description for only test lacking one.

Wording of description taken from 'perlvar.pod' section on $EVAL_ERROR ($@).

7 years agoAvoid unused warnings from locale-less systems.
Jarkko Hietaniemi [Thu, 3 Jul 2014 15:33:22 +0000 (11:33 -0400)]
Avoid unused warnings from locale-less systems.

From Brian Fraser: "Technically, any Perl compiled with
-Accflags="-UUSE_LOCALE", or -Ui_locale -Ud_setlocale...
realistically, for Android".

7 years agoDeprecate /\C/
David Mitchell [Thu, 3 Jul 2014 15:08:56 +0000 (16:08 +0100)]
Deprecate /\C/

This character class matches a single byte within a multi-byte utf8 char,
and so breaks encapsulation. It is buggy, and can corrupt utf8 strings.

This deprecation was agreed in the thread starting

7 years agoUpdate Pod-Usage to CPAN version 1.64
Chris 'BinGOs' Williams [Thu, 3 Jul 2014 08:56:20 +0000 (09:56 +0100)]
Update Pod-Usage to CPAN version 1.64


1.64 (marekr)
- CPAN#93170: Abstract in pod doesn't follow convention
    fixed the NAME section
- CPAN#86932: documentation does not work with use strict
    rewritten parts of the DESCRIPTION section
- CPAN#86776: "pod2usage -verbose 2 FILE" is broken when perldoc is missing
    verified that pod2usage acts decently if perldoc is missing
    note: a Perl installation without perldoc is like a haunted house
    without ghosts

7 years agoSync Storable version for CPAN release
Chris 'BinGOs' Williams [Thu, 3 Jul 2014 08:54:20 +0000 (09:54 +0100)]
Sync Storable version for CPAN release

7 years agoperldelta for 3f7602fa4cd6
Tony Cook [Thu, 3 Jul 2014 06:33:00 +0000 (16:33 +1000)]
perldelta for 3f7602fa4cd6

7 years agoextra tests for grok_number(_flags)()
Tony Cook [Thu, 1 May 2014 05:37:08 +0000 (15:37 +1000)]
extra tests for grok_number(_flags)()

7 years ago[perl #3330] warn on increment of an non number/non-magically incable value
Tony Cook [Thu, 3 Jul 2014 05:09:32 +0000 (15:09 +1000)]
[perl #3330] warn on increment of an non number/non-magically incable value

The original version of this patch, introduced in
v5.19.2-389-g8140a7a, warned confusingly when incrementing strings
like "123abc".  This version includes the fix patch from [perl

7 years agouniversal.c gv_stashpv -> gv_stashpvs
Daniel Dragan [Wed, 2 Jul 2014 20:19:37 +0000 (16:19 -0400)]
universal.c gv_stashpv -> gv_stashpvs

avoids a strlen()

7 years agos///e on tainted utf8 strings got pos() messed up
David Mitchell [Wed, 2 Jul 2014 16:13:45 +0000 (17:13 +0100)]
s///e on tainted utf8 strings got pos() messed up

RT #122148: In 5.20, commit 25fdce4a165 changed the way pos() was stored
in magic attached to SVs from being a byte offset to a char offset,
*except* that, for efficiency, strings being used for pattern matching
were kept as byte offsets (with a flag indicating thus), *except* where
the SV already had magic attached (such as taint, as in the bug report and
in this commit's test), in which case it kept it as chars.

The code that updated pos() after an iteration of s///e was faulty: the
string buffer it used for converting byte legnths to char lengths (via
utf8_length()) was the wrong buffer: rather than using the src string
being matched against, it was using the destination string being built up
via iterations of s///. Once double-byte utf8 chars were involved, all the
pos() calculations went wrong, and utf8 warnings started mysteriously

7 years agoEnable _FORTIFY_SOURCE only if optimizing, as specified.
Jarkko Hietaniemi [Wed, 2 Jul 2014 11:55:33 +0000 (07:55 -0400)]
Enable _FORTIFY_SOURCE only if optimizing, as specified.

As noticed by Dave Mitchell.

7 years agoUpdate ChangeLog before release
Abhijit Menon-Sen [Wed, 2 Jul 2014 11:00:40 +0000 (16:30 +0530)]
Update ChangeLog before release

7 years agopassthrough make options to make -s can speed up the build
Tony Cook [Wed, 2 Jul 2014 05:53:23 +0000 (15:53 +1000)]
passthrough make options to make -s can speed up the build

also quieten by default

7 years agopass through -s if supplied, instead of defaulting to it
Tony Cook [Wed, 2 Jul 2014 05:50:02 +0000 (15:50 +1000)]
pass through -s if supplied, instead of defaulting to it

7 years agoallow the build to specify verbose
Tony Cook [Wed, 2 Jul 2014 05:49:40 +0000 (15:49 +1000)]
allow the build to specify verbose

7 years agoSupport faster make -s
Reini Urban [Fri, 25 Nov 2011 04:04:20 +0000 (22:04 -0600)]
Support faster make -s

Skip chdir messages by passing make -s through
Also silence to speed up the build process. use -v or --verbose

7 years agoModule::CoreList::TieHashDelta stop massive recursion
Daniel Dragan [Wed, 28 May 2014 12:17:25 +0000 (08:17 -0400)]
Module::CoreList::TieHashDelta stop massive recursion

On machines with very limited memory, EXISTS calling EXISTS caused a very
high number of C stack frames, magic stack swaps, and Perl call stack
frames leading to memory exhaustion. Turn the recursion into a loop. See
[perl #121974].

7 years agoallow tests to use C<< plan skip_all => $reason >>
Tony Cook [Wed, 2 Jul 2014 00:30:07 +0000 (10:30 +1000)]
allow tests to use C<< plan skip_all => $reason >>

This pointless incompatibility with Test::More has resulted in
many annoyingly broken test scripts.

7 years agoavoid an undefined value warning from libperl.t
Tony Cook [Wed, 2 Jul 2014 00:05:33 +0000 (10:05 +1000)]
avoid an undefined value warning from libperl.t

and fix a doubly broken C<< plan skil_all => "no GNU nm"; >>

7 years agoperldelta for 5afb921fc188
Tony Cook [Tue, 1 Jul 2014 23:30:49 +0000 (09:30 +1000)]
perldelta for 5afb921fc188

7 years agoMake the "no name list" catcher more relaxed.
Jarkko Hietaniemi [Tue, 1 Jul 2014 12:58:30 +0000 (08:58 -0400)]
Make the "no name list" catcher more relaxed.

7 years agoIf building with DTrace, the dtrace header is required.
Peter Martini [Thu, 19 Dec 2013 10:21:19 +0000 (05:21 -0500)]
If building with DTrace, the dtrace header is required.

It's generated and its a dependency, so make depend seems like a natural
fit for it.

7 years agoSome more cppsymbols to scan.
Jarkko Hietaniemi [Sun, 29 Jun 2014 01:38:08 +0000 (21:38 -0400)]
Some more cppsymbols to scan.

Weirdly, BYTEORDER/BYTE_ORDER was missing.  Note that  __BYTE_ORDER__
might be '1234' even though longs are 64 bit, since probably the symbol
is used to tell the order in a "word", which still is "int", not "long".
In other words, Perl's BYTEORDER will likely be longer the the cppsymbol.

Gleaned some extra CPUs and OSes from, though didn't go
all crazy on platforms with many variants like ARM, HPPA, SH.

7 years agoANSI prototypes for Berkeley DB API testing.
Jarkko Hietaniemi [Mon, 30 Jun 2014 01:10:02 +0000 (21:10 -0400)]
ANSI prototypes for Berkeley DB API testing.

So that g++ doesn't go apoplectic.  K&R prototypes in 2014?  Impressive.

7 years agoIn nonblock test do not ignore the syscall returns.
Jarkko Hietaniemi [Mon, 30 Jun 2014 01:02:47 +0000 (21:02 -0400)]
In nonblock test do not ignore the syscall returns.

Makes for less whining with warn_unused_result.

7 years ago-D_FORTIFY_SOURCE is no more a todo.
Jarkko Hietaniemi [Sat, 28 Jun 2014 22:24:23 +0000 (18:24 -0400)]
-D_FORTIFY_SOURCE is no more a todo.

7 years agoMention -D_FORTIFY_SOURCE in perldelta.
Jarkko Hietaniemi [Mon, 30 Jun 2014 13:03:49 +0000 (09:03 -0400)]
Mention -D_FORTIFY_SOURCE in perldelta.

7 years agoAdd -D_FORTIFY_SOURCE=2 when applicable.
Jarkko Hietaniemi [Sat, 28 Jun 2014 22:30:13 +0000 (18:30 -0400)]
Add -D_FORTIFY_SOURCE=2 when applicable.

Added only if available (gcc 4.*) and not already there.

Not already there might be explicit via ccflags, or less explicitly
via cppsymbols: for example in OS X 10.5+ the -D_FORTIFY_SOURCE=2
is already the default with the system cc (gcc earlier, now clang).

Some Linux distros (e.g. Fedora) also have enabled it for years,
either for all user code, or for their own builds.

7 years agoAssert before deref due to possible NULL.
Jarkko Hietaniemi [Sun, 29 Jun 2014 12:47:17 +0000 (08:47 -0400)]
Assert before deref due to possible NULL.

Coverity perl5 CID 68587.

7 years agoCoverity 45351 - Refactor to skip a loop iteration
Peter Martini [Mon, 2 Jun 2014 10:32:12 +0000 (06:32 -0400)]
Coverity 45351 - Refactor to skip a loop iteration

The loop in this case will remove an element from a list,
so it keeps a pointer to the previous element.  Before this
change, the pointer to the previous element was only initialized
at the end of the first pass, so Coverity flagged the code
as potentially using a NULL pointer.  It's a false positive
because the first element in the list is always a pushmark, so
the branch that actually used that pointer would never be hit
while it's still NULL, but changing the code to explicitly set
lasto before entering the loop (and asserting that its a PUSHMARK)
makes the logic clearer to both humans and Coverity.

7 years agoCosmetic consistency on "unimplemented function" deaths.
Jarkko Hietaniemi [Mon, 30 Jun 2014 02:03:45 +0000 (22:03 -0400)]
Cosmetic consistency on "unimplemented function" deaths.

7 years agolibperl.t: Allow tetss to pass if perl was compiled with -fno-common
Brian Fraser [Sun, 29 Jun 2014 13:16:45 +0000 (15:16 +0200)]
libperl.t: Allow tetss to pass if perl was compiled with -fno-common

If Perl was compiled with -fno-common, there will be no common
section in nm's output; the symbols we're looking for will be
under bss ('unknown').

7 years agoregcomp.c: Rewrite S_put_byte() which dumps regex []
Karl Williamson [Sat, 28 Jun 2014 21:12:45 +0000 (15:12 -0600)]
regcomp.c: Rewrite S_put_byte() which dumps regex []

This routine is used to display what the latin1 portions of a bracketed
character class matches.  It is what used, for example,  under -Dr to
display [] contents.  I noticed that the display had more hex characters
than needed, which are less intelligible than just printing out ASCII
printables.  And hence was harder to discern what is really going on.
This improves that, see the comments for more details.

7 years agotoke.c: White-space alignment, add comment
Karl Williamson [Sat, 28 Jun 2014 15:56:53 +0000 (09:56 -0600)]
toke.c: White-space alignment, add comment

7 years agopod/.gitignore: delete entries for now-unsupported arch-specific pods
Aaron Crane [Sat, 28 Jun 2014 17:54:02 +0000 (18:54 +0100)]
pod/.gitignore: delete entries for now-unsupported arch-specific pods

The build process creates a symlink pod/perl$platform.pod pointing to
README.$platform for each relevant platform, and we therefore want Git to
ignore those generated symlinks. Where we no longer support a given
platform, there's no need to have a gitignore entry for it.

7 years agoCapture the stderr of "nm libperl.a".
Jarkko Hietaniemi [Sat, 28 Jun 2014 18:05:41 +0000 (14:05 -0400)]
Capture the stderr of "nm libperl.a".

Ignore one known darwin seemingly harmless whine,
but die if otherwise unexpected stderr output.

Cannot do simply 2>&1 because buffering conflicts.

7 years agoskip works only with SKIP.
Jarkko Hietaniemi [Sat, 28 Jun 2014 02:10:16 +0000 (22:10 -0400)]
skip works only with SKIP.

Cannot do POSIX::blah() unless POSIX loaded ok.

(Makes freshly built "minitest" to pass again.)

7 years agoIncrement to $VERSION in ext/Hash-Util/lib/Hash/
James E Keenan [Sat, 28 Jun 2014 01:59:46 +0000 (03:59 +0200)]
Increment to $VERSION in ext/Hash-Util/lib/Hash/

Change to ext/Hash-Util/Util.xs requires this.

7 years agoFix like/is problem in perl5db.t.
Craig A. Berry [Fri, 27 Jun 2014 19:37:43 +0000 (14:37 -0500)]
Fix like/is problem in perl5db.t.

aaa63daea7f8ece changed like to is, which turned out not to be
quite the same thing for these tests on VMS, where the last line
of a file always gets a newline before EOF willy nilly.  So keep
is but make the newline explicit.

7 years agoGuard cur_curlyx at least with an assert.
Jarkko Hietaniemi [Sat, 28 Jun 2014 00:15:26 +0000 (20:15 -0400)]
Guard cur_curlyx at least with an assert.

Keep on keeping Coverity happy (Coverity perl5 CID 45352).

7 years agohv can be left uninitialized.
Jarkko Hietaniemi [Fri, 27 Jun 2014 23:47:32 +0000 (19:47 -0400)]
hv can be left uninitialized.

Stronger statement would be assert(), even stronger croak(),
but don't know the code well enough.

Coverity perl5 CIDs 68584 and 68585.

7 years agoprintf formats are hard.
Jarkko Hietaniemi [Fri, 27 Jun 2014 23:43:44 +0000 (19:43 -0400)]
printf formats are hard.

Coverity perl5 CID 68582.

7 years agoassert(*p = '#'); /* Spot the oops. */
Jarkko Hietaniemi [Fri, 27 Jun 2014 23:35:31 +0000 (19:35 -0400)]
assert(*p = '#');  /* Spot the oops. */

Triggered new Coverity perl5 CIDs 68580, 68581 and 68583.
(Side effect in assertion, and assignment where comparison
probably indented, the last twice because of ext/re.)

7 years agoTime functions in time64.c need thread context on VMS.
Craig A. Berry [Fri, 27 Jun 2014 12:58:42 +0000 (07:58 -0500)]
Time functions in time64.c need thread context on VMS.

In order to support the C<vmsish 'time'> pragma, the wrappers for
gmtime and localtime need thread context, which in the case of
localtime was removed in 23491f1decae17401c5.  Add it back but
make it VMS-specific and turn the comment indicating why it's
there into a more direct statement.

7 years agoregexec.c: Move some macro definitions around
Karl Williamson [Thu, 26 Jun 2014 19:09:25 +0000 (13:09 -0600)]
regexec.c: Move some macro definitions around

This puts them in a more logical order

7 years agoPATCH: [perl #122090] Non-word-boundary doesn't match EOS
Karl Williamson [Mon, 23 Jun 2014 01:52:59 +0000 (19:52 -0600)]
PATCH: [perl #122090] Non-word-boundary doesn't match EOS

The root cause of this is that the loop is a "< strend" instead of a
"<=".  However, the macro that is called to form the loop is used
elsewhere where "<" is appropriate.  So I opted to just repeat the
small salient portions of the loop right after it so it gets executed
one final time in the final position.

This code:
 if ((!prog->minlen && tmp) && (reginfo->intuit || regtry(reginfo, &s))) \
           goto got_it;
had the effect previously of causing \b to look at the position just
after the string, but not \B.  By doing it the other way, this is no
longer needed, except I don't understand the prog->minlen portion.  It
isn't used anywhere else in the tests which see if one should goto
got_it, nor do any test suite tests fail by removing it.  I think it is
a relic, but if a bisect should end up blaming this commit, I'd start
with that.

7 years agoregexec.c: Comments and white-space only
Karl Williamson [Mon, 23 Jun 2014 01:51:11 +0000 (19:51 -0600)]
regexec.c: Comments and white-space only

7 years agoregexec.c: Exchange 2 lines of code and add comments
Karl Williamson [Mon, 23 Jun 2014 01:48:23 +0000 (19:48 -0600)]
regexec.c: Exchange 2 lines of code and add comments

This reorders things to avoid unnecessary work, as the now-first line
may jump, making the assignment that used to be unconditional irrelevant

7 years agoregexec.c: More cleaning of FBC macro/code interface
Karl Williamson [Mon, 23 Jun 2014 01:41:25 +0000 (19:41 -0600)]
regexec.c: More cleaning of FBC macro/code interface

The definition of \w is now compiled into the Perl core.  This allows
the complicated swash_fetch function call to be replaced by
isWORDCHAR_utf8, which takes a single parameter, so the interface can be
simplified. [1].

This macro will execute faster on Latin1-range inputs, as it doesn't do
a swash_fetch on them, but slower on other code points due to function
call overhead, and some currently in-place error checking that wasn't
done previously.  This overhead could be removed by using inline
functions, and perhaps a different interface for known non-malformed
input (though I'm actually not sure the input is known to be well-formed
in this case).

These macros still depend on and modify outside variables.  That could
be cleaned up by adding additional parameters to them, but I'm not going
to do it now.  I don't like these kinds of code-generating macros, and
have been tempted to rewrite these as inline functions, but it's not a
trivial task to do.

[1] I hadn't realized it before, but the interface could have been
cleaned up instead by introducting a macro that makes it look like a
single parameter is used uniformly to existing macros, looking like
 #define FBC_BOUND_SWASH_FETCH(s)  \
    cBOOL(swash_fetch(PL_utf8_swash_ptrs[_CC_WORDCHAR], s, utf8_target))
But it seems better to me to use isWORDCHAR_utf8 as it is faster for
Western European languages, and can be made nearly the same speed as the
alternative if experience tells us that this is a slow spot that should
be sped up.

7 years agoregexec.c: Clean up macro/code interface slightly
Karl Williamson [Mon, 23 Jun 2014 01:13:10 +0000 (19:13 -0600)]
regexec.c: Clean up macro/code interface slightly

These code-generating macros are a pain to maintain.  These have side
effects and use variables outside the macros.  This commit cleans up the
interface slightly by separating the operand of a function from the
function, so the code outside doesn't have to know as much about the
macro as before.  The next commit will fix up the remaining parameter
like this.

7 years agoregexec.c: Change names of 4 macros
Karl Williamson [Mon, 23 Jun 2014 00:58:39 +0000 (18:58 -0600)]
regexec.c: Change names of 4 macros

The names were misleading; they were not for general use, but only for
this FBC area of the code, and they no longer have to do with having to
load utf8 or not.

7 years agoregexec.c: Change MiXeD cAsE formal macro parameters
Karl Williamson [Sun, 22 Jun 2014 03:53:03 +0000 (21:53 -0600)]
regexec.c: Change MiXeD cAsE formal macro parameters

I find these a pain to type and read, and there is no reason for them.
As formal parameters there is no possibility of conflict with the code
outside them.  And I'm about to do some editing of these, so this change
will make that easier.

7 years agoSync Module-CoreList version in for CPAN release
Chris 'BinGOs' Williams [Thu, 26 Jun 2014 09:13:38 +0000 (10:13 +0100)]
Sync Module-CoreList version in for CPAN release

7 years agoFollow-up to 51f14a05: not ignoring my_snprintf return values.
Jarkko Hietaniemi [Wed, 25 Jun 2014 19:44:49 +0000 (15:44 -0400)]
Follow-up to 51f14a05: not ignoring my_snprintf return values.

7 years agoGlossary for d_ptrdiff_t
Brian Fraser [Wed, 25 Jun 2014 13:30:37 +0000 (15:30 +0200)]
Glossary for d_ptrdiff_t

7 years agoAdd a small test case for c445c5b
Matthew Horsfall [Wed, 25 Jun 2014 14:31:20 +0000 (10:31 -0400)]
Add a small test case for c445c5b

7 years agoAdd t/porting/libperl.t
Jarkko Hietaniemi [Mon, 23 Jun 2014 18:11:50 +0000 (14:11 -0400)]
Add t/porting/libperl.t

For sanity checking libperl.a on those platforms that have one,
and extra checking for the rare configurations of -DPERL_GLOBAL_STRUCT
and -DPERL_GLOBAL_STRUCT_PRIVATE which restructure the symbol exports.

7 years agoRemove or downgrade unnecessary dVAR.
Jarkko Hietaniemi [Mon, 23 Jun 2014 11:01:34 +0000 (07:01 -0400)]
Remove or downgrade unnecessary dVAR.

You need to configure with g++ *and* -Accflags=-DPERL_GLOBAL_STRUCT
or -Accflags=-DPERL_GLOBAL_STRUCT_PRIVATE to see any difference.
(g++ does not do the "post-annotation" form of "unused".)

The version code has some of these issues, reported upstream.

7 years agoUnused snprintf results noticed by clang in Linux.
Jarkko Hietaniemi [Tue, 24 Jun 2014 22:08:58 +0000 (18:08 -0400)]
Unused snprintf results noticed by clang in Linux.

A bit dubious to ignore the snprintf results, but at least
for these cases it looks acceptable.  We could assign the
length and croak if not expected, but it seems that with
gcc brace groups we already do that.

7 years agoFix RT #122171, lookahead issues with .*
Yves Orton [Wed, 25 Jun 2014 07:27:02 +0000 (09:27 +0200)]
Fix RT #122171, lookahead issues with .*

Patterns of the form /(?=.*X)X/ were getting the IMPLICIT flag set,
which does not work with lookahead.

This used to not be a problem as we did not look at the contents of
lookahead patterns in the start-class optimizer. In 07be1b83a6b2d24
I changed this, and this case would break. That it took this long
to come to light is probably because the pattern does not really make

7 years agoperldelta for 0f9d28587772, cd293174fae5
Tony Cook [Wed, 25 Jun 2014 05:13:53 +0000 (15:13 +1000)]
perldelta for 0f9d28587772cd293174fae5

7 years agobump dist/Math-BigInt/ $VERSIONS to 1.9996
Tony Cook [Wed, 25 Jun 2014 04:53:50 +0000 (14:53 +1000)]
bump dist/Math-BigInt/ $VERSIONS to 1.9996

7 years agoCPAN RT 96254 and 96329: Correct the handling of subclasses.
Peter John Acklam [Sun, 15 Jun 2014 16:46:52 +0000 (18:46 +0200)]
CPAN RT 96254 and 96329: Correct the handling of subclasses.

Don't check for an exact class name; trust inheritance.

7 years agobump $XS::APItest::VERSION to 0.62
Matthew Horsfall (alh) [Wed, 25 Jun 2014 02:20:37 +0000 (22:20 -0400)]
bump $XS::APItest::VERSION to 0.62

7 years agoUnder GLOBAL_STRUCT mgvtbl-s are const, which angers g++.
Jarkko Hietaniemi [Tue, 24 Jun 2014 22:57:19 +0000 (18:57 -0400)]
Under GLOBAL_STRUCT mgvtbl-s are const, which angers g++.

See also 31114fe99.

7 years agoUnused contexts found under PERL_GLOBAL_STRUCT.
Jarkko Hietaniemi [Mon, 23 Jun 2014 11:01:34 +0000 (07:01 -0400)]
Unused contexts found under PERL_GLOBAL_STRUCT.

7 years agoDrop debugging, and handle HP-UX opt flags.
Jarkko Hietaniemi [Tue, 24 Jun 2014 14:28:05 +0000 (10:28 -0400)]
Drop debugging, and handle HP-UX opt flags.

7 years agoFurther hpux hints simplifying suggested by H.Merijn.
Jarkko Hietaniemi [Tue, 24 Jun 2014 13:12:48 +0000 (09:12 -0400)]
Further hpux hints simplifying suggested by H.Merijn.

7 years agoHP-UX release wider match; also drop gcc opt level.
Jarkko Hietaniemi [Tue, 24 Jun 2014 12:46:58 +0000 (08:46 -0400)]
HP-UX release wider match; also drop gcc opt level.

7 years agoRMG - Add comment that git clean -xdf after distclean should be no-op
Matthew Horsfall [Tue, 24 Jun 2014 12:11:37 +0000 (08:11 -0400)]
RMG - Add comment that git clean -xdf after distclean should be no-op

7 years agoDocument empty conditional in for/while
Peter Martini [Mon, 2 Jun 2014 11:16:59 +0000 (07:16 -0400)]
Document empty conditional in for/while

An empty conditional in both for and while has been treated
as true since perl 1.0.  This has a clear analogue in C/C++
in the case of for(;;), but while() is not legal C and should
be documented.

7 years agoremove extraneous setting of $TODO
Tony Cook [Tue, 24 Jun 2014 11:01:16 +0000 (21:01 +1000)]
remove extraneous setting of $TODO

dumper.t uses its own test framework, so it never did anything anyway.

7 years agofix multi-evals problems in pad name list api
Daniel Dragan [Wed, 11 Jun 2014 23:19:17 +0000 (19:19 -0400)]
fix multi-evals problems in pad name list api

The PAD_COMPNAME api, created in dd2155a49b , originally had alot of,
multi-eval problems, since av_fetch would be repeatedly called in macro
expansions. Later in commit b21dc0313d , an incomplete attempt at removing
multi-eval was done. Also in commit 035dab7448 added more multi-eval
problems. Prior to commit dd2155a49b , the code used a seemingly random
mix of av_fetch and AvARRAY, so both are ok. To fix this, replace av_fetch
with func-free AvARRAY. Since existing code has lval 0 to av_fetch and
unconditional deref on ret, a segv is fine to detect breakage.

A #define PAD_COMPNAME_SV(po) \
shows the AV is ! magical/tied during smoke. The assert was not added for
perf reasons on debugging builds. Inline funcs were not used for better
compiler optimizing if PAD_COMPNAME_FLAGS_isOUR is immediatly
followed by PAD_COMPNAME_OURSTASH (2 statements), as in scan_inputsymbol.
Inlines are not guaranteed to be inlined all the time on all compilers in all
situations, Visual C especially. Also inline is more likely to cause readding of
multi-eval problems than the macro if future changes to the API put the inline
func in a multi-eval macro.

On VC 2003 32bit .text section of perl521.dll dropped from 0xC296F to
0xC281F bytes of machine code with this patch.

7 years agoPERL_UNUSED_CONTEXT -> remove interp context where possible
Daniel Dragan [Sun, 15 Jun 2014 21:49:32 +0000 (17:49 -0400)]
PERL_UNUSED_CONTEXT -> remove interp context where possible

Removing context params will save machine code in the callers of these
functions, and 1 ptr of stack space. Some of these funcs are heavily used
as mg_find*. The contexts can always be readded in the future the same way
they were removed. This patch inspired by commit dc3bf40570. Also remove
PERL_UNUSED_CONTEXT when its not needed. See removal candidate rejection
rational in [perl #122106].

-Perl_hv_backreferences_p uses context in S_hv_auxinit
 commit 96a5add60f was wrong
-Perl_whichsig_sv and Perl_whichsig_pv wrongly used PERL_UNUSED_CONTEXT
 from inception in commit 84c7b88cca
-in authors opinion cast_* shouldn't be public API, no CPAN grep usage,
 can't be static and/or inline optimized since it is exported
-Perl_my_unexec move to block where it is needed, make Win32 block, context
 free, for inlining likelyhood, private api and only 2 callers in core
-Perl_my_dirfd make all blocks context free, then change proto
-Perl_bytes_cmp_utf8 wrongly used PERL_UNUSED_CONTEXT
 from inception in commit fed3ba5d6b

7 years ago[perl #116925] discuss modifying an SV's buffer
Tony Cook [Mon, 23 Jun 2014 23:55:39 +0000 (09:55 +1000)]
[perl #116925] discuss modifying an SV's buffer

7 years agoRMG - Per review - keep make distclean as a test, add comment
Matthew Horsfall [Mon, 23 Jun 2014 15:39:26 +0000 (11:39 -0400)]
RMG - Per review - keep make distclean as a test, add comment

7 years agoRMG - Per review - only -Dusedevel for development builds
Matthew Horsfall [Mon, 23 Jun 2014 15:37:34 +0000 (11:37 -0400)]
RMG - Per review - only -Dusedevel for development builds

7 years agoRMG - Clarify that updating %released/CAVEATS isn't for RCs
Matthew Horsfall [Mon, 23 Jun 2014 14:59:03 +0000 (10:59 -0400)]
RMG - Clarify that updating %released/CAVEATS isn't for RCs

7 years agoNo need for else after exit...
Matthew Horsfall [Mon, 23 Jun 2014 13:23:02 +0000 (09:23 -0400)]
No need for else after exit...

7 years agoDon't add a perldelta entry for _charnames.
Matthew Horsfall [Mon, 23 Jun 2014 12:59:59 +0000 (08:59 -0400)]
Don't add a perldelta entry for _charnames.

It's never been listed and it doesn't link properly, breaking

7 years agoRMG: Show that version'd binaries might be used
Matthew Horsfall [Mon, 23 Jun 2014 12:41:09 +0000 (08:41 -0400)]
RMG: Show that version'd binaries might be used

7 years agoRMG: Use -Dusedevel in ./Configure, like everywhere else in the RMG.
Matthew Horsfall [Mon, 23 Jun 2014 12:40:08 +0000 (08:40 -0400)]
RMG: Use -Dusedevel in ./Configure, like everywhere else in the RMG.

7 years agoRMG: Remove useless 'make distclean', save time
Matthew Horsfall [Mon, 23 Jun 2014 12:37:31 +0000 (08:37 -0400)]
RMG: Remove useless 'make distclean', save time

7 years agoUpdate makerel to provide a helpful error message if chmod fails on @writables
Matthew Horsfall [Mon, 23 Jun 2014 12:31:47 +0000 (08:31 -0400)]
Update makerel to provide a helpful error message if chmod fails on @writables

7 years agoUpdate RMG to mention new policy of perlhist containing RMs names.
Matthew Horsfall [Mon, 23 Jun 2014 12:15:11 +0000 (08:15 -0400)]
Update RMG to mention new policy of perlhist containing RMs names.

7 years agoIf we do not have vsnprintf, the len is unused.
Jarkko Hietaniemi [Sat, 21 Jun 2014 12:05:08 +0000 (08:05 -0400)]
If we do not have vsnprintf, the len is unused.

7 years agoIn response to 7fefc6c, make autodoc smarter.
Matthew Horsfall [Sun, 22 Jun 2014 11:37:09 +0000 (07:37 -0400)]
In response to 7fefc6c, make autodoc smarter.

All non-blank lines after a =head1 are now skipped to find
header documentation. This unearthed a few more missed headings
that I think are useful, and makes sure others aren't missed in
the future.

Also, fix line number counting/error reporting in, it
was woefully inaccurate.

7 years agoAllow enabling -pedantic(-errors) (with -std=c89) for the core.
Jarkko Hietaniemi [Fri, 20 Jun 2014 13:01:53 +0000 (09:01 -0400)]
Allow enabling -pedantic(-errors) (with -std=c89) for the core.

The core was already -pedantic clean (with default config, with
DEBUGGING almost), cflags just needed some juggling, somewhat
complicated by the logic of -Werror=declaration-after-statement.

Still requires -Dgccansipedantic to enable the -pedantic because
otherwise we do not use things like gcc brace groups and C99 variadic
macros, which are nice.  Also, see below about usedtrace complications.
(the gccansipedantic logic was odd, probably not used for a while.)

Under -DDEBUGGING the -pedantic requires quieting the warnings
about overlong string literals.

Disable -pedantic et al if usedtrace is enabled, it uses too many
non-standard features (and it depends on code generated by an external

7 years agog++ -DPERL_GLOBAL_STRUCT_PRIVATE died on this.
Jarkko Hietaniemi [Sun, 22 Jun 2014 02:52:43 +0000 (22:52 -0400)]

7 years agoTry simpler usage for file-level GCC_DIAG_RESTORE.
Jarkko Hietaniemi [Fri, 20 Jun 2014 13:04:34 +0000 (09:04 -0400)]
Try simpler usage for file-level GCC_DIAG_RESTORE.

7 years agoIPC-Open3.t: For d3ae482b also need the "SKIP" label from the original.
George Greer [Sat, 21 Jun 2014 23:17:09 +0000 (19:17 -0400)]
IPC-Open3.t: For d3ae482b also need the "SKIP" label from the original.

7 years agoFix black Win32 smoke broken by my v5.21.1-11-g4077a6b (yet again)
Ævar Arnfjörð Bjarmason [Sat, 21 Jun 2014 21:13:43 +0000 (21:13 +0000)]
Fix black Win32 smoke broken by my v5.21.1-11-g4077a6b (yet again)

I was being silly with my test improvements in v5.21.1-12-g826af13 and
v5.21.1-13-g43d7f0d. I haven't been able to test this on Win32 where
it's breaking, but if the sprintf format isn't recognized there we're
going to get a warning about a *redundant* argument in printf, not a
missing one, i.e. just like this:

    $ ./perl -Ilib -we 'printf "%æ\n", "str"'
    Invalid conversion in printf: "%\303" at -e line 1.
    Redundant argument in printf at -e line 1.

So fix that, I'm certain this will fix the black smoke on Win32, if not
I have no idea what's going on.

7 years agoFix test file per George Greer comment.
James E Keenan [Sat, 21 Jun 2014 21:14:46 +0000 (23:14 +0200)]
Fix test file per George Greer comment.

7 years agolib/locale.t: Improve debug info
Karl Williamson [Sat, 21 Jun 2014 19:23:42 +0000 (13:23 -0600)]
lib/locale.t: Improve debug info

I find myself needing to get the percentage of locales failing when
looking at field reports with debugging on.  This commit adds that

7 years agoMake like() and unlike() in t/ refuse non-qr// arguments
Ævar Arnfjörð Bjarmason [Sat, 21 Jun 2014 17:44:20 +0000 (17:44 +0000)]
Make like() and unlike() in t/ refuse non-qr// arguments

As I noted in v5.21.1-12-g826af13 we have subtle bugs in the test suite
because you can do e.g. like($@, '') now which'll be a passing test even
when we have an error, because $@ =~ // will be true.

I'm just changing t/ to not accept non-Regexp arguments, and
fixing up a bunch of test failures that resulted from that. There might
still be more of these in tests that I'm just not running, I've also
changed some of these from $str =~ /foo/ to $str eq 'foo'
(i.e. s/like/is/) in cases where that appeared to work, but it might
break some systems.

Let's just find that out via the smokers.

7 years agoFix black Win32 smoke broken by my v5.21.1-11-g4077a6b (again)
Ævar Arnfjörð Bjarmason [Sat, 21 Jun 2014 17:57:40 +0000 (17:57 +0000)]
Fix black Win32 smoke broken by my v5.21.1-11-g4077a6b (again)

Fix a silly bug in v5.21.1-12-g826af13, when $Q was false I added 12
extra test, but didn't declare them, do that.

7 years agoFix black Win32 smoke broken by my v5.21.1-11-g4077a6b
Ævar Arnfjörð Bjarmason [Sat, 21 Jun 2014 16:23:29 +0000 (16:23 +0000)]
Fix black Win32 smoke broken by my v5.21.1-11-g4077a6b

The problem with this was that I didn't test this on a system where $Q
was false. On those systems we not only warn about "Invalid conversion"
but also about the missing sprintf argument, as intended.

The test broke because the local __WARN__ handler would clobber the
"conversion" warning with the "missing" warning, the fix is easy, just
accumulate the warnings with ".=".

While poking at this I discovered a bug that's been here ever since the
test was added back in v5.10.0-1461-g53f65a9. If we emitted a warning on
systems where $Q was true we'd pass the test, this is because the empty
regex will match warnings, the test actually meant to use is() not
like(). It's also a bug that t/ accepts non-regexes as the second
argument, Test::More doesn't:

    $ perl -MTest::More -wle 'like "", ""'
    not ok 1
    #   Failed test at -e line 1.
    #     '' doesn't look much like a regex to me.
    # Tests were run but no plan was declared and done_testing() was not seen.

This change might cause some additional black smoke because we'll now be
detecting warnings that we previously ignored, but I don't think this'll
fail on systems where $Q is true.

7 years agoAdd a new warning about redundant printf arguments
Ævar Arnfjörð Bjarmason [Sun, 19 Jan 2014 16:27:58 +0000 (16:27 +0000)]
Add a new warning about redundant printf arguments

Implement RT #121025 and add a "redundant" warning category that
currently only warns about redundant arguments to printf. Now similarly
to how we already warned about missing printf arguments:

    $ ./miniperl -Ilib -we 'printf "%s\n", qw()'
    Missing argument in printf at -e line 1.

We'll now warn about redundant printf arguments:

    $ ./miniperl -Ilib -we 'printf "%s\n", qw(x y)'
    Redundant argument in printf at -e line 1.

The motivation for this is that I recently fixed an insidious
long-standing 6 year old bug in a codebase I maintain that came down to
an issue that would have been detected by this warning.

Things to note about this patch:

 * It found a some long-standing redundant printf arguments in our own
   ExtUtils::MakeMaker code which I submitted fixes to in and,
   those fixes were merged into blead in v5.19.8-265-gb33b7ab

 * This warning correctly handles format parameter indexes (e.g. "%1$s")
   for some value of correctly. See the comment in t/op/sprintf2.t for
   an extensive discussion of how I've handled that.

 * We do the correct thing in my opinion when a pattern has redundant
   arguments *and* an invalid printf format. E.g. for the pattern "%A%s"
   with one argument we'll just warn about an invalid format as before,
   but with two arguments we'll warn about the invalid format *and* the
   redundant argument.

   This helps to disambiguate cases where the user just meant to write a
   literal "%SOMETHING" v.s. cases where he though "%S" might be a valid
   printf format.

 * I originally wrote this while the 5.19 series was under way, but Dave
   Mitchell has noted that a warning like this should go into blead
   after 5.20 is released:

       "[...] I think it should go into blead just after 5.20 is
       released, rather than now; I think it'd going to kick up a lot of
       dust and we'll want to give CPAN module owners maximum lead time
       to fix up their code. For example, if its generating warnings in
       cpan/ code in blead, then we need those module authors to fix
       their code, produce stable new releases, pull them back into
       blead, and let them bed in before we start pushing out 5.20 RC

   I agree, but we could have our cake and eat it too if "use warnings"
   didn't turn this on but an explicit "use warnings qw(redundant)" did.
   Then in 5.22 we could make "use warnings" also import the "redundant"
   category, and in the meantime you could turn this on

   There isn't an existing feature for adding that kind of warning in
   the core. And my attempts at doing so failed, see commentary in RT

The warning needed to be added to a few places in sv.c because the "",
"%s" and "%-p" patterns all bypass the normal printf handling for
optimization purposes. The new warning works correctly on all of
them. See the tests in t/op/sprintf2.t.

It's worth mentioning that both Debian Clang 3.3-16 and GCC 4.8.2-12
warn about this in C code under -Wall:

    $ cat redundant.c
    #include <stdio.h>

    int main(void) {
        printf("%d\n", 123, 345);
        return 0;
    $ clang -Wall -o redundant redundant.c
    redundant.c:4:25: warning: data argument not used by format string [-Wformat-extra-args]
        printf("%d\n", 123, 345);
               ~~~~~~       ^
    1 warning generated.
    $ gcc -Wall -o redundant redundant.c
    redundant.c: In function ‘main’:
    redundant.c:4:5: warning: too many arguments for format [-Wformat-extra-args]
         printf("%d\n", 123, 345);

So I'm not the first person to think that this might be generally

There are also other internal functions that could benefit from
missing/redundant warnings, e.g. pack. Neither of these things currently
warn, but should:

    $ perl -wE 'say pack "AA", qw(x y z)'
    $ perl -wE 'say pack "AAAA", qw(x y z)'

I'll file a bug for that, and might take a stab at implementing it.

7 years agoSplit up the fake "missing" warning category into an actual category
Ævar Arnfjörð Bjarmason [Sun, 19 Jan 2014 14:58:02 +0000 (14:58 +0000)]
Split up the fake "missing" warning category into an actual category

Ever since the warning for missing printf arguments was added in
v5.11.2-116-g7baa469 the "missing" warning category has been defined in
terms of the "uninitialized" category, so you couldn't turn it on/off
independently of that.

As discussed in RT #121025 I'm hacking on adding a new "reduntant"
category for too many printf arguments. So add the long-missing
"missing" category in preparation for that for consistency.