Re: [perl #71852] Warning "/* within comment" in Win32 config.h-related files

(Applied patch and regenerated canned config files)

Message-ID: <4B44FB2B.8080509@volny.cz>

Retain builtin attributes from pre-declaration. Fixes [perl #68758].

Ignore a lvalue attribute after the subroutine has been defined, and warn about it. Fixes part of [perl #68758].

[perl #71924] shmget/semget may return zero as valid id

Documentation in perlipc thus fixed by using // instead of ||

Completely avoid autovivification of CORE::GLOBAL::caller

(by using symbolic references as suggested by Vincent)

Typo fix : caller:: isn't caller

(Noticed by Aaron Crane)

Add new prototype warning test file to MANIFEST

Perldelta presentation nits

Move prototype parsing related warnings from the 'syntax' top level warnings category to a new 'illegalproto' subcategory.

Two warnings can be emitted when parsing a prototype -

  Illegal character in prototype for %s : %s
  Prototype after '%c' for %s : %s

The first one is emitted when any invalid character is found, the latter
when further prototype-type stuff is found after a slurpy entry (i.e. valid
character but in such a place as to be a no-op, and therefore likely a bug).

These warnings are distinct from those emitted when a sub is overwritten by
one with a different prototype, and when calls are made to subroutines with
prototypes - those are in the pre-existing sub-category 'prototype'.

Since modules such as signatures.pm and Web::Simple only need to disable
the warnings during parsing, I chose to add a new category containing only
these. Moving these warnings into the 'prototype' sub-category would have
forced authors to disable more warnings than they intended, and the entire
raison d'etre of this patch is to allow the specific warnings involved to
be disabled.

In order to maintain compatibility with existing code, the new location
needed to be a sub-category of 'syntax' - this means that

  no warnings 'syntax';

will continue to work as expected - even in cases like Web::Simple where all
subcategories extant prior to this patch are re-enabled (this is another
reason why a move into the 'protoype' category would not achieve the desired
goal).

The category name 'illegalproto' was chosen because the most common warning
to encounter is the "Illegal character" one, and therefore 'illegalproto'
while minorly inaccurate by ignoring the (relatively recent and unknown)
second warning is an easy name to spot on an initial skim of perllexwarn
and will behave as expected by also disabling the case of an unusual prototype
that happens to look like a normal one.

This patch updates pod/perllexwarn.pod, perldiag.pod and perl5113delta.pod
to document the new category, toke.c and warnings.pl to create and implement
the new category, and a new test t/op/protowarn.t that verifies the new
behaviour in a number of cases. It also includes the files generated by
regen.pl that are found in the repo - notably warnings.h and lib/warnings.pm.

2.024

[perl #71948] Documentation error for (*MARK)

The NAME portion of (*MARK:NAME) is not optional.

Update skip count in perlio.t.

Follow-up to af9379e9ed4daaed65ba42baa492afc842917dd5.

PL_sv_serial only exists ifdef DEBUG_LEAKING_SCALARS

Fix for 4149198fba64273f3fea8fc073ccb5d080059f4a which broke
debugging build. Whoops!

Update Archive-Extract to cpan version 0.38

  Changes for 0.38    Wed Jan  6 23:48:52 2010
  ============================================
  * Apply a patch from Michael G Schwern RT #53246
    extract() is vulnerable to print globals.

fix for [perl #66108] Leaked scalars

@DB::args is a hack: it gets set with non-refcounted aliases of the
caller's @_ elements.

Once the sub that ran caller() has exited, @DB::args will contain garbage:
elements will be SVs that have been freed, re-assigned etc.

So as a minimum, when cloning an interpreter, skip cloning @DB::args.

reset sv_serial for each new interpreter

to a value of 0 + 1000000*interpreter_number
where interpreter_number is currently just a dirty hash of my_perl

in perl_clone() move common code outside ifdef/else blocks

e diff looks confusing, but basically it changes:

    #ifdef PERL_IMPLICIT_SYS
      ...
      COMMON_BLOCK
    #else
      ...
      COMMON_BLOCK
    #end

into
    #ifdef PERL_IMPLICIT_SYS
      ...
    #else
      ...
    #end
    COMMON_BLOCK

in perl_clone() shuffle some ifdefs about a bit

No change in logic.

unify #ifdef indent on both sides of PERL_IMPLICIT_SYS in perl_clone()

include sv_debug_serial field in debugging output

Avoid race conditions with files in /tmp, by explicitly checking dev & inode.

(Concerns raised by and the form of the solution suggested by Bram.)

Correct errors in the use of multiple targets, which could break parallel make.

Where a single script (or program) generates multiple files, the Makefile rule
was written with those multiple files as targets on the same line. This would
be correct if that syntax meant "these $n things are built by these commands".

However it doesn't - it means "to build any of these targets, run these
commands", and hence to run them once for each of the targets. This can result
in race conditions with parallel makes, with files being trampled on and
strange failure modes.

Hence the correct style is to give one of the targets on the rule, and rules
that make all the other targets that are generated depend on that target.

Unlink PerlIO's tempfiles for the case of no -T, but bogus $ENV{TMPDIR}

When -T is enabled, or when $ENV{TMPDIR} is bogus, perlio.c used a pathname
matching </tmp/PerlIO_??????>. However, it was only correctly unlinking the
file for the case of -T enabled.

Cleanup PerlIO temp files

[perl #71870] Use of uninitialized value in bitwise and B::Deparse

It's better to just silence it in Deparse rather than stopping
B::PMOP::reflags from returning undef because of a non-constant regexp.
That way, we keep the possibility to test for this situation, and it stays
in line with B::PMOP::pregcomp.

Also bump $B::Deparse::VERSION.

Fix typo in reference

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

Document that interpolating a '(??{ code })' construct in a regular
expression also requires "use re 'eval'", just as '(?{ code })' does.

Fix a couple of typos and standardize on using 'regexp' instead of
using a 'regex'/'regexp' mash.

vaproto for the other OS's

Follow-up to d03b3b00ac22f32af87a752669a46d9d06ae1561

Increase default pipe buffer size on VMS to 8192 on 64-bit systems.

As Charles Lane's piping code reaches its 10-year anniversary, it's
time to bump up the default to something slightly more appropriate
for modern systems.  In addition to other benefits, this fixes test
failures when a test script outputs a large amount of text on one
line (e.g., t/re/reg_posixcc.t).

vaproto for VMS.

Follow-up to d03b3b00ac22f32af87a752669a46d9d06ae1561, which broke
the build.

[perl #71748] Bleadperl f0e67a1 breaks CPAN: Template::Plugin::YAML::Encode 0.02

Unsurprisingly, the nature of the bug is that I accidentally changed
the logic of one of the several types of space skipping.  Fix attached.

Avoid a possible race condition where a parallel make might
attempt to update git_version.h and lib/Config_git.pl in
concurrent processes.

Sort the initialisers to non_bincompat_options[], and fix a whitespace "error".

Add USE_ATTRIBUTES_FOR_PERLIO to the list of -V's compile-time options.

Tie::Hash::NamedCapture::* shouldn't abort if passed bad input [RT #71828]

Note which t/ subdirs are not suitable for test.pl use

FD 4 is not (yet) open at this stage.

Regenerated after backporting 88a6f4fc380d30c40
Please *do* remember to notify the metaconfig folk when directly patching Configure
Bring back Missing parts

PATCH: minor typo cleanup of pod/ directory

These are all in the pod/ directory, and only the first is a code fix.
There was also a single lingering ISO 8859-1 encoding that missed the
UTF-8 upconvert.  The rest are cleanups for typos, some of which seem
to have been around for a rather long time: spelling errors, incorrect
possessives, and extra, missing, or duplicated words.

If you actually read through, I bet you'll realize what sparked this. :)

--tom

Signed-off-by: Abigail <abigail@abigail.be>

perlguts: mutli-threaded typo

--
Reini Urban
http://phpwiki.org/           http://murbreak.at/

From 76ebe81f50d97e606c4243d5060d4749f7408deb Mon Sep 17 00:00:00 2001
From: Reini Urban <rurban@x-ray.at>
Date: Mon, 4 Jan 2010 18:09:25 +0000
Subject: [PATCH 1/2] perlguts: mutli-threaded typo

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

perlguts: B::Generate already works

--
Reini Urban
http://phpwiki.org/           http://murbreak.at/

From 520f6e768bdf71ad35740698df2d4c7f0bb7ae9e Mon Sep 17 00:00:00 2001
From: Reini Urban <rurban@x-ray.at>
Date: Mon, 4 Jan 2010 18:16:14 +0000
Subject: [PATCH 2/2] perlguts: B::Generate already works

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

A mechanism for inlineable OP equivalents of XSUBs is a TODO.

Following fd909433c7437296, check that {sub f} correctly stubs the subroutine.

Allow "{sub f}" to compile

Bump version of UNIVERSAL.pm

dial back warnings on UNIVERSAL->import

still warn on UNIVERSAL->import(@args)
do not warn on UNIVERSAL->import; this means "use UNIVERSAL;" is
still accepted without warning (for better or worse)

advice regarding make regen changes

CC compiler updates for >=5.10

    PP_EVAL: nullify the string loop, analog to the old retstack.
             pp_leaveeval sets: retop = cx->blk_eval.retop
    PP_ENTERTRY: See PERL_FLEXIBLE_EXCEPTIONS in cop.h

TT is not a terminal in lots of situations.

d1585ceaf66a98c6f5f3520ecf3dec4ccc98a1d6 didn't go far enough.  TT may
be a mailbox when we're in a subprocess, for example.  So skip on VMS
unless TT translates to one of the more common terminal device names.

s/text message/text editor/ in the last change to perlrepository.pod

Spotted by David E. Wheeler.

Update perlrepository.pod with new guidelines on commit messages and patches.

Karl Williamson pointed out to me that our existing guidelines on
commit messages in 'perlrepository.pod' were somewhat less than ideal.

This commit improves our documentation for commit messages, though it
may want cleanup or further improvement.

Correct \p{print} to not match LINE SEPARATOR nor PARAGRAPH SEPARATOR

The Unicode Standard defines (as a recommendation) that Print be based on
graphical characters and blank characters (minus controls).  Perl's has been
based on space rather than blank. The only practical effect this has is that
Perl erroneously matches the LINE SEPARATOR and PARAGRAPH SEPARATOR, which
clearly are not printable characters.

Signed-off-by: Abigail <abigail@abigail.be>

Add a usage scenario and common use pattern to perldoc -f quotemeta

PATCH: correct grammatical error in perlunicode.pod

Attached

From 75bb462da5f7ea844447dfdd7d9aadfe15f6dcf3 Mon Sep 17 00:00:00 2001
From: Karl Williamson <khw@khw-desktop.(none)>
Date: Tue, 29 Dec 2009 13:08:28 -0700
Subject: [PATCH] Correct grammatical error in perlunicode.pod

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

Guard open.t against Encode not being available

Similar changes should/could be made to cpan/CGI/t/utf8.t
and ext/XS-APItest/t/utf16_to_utf8.t

Reasoning is my smokes being built without Encode, because
it takes too long.

PATCH: document all Perl Unicode \p{} extensions

This also changes some C<> constructs.

From d01b049b3aa9bc3a394adb30d6db735f5dd52321 Mon Sep 17 00:00:00 2001
From: Karl Williamson <khw@khw-desktop.(none)>
Date: Mon, 28 Dec 2009 09:14:48 -0700
Subject: [PATCH] Document all perl Unicode \p extensions

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

PATCH: Makefile in lib/unicore shouldn't clean .t file

Attached.

From 35eb2eee53740951b4f3dd7e96dc030d38e84ba7 Mon Sep 17 00:00:00 2001
From: Karl Williamson <khw@khw-desktop.(none)>
Date: Sun, 27 Dec 2009 20:30:00 -0700
Subject: [PATCH] make clean shouldn't remove .t file

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

* fix a perlop fix from debian: bugs.debian.org/514814

The original said

     C<< <I<filehandle>> >>

The italic is probably meant for "filehandle", which is a stand-in
for a concrete filehandle. The italic should not apply to the
syntax for the line input operator.

The fix turned it inside out and brought back the ugly interior
sequences:

     I< C< E<lt>filehandleE<gt> > >

It's much better without worrying about the italic face.

Update pods

Signed-off-by: Abigail <abigail@abigail.be>

Update .pods

Signed-off-by: Abigail <abigail@abigail.be>

[perl #70748] threads panic in del_backref

This was caused by change 34210/41fae7a, which simply reveals a bug that
already existed.

A sub returned from a thread brings a lot of baggage with it, including
some globs. There is this comment near the top of Perl_sv_dup in the
if(param->flags & CLONEf_JOIN_IN) block that reads:
        /** don't clone stashes if they already exist **/

Then later on, under case SVt_PVGV:
        /* Don't call sv_add_backref here as it's going to be
           created as part of the magic cloning of the symbol
           table. */

So what’s happening is that there is a glob with no back-reference in its
stash, which glob is sometimes freed after the stash, so it induces the
panic.

Fix up pods for \X

expand-macro.pl was printing the header include to STDOUT by mistake.

This stopped it expanding macros in header files not directly or indirectly
included via perl.h or EXTERN.h

The bug was introduced with change 4784c5e80ccedd3734ce162e02ed1c7e25e60c01.

The use of PTH will be unsupported in perl-5.12 for HP-UX 10.20

have corelist-perldelta.pl generate add'l sections

fix CPAN.pm test broken by hotfixes

local version number bumps for CPAN.pm hotfixes

note CPAN hotfixes in perldelta

hotfix CPAN.pm to force only ftp URLs

CPAN::FTP has problems with curl and other command line
programs.  Having mirror autoconfig prefer http URLs was
triggering these bugs for any bootstrap installation without
LWP.

This hotfix forces only ftp URLs from autoconfig which
hides the problem for now until CPAN::FTP can be fixed
upstream.

hotfix App::Cpan

Suppresses uninitialized warnings and NullLogger output.  Changes
have already been submitted to upstream repos.

fix pod typo in how_to_write_a_perldelta.pod

Update Module::CoreList for the new Encode (and seeming missing updates from M::B update)

First pass at noting the update to Encode 2.39 rgs applied

Full initial pass at perldelta. ready for sanity checking

Merge branch 'blead' of git+ssh://perl5.git.perl.org/perl into blead

* 'blead' of git+ssh://perl5.git.perl.org/perl:
  Unicode documentation updates

Note U+0FFFF regex support in perldelta

Unicode documentation updates

Merge branch 'blead' of git+ssh://perl5.git.perl.org/perl into blead

* 'blead' of git+ssh://perl5.git.perl.org/perl:
  Upgrade to Encode 2.39

further perldelta triage

Upgrade to Encode 2.39

Further perldelta triage.

Removing pod boilerplate

Merge branch 'blead' of git+ssh://perl5.git.perl.org/perl into blead

* 'blead' of git+ssh://perl5.git.perl.org/perl:
  change non-char warning message from malformed
  Remove spurious case of warning "Use of %s without parentheses is ambiguous"

Further perldelta hacking

change non-char warning message from malformed

Remove spurious case of warning "Use of %s without parentheses is ambiguous"

Eric Brine pointed out that this warning doesn't apply to ".",
as in C<rand . 1>, that shouldn't warn since C<. 1> cannot be
mistaken for a floating point number.

more perldelta updates

describe new CPAN.pm, ExtUtils::MakeMaker and Module:Build

Small fixups from corion

update perldelta with module and pragma version updates

Add 5.11.3 to perlhist.

Remove Roman numerals from README.vms after confirmation from Craig that it's not actually important

Update AUTHORS file for 5.11.3

Update Module::CoreList for Module::Build update to 0.36

Updated Module::Build to 0.36

0.36 - Sun Dec 20 15:02:38 EST 2009

No changes from 0.35_15 other than the version number.

Summary of major changes since 0.35:

 Enhancements:

 - Added 'Build installdeps' action to install needed dependencies via
   a user-configurable command line program.  (Defaults to 'cpan'.)

 - Command line options may be set via the PERL_MB_OPT environment
   variable (similar to PERL_MM_OPT in ExtUtils::MakeMaker)

 - Generates MYMETA.yml during Build.PL (new standard protocol for
   communicating configuration results between toolchain components)

 - Reduced amount of console output under normal operation (use --verbose
   to see all output)

 - Added experimental inc/ bundling; see Module::Build::Bundling for
   details.

 New or changed properties:

 - Added 'share_dir' property to provide File::ShareDir support;
   File::ShareDir automatically added to 'requires' if 'share_dir' is set

 - Added 'needs_compiler' property.  Defaults to true if XS or c_source
   exist.  If true, ExtUtils::CBuilder is also added to build_requires.

 - 'C_support' is no longer an optional feature.  Modern ExtUtils::CBuilder
   and ExtUtils::ParseXS added to the 'requires' list.  This ensures that
   upgrading Module::Build will upgrade these critical modules.

 - Clarified that 'apache' in the license attribute indicates the Apache
   License 2.0 and added 'apache_1_1' for the older version of the license
   (RT#50614)

 Deprecations:

 - Module::Build::Compat 'passthrough' style has been deprecated.  Using
   'passthrough' will issue warnings on Makefile.PL generation.  See
   Module::Build::Compat documentation for rationale.

 Internals:

 - Replaced use of YAML.pm with YAML::Tiny; Module::Build::YAML is now
   based on YAML::Tiny as well

 - A new get_metadata() method has been added as a simpler wrapper around
   the old, kludgy prepare_metadata() API.

 - Replaced guts of new_from_context().  Build.PL is now executed in a
   separate process before resume() is called.  (This is generally only of
   interest to Module::Build or toolchain developers) (RT#49350)

 - Add support for 'package NAME VERSION' syntax added in Perl 5.11.1

 Notable bug fixes:

 - The "test" action now dies when using the 'use_tap_harness'
   option and tests fail, matching the behavior under Test::Harness.
   (RT#49080) [initial patch from David Wheeler; revised by David Golden]

 - Updated PPM generation to PPM v4 (RT#49600) [Olivier Mengue]

 - When module_name is not supplied, no packlist was being written; fixed
   by guessing module_name from dist_version_from or the directory name
   (just like ExtUtils::Manifest does without NAME) [David Golden]

 - Failure to detect a compiler will now warn during Build.PL and be a
   fatal error when trying to compile during Build. (RT#48918) [David
   Golden]

 - Auto-detection of abstract and author fixed for mixed-case POD headers
   (RT#51117) [David Wheeler]

 - resume() was not restoring additions to @INC added in Build.PL
   (RT#50145) [David Golden]

 - When tarball paths are less than 100 characters, disables 'prefix'
   mode of Archive::Tar for maximum compatibility (RT#50571) [David Golden]

 - Merging 'requires' and 'build_requires' in Module::Build::Compat could
   lead to duplicate PREREQ_PM entries; now the highest version is used
   for PREREQ_PM. (RT#50948) [David Golden]

 - Module::Build::Compat will now die with an error if advanced,
   non-numeric prerequisites are given, as these are not supported by
   ExtUtils::MakeMaker in PREREQ_PM [David Golden]

update perldetla instructions to mention the new test-file autogenerater

Update perldelta with newly added tests, using corion++'s new automation

Generate a stub for newly added tests since the last release (2nd try, with fixed pod)

Regenerate Module::CoreList and bump to 2.24 for Perl 5.11.3