regcomp.c: Add comment

regcomp.c: Add comment, reorder #define more logically

re/pat.t: Remove obsolete comment

(?foo:...) loses passed in charset

This commit looks for the passed-in charset, and overrides it only if it
is /d and the pattern requires /u.  Previously the passed-in value was
ignored.

Patch [perl #111400] [:upper:] broken for above Latin1

This was an off-by-one error caused by my failing to realize that things
had to be done differently at the 255/256 boundary depending on whether
U+00FF matched or did not match the property.

Two properties were affected, [:upper:] and [:punct:].  The bug was that
all code points above the first one > 255 that legitimately matches the
property will match whether or not they should.  In the case of
[:upper:], this meant that effectively anything from 256..infinity
matched.  For [:punct:], it was anything above U+037D.

Corion's release is done. Check him off

av_fetch: de-duplicate small bit of code

make the code slightly smaller by changing

    if (A)
return X;
    if (B)
return X;

into
    `
    if (A || B)
return X;

Merge the fixes for RT #37033 into blead.

The parser should always close the file handle that it opened.

Previously it would leave the file handle open if it was (equal to) stdin,
on the assumption that this must have been because no script name was
supplied on the interpreter command line, so the interpreter was defaulting
to reading the script from standard input.

However, if the program has closed STDIN, then the next file handle opened
(for any reason) will have file descriptor 0. So in this situation, the
handle that require opened to read the module would be mistaken for the above
situation and left open. Effectively, this leaked a file handle.

This is now fixed, by explicitly tracking from parser creation time whether
it should keep the file handle open, and only setting this flag when
defaulting to reading the main program from standard input. This resolves
RT #37033.

Move the close-on-exec logic to one place, at the end of S_open_script().

Now that the logic for stdin is implemented as an early return of NULL from
S_open_script(), in all cases that reach the end of S_open_script(), rsfp
is non-NULL, and a file handle that we wish to set to close on exec.

Change S_open_script() to return NULL to signal "read from stdin".

Move the logic to assign PerlIO_stdin() to rsfp from S_open_script() to its
only caller, S_parse_body().

In perl.c, change S_open_script() to return rsfp.

Previously it was being passed &rsfp as a parameter, because it was
returning another value, fdscript. However, the return value has been
ignored since commit cc69b689ee7c2745 removed suidperl in January 2009.

In struct yy_parser, change lex_flags to a U8, from part of a bitfield.

lex_flags holds 4 flag bits, with multiple flag bits manipulated together
at times, so they can't be split out into individual bitfields. This change
permits the C compiler to generate simpler code, reducing toke.o by about
400 bytes on this platform, but doesn't change the size of the structure.

lex_flags was added in commit 802a15e9c01d1a0b in August 2011, so is not in
any stable release.

Improve the tests for -x

Test the error message generated when -x can't find a "#!perl" line.
Test that this error message still appears when -x is used with -e.

Improve the tests for the -n and -p switches.

Verify that -p actually runs the code in the program body.
Verify that -n doesn't implicitly print out the contents of $_.
For both, verify that an END block runs after the implicit loop.

Simplify platform specific code in t/run/cloexec.t

Don't assign to two lexical variables, $Is_VMS and $Is_Win32, only to use
them immediately for the same purpose - to skip the entire test.
In turn, there's no need to conditionally set $quote to a value suitable for
VMS or Win32, when neither OS ever runs the test.

The code has been this way since the file was added by commit
742218b34f58f961 in Nov 2006. Hence I don't think that the vestigial $quote
logic corresponds to pre-commit version that did run on these platforms.
Instead I infer that it has come from t/op/exec.t, used as a template for
running sub-scripts in a portable fashion.

perldiag: Reflow for better splain output

Add an example how to fix '%s: command not found'

Increase $attributes::VERSION to 0.18

doco improvement for attributes.pm

It seems that many people have trouble understanding how to add custom
attributes to their subroutines.  Here's a doc patch that will
hopefully make things clearer:

Make CPAN upstream for File::Temp

It was set to undef, which meant it hadn’t been discussed.  In actuality,
it is actively maintained on CPAN.  See, for instance:

    https://rt.cpan.org/Ticket/Display.html?id=75077#txn-1038945

So this brings Maintainers.pl closer in line with reality.

Don’t ‘normalise’ hints for bare ‘no feature’

‘Normalise’ in this case means to set $^H to indicate that features
are in %^H (FEATURE_BUNDLE_CUSTOM) and to make %^H contain the current
feature set.

Since ‘no feature’ sets the default feature bundle in $^H, this is
unnecessary in that case.

regen/feature.pl: Show perl.h line num in error msg

perlfunc: bad wording

It is not features not in the current version, but those not in the
requested version, that are disabled.

arybase.xs: Wrap PL_check safely

Increase $arybase::VERSION to 0.05

don't taint $$ determined by getpid()

Reading $$ in a tainted expression was tainting the internal sv_setiv()
on $$.  Since the value being set came directly from getpid(), it's
always safe, so override the tainting there.  Fixes [perl #109688].

update perlfunc.html generator

The format of the individual function HTML files has changed.  The index
generator needs to update to successfully extract the NAME sections.
Fixes [perl #107870].

delay allocating trans table until needed

Previously, a table was being allocated for OP_TRANS(|R), in a
PVOP arrangement, as soon as the op was built.  However, it wasn't
used immediately, and for UTF8-flagged ops it would be thrown away,
replaced by an SV-based translation table in a SVOP or PADOP arrangement.
This mutation of the op structure occurred in pmtrans(), some time after
original op building.  If an error occurred before pmtrans(), requiring
the op to be freed, op_clear() would be misled by the UTF8 flags into
treating the PV as an SV or pad index, causing crashes in the latter
case [perl #102858].  op_clear() was implicitly assuming that pmtrans()
had been performed, due to lacking any explicit indication of the op's
state of construction.

Now, the PV table is allocated by pmtrans(), when it's actually going to
populate it.  The PV doesn't get allocated at all for UTF8-flagged ops.
Prior to pmtrans(), the op_pv/op_sv/op_padix field is all bits zero,
so there's no problem with freeing the op.

Update the link for Module-Build/core integration

mktables: Correct Unicode 6.1 omission

Unicode 6.1 erroneously omitted Takri as a script that uses two
characters, and have voted to publish the correction that this patch
makes.  There isn't an official Corrigendum yet.

correct a Pod foible that broke blead

Add description of new change process for dev.perl.org

Leo prefers now direct pull requests via Github.

regcomp.c: Add cast to silence Solaris warning

Remove ExtUtils-CBuilder's MANIFEST.SKIP

Apparently, 'make distclean' tries to use it in confusing and
counterproducive ways.  Easier to remove it than fix the other.

work harder to get useful diagnostics on Win32

The pod2html tests can do diffs of big hunks, and so they try to
use /usr/bin/diff to show how things differ.  If there is no such
program, we just say "it wasn't good."

Unfortunately, Win32 is where we keep having problems, and where
we are least likely to have diff available.  When there is no diff,
we now use Test::More::is, which will output the whole differing file.

Noisy, yes, but now smoke reports with failing Pod::Html tests will
be more useful.

ExtUtils::CBuilder: add support information

Adds a documentation note about support.  Adds a README.patching
file with instructions for bumping versions, Changes, etc.

Update Maintainers.pl for new ExtUtils::CBuilder

Also updates release instructions with a reminder to
update Maintainers.pl

add ext/Pod-Functions/.gitignore

ExtUtils::CBuilder - add dual-life support files

Adds Makefile.PL, LICENSE, etc. and updates Changes for
work in blead from 5.14.0

ignore all MYMETA files, not just .yml

"no feature" now means reset to default

See https://rt.perl.org/rt3/Ticket/Display.html?id=108776

"no feature" now resets to the default feature set.  To disable all
features (which is likely to be a pretty special-purpose request, since
it presumably won't match any named set of semantics) you can now
write "no feature ':all'"

unixify paths a bit more systematicaly in Pod-Html

Rather than unixifying paths sort of once off, we now do what I
threatened earlier: unixify all the --switch supplied paths during
command line parsing.

There are other bits of File::Spec being sprinkled around, quite
possibly in too-interior sections of the program, but this change
gets *all* tests passing on Win32 as well as GNU/Linux.

I think an audit of filepath normalization "border security" is
still due, but this may tide us over for now.

don't assume that filesys starts at / in Pod-Html

The test expectations for the --htmlroot feature wanted to look for
an absolute path, so it concatenated / with the relative-to-root path.

That failed on Win32, where the cwd was C:\etc

I added a new token to the test-expectations-munger for the absolute
cwd.  I'm not 100% sure about the features' behavior, but the tests
are now less platform-specific.  I look forward to input on whether
the code is doing the right thing now...

unixify the --outfile arg to pod2html

I think we probably need a more systematic regimen of of unixify
calls, but for now, this gets tests passing on Win32.

Make corelist-perldelta.pl keep the existing heading

86c08a2ca2546ef08513c65dabf686423cade2f3 changed
Porting/corelist-perldelta.pl to be more accepting in section headings
when reading in an existing perldelta.

caaa1415975f6b8763b186234d77803148a4fd37 changed it to output
the correctly pluralized form of "Pragmata".

corelist-perldelta.pl used the same data structure for both, input
and output. This patch divides up the logic between input and output
in two data structures, and keeps the heading of the input
file when generating a fresh pod section.

Fix typo

Pluralize "Modules and Pragma" to "Modules and Pragmata"

Add perl5158delta.pod

Add epigraph for 5.15.8

Fix bug in pod2html crossreferencing

makerel should not add -uncommitted to tarballs

For some time now (I suspect since 691ce773f), makerel has always tried
to build releases as perl-5.x.y-uncommitted.tar.gz.  This always
confuses and/or panics new release engineers, who see the error before
the "don't worry, use -s ''" advice in the RMG.

This change just makes makerel skip the usually-#ifdef-ed-away line that
is getting that -uncommitted.  makerel still picks up other
locally-applied patches, but will skip uncommitted.

Merge branch 'release-5.15.8' into blead

Fix typos in pod/perldelta.pod

Update pod/perldelta.pod from the version numbers

add new release to perlhist

Update Module::CoreList for v5.15.8

Clean out XXX in preparation for Module::CoreList update

Bump versions from 5.15.7 to 5.15.8

Update perldelta.pod for 5.15.8 for review

Pending remains the list of changed/upgraded module versions, to be
done after the version bump.

handy.h: Silence Solaris compiler warning

Making this an unsigned constant silences the scary and wrong Solaris
warnings about integer overflow

podcheck.t: Typo in pod

Avoid defining the same global between perl and re.pm

This caused -Uusedl builds to fail due to the duplicate symbol.

A more complex fix can be expected after 5.16.

Make t/porting/authors.t work on Windows

The Windows shell cmd.exe does not know about single quotes. Double
quotes are the only thing available here. This patch selects the kind
of quote to use based on the operating system and constructs
the pipeline accordingly.

Revert updates to compression libraries

Revert "Update Compress-Raw-Bzip2 to CPAN version 2.049"
This reverts commit db98b5a25ac5537be04f17243c5e929aceb3560b.

Revert "Update Compress-Raw_Zlib to CPAN version 2.049"
This reverts commit 8b782679b6ea1247bd559efc311e7366582feaad.

Revert "Update IO-Compress to CPAN version 2.049"
This reverts commit 785e05e1c22f7d7c6ed38839cc28cc6d13a286de.

epigraphs.pod: Rmv illegal spaces after | in L<>

perlintro: Add missing closing ">"

...and add unadded file

remove MANIFEST entries for pruned files

Refactor the Tk event loop to allow for other event loops

Term::ReadLine supports any event loop, including unpubished ones and
simple IO::Select loops without the need to rewrite existing code for
any particular framework.

In S_validate_suid(), move declarations after the first statement.

Otherwise exacting C89 compilers refuse to build the code. And at least
one Win32 compiler is exacting on this.

Refactor code in Data::Dumper to assume that >=5.8.0 is the common case.

The 5.8.0 versions of &init_refaddr_format and &format_refaddr are now
compiled by default, and then redefined with their 5.6.x replacements if
it turns out that we're on 5.6.1 or 5.6.2.

Previously both versions were compiled, and the correct pair bound using
typeglob assignment.

Fix typo in 985213f2fede57.

Which broke the build on both VMS and Win32.

Remove gete?[ug]id caching

Currently we cache the UID/GID and effective UID/GID similarly to how
we used to cache getpid() before v5.14.0-251-g0e21945. Remove this
magical behavior in favor of always calling getpid(), getgid()
etc. This resolves RT #96208.

A minimal testcase for this is the following by Leon Timmermans
attached to RT #96208:

    eval { require 'syscall.ph'; 1 } or eval { require 'sys/syscall.ph'; 1 } or die $@;

    if (syscall(&SYS_setuid, $ARGV[0] + 0 || 1000) >= 0 or die "$!") {
            printf "\$< = %d, getuid = %d\n", $<, syscall(&SYS_getuid);
    }

I.e. if we call the sete?[ug]id() functions unbeknownst to perl the
$<, $>, $( and $) variables won't be updated. This results in the same
sort of issues we had with $$ before v5.14.0-251-g0e21945, and
getppid() before my v5.15.7-407-gd7c042c patch.

I'm completely eliminating the PL_egid, PL_euid, PL_gid and PL_uid
variables as part of this patch, this will break some CPAN modules,
but it'll be really easy before the v5.16.0 final to reinstate
them. I'd like to remove them to see what breaks, and how easy it is
to fix it.

These variables are not part of the public API, and the modules using
them could either use the Perl_gete?[ug]id() functions or are working
around the bug I'm fixing with this commit.

The new PL_delaymagic_(egid|euid|gid|uid) variables I'm adding are
*only* intended to be used internally in the interpreter to facilitate
the delaymagic in Perl_pp_sassign. There's probably some way not to
export these to programs that embed perl, but I haven't found out how
to do that.

Add t/porting/pending-author.t, fixing a limitation of t/porting/authors.t

t/porting/pending-author.t attempts to avoid the problem of C<make test>
passing 100%, but the subsequent git commit causing F<t/porting/authors.t>
to fail, because it uses a "new" e-mail address.

This test is only run if one is building inside a git checkout, B<and> one
has made local changes. Otherwise it's skipped.

t/porting/authors.t only needs to pass the Author to checkAUTHORS.pl

Commit 3ea0c581844689ab didn't go far enough in pruning the input.
When Porting/checkAUTHORS.pl is invoked with --tap it uses
parse_commits_from_stdin_authors() instead of parse_commits_from_stdin(),
which only looks for lines matching /^Author:/

This reduces runtime by a further 8%.

Correct the location of t/porting/perlfunc.t in perldelta.pod

It was somewhere completely wrong, thanks to no human double checking of
a git rebase during development.

Longer filenames for while_readdir.t.

With filenames varying between only one and five characters in
length, it was fairly easy to get two files differing only in
case ('A' and 'a', for example).  Which on non-case-sensitive file
systems could generate a warning at unlink time because it would
unlink 'a' and then check for the existence of 'a' and get true
because 'A' was still there and indistinguishable from 'a'.

So just use longer filenames to make the possibility of collision
vanishingly small.

perlvar: $] is not deprecated

See <20011022034838.B1676@blackrider> and commit 0c8d858bc.

Update IO-Compress to CPAN version 2.049

  [DELTA]

  2.049 18 February 2012

      * IO::Compress::Zip
        Error in t/cz-03zlib-v1.t that caused warnings with 5.15
        [RT# 110736: warnings from cpan/IO-Compress/t/cz-03zlib-v1.t]

Update Compress-Raw_Zlib to CPAN version 2.049

  [DELTA]

  2.049 30 January 2012

      * Include zlib 1.2.6 source.

Update Compress-Raw-Bzip2 to CPAN version 2.049

  [DELTA]

  2.049 18 February 2012

      * No Changes

Update perlfaq to CPAN version 5.0150039

  [DELTA]

  5.0150039 Sat 18 Feb 2012 15:33:17 +0100
    * Rewrite "How do I create a module?" (ranguard)
    * Remove more old questions (ranguard)
    * Improve Email::MIME example (madsen)
    * Fix to regex (RT #74215)

populate the Future Deprecations section

These items aggregate the bullets put forth on the perl5-porters
list, omitting those to which there was an immediate objection.

Merge the Pod::Functions refactoring to blead.

Describe the update to Pod::Functions in perldelta.

Add metadata to perlfunc.pod for which version or feature added a function.

Whilst this isn't used (yet), it makes sense to put it in now, before any
external program decides that parsing the private Pod::Function blocks in
perlfunc.pod is fair game, and then becomes dependent on their format.

Test that Pod::Function's descriptions are stylistically consistent.

Function descriptions should start with a lowercase letter, or the proper
noun SysV.

Test that functions in the groups in perlfunc.pod are in sorted order.

This ensures consistency in the documentation, and that the functions in
%Pod::Functions::Kinds are in sorted order.

Test that every function in perlfunc.pod has a summary for Pod::Functions.

Historically Pod::Functions has failed to get updated when functions are
added. This should solve that.

Add t/porting/perlfunc.t so that porting tests catch problems with perlfunc

Pod::Functions is now generated from pod/perlfunc.pod by
ext/Pod-Functions/Functions_pm.PL

If it can't parse pod/perlfunc.pod, it will abort, which will cause the
build to break. It's really not possible for it to carry on, hence aborting
is the only option. However, innocent-seeming changes to documentation
shouldn't break the build, and we expect everyone to run (at least)
the porting tests, hence this test, to catch such problems before it's too
late. To avoid duplicating the parsing logic, we make Functions_pm.PL take
a --tap option, to test that all is well.

In the generated Pod::Functions, store all function data with tab separation.

Previously the data was stored with a comma separated list inside a tab
separated list.

Generate Pod::Functions from perlfunc.pod

This avoids it getting out of synchronisation.

Add a dependency for ext/Pod-Functions/Functions_pm.PL on pod/perlfunc.pod

This ensures that it gets re-run, and lib/Pod/Functions.pm rebuilt, if
perlfunc.pod is changed.

Ensure Pod::Simple is built before Pod::Functions.

This will allow Pod::Functions to use Pod::Simple as part of its build
process.

Change ext/Pod-Functions to generate Functions.pm from a Perl script.

For the first step, simply output the current file in one hit.

Add annotations to perlfunc.pod for Pod::Functions.

In the list for "Perl Functions by Category" each item is annotated with its
type as used internally by Pod::Functions.

In the "Alphabetical Listing of Perl Functions" each function is annotated
with the summary description returned by %Pod::Functions::Flavor.

Move Pod::Functions from lib/ to ext/

Bring the joy of strict and warnings to Functions.t

This reveals that use_ok() was not in a BEGIN block, and in turn that the
test count needs to be declared before this BEGIN block runs. Now fixed.

Terser code in Pod::Functions to generate $Type_Description and @Type_Order.