[perl #92432] Storable::nfreeze shouldn't stringify ints

Storable::nfreeze's integer size check doesn't work correctly:

Using nfreeze to store an int larger than one byte produces a
frozen representation that thaws into a stringified integer, without
my having done string operations on the scalar beforehand.

I've confirmed this bug with the three versions I had available to me:
5.8.9 on 64-bit CentOS 4, 5.12.1 on 64-bit OS X 10.6, and
5.14.0 on 64-bit Debian squeeze.

The constants involved in the comparisons get created as 32-bit integers,
which wrap around and cause the comparison to yield true when it shouldn't.

This patch casts the constants before using them, which should make this
port correctly to 64-bit architectures. Running tests on a 64-bit Debian
host shows no problems.

In pod/buildtoc, use abs_from_top() to generate pathnames.

88fb56ecc95f23db used 'pod/perldelta.pod' instead of abs_from_top(), and broke
the build on Win32, which runs buildtoc from the win32 directory.

update Time-HiRes to CPAN version 1.9724

- Correct XS parameter list, and therefore prototype, for
  unimplemented-on-this-platform version of clock_nanosleep()
  [rt.cpan.org #68700].
- Declare package variables with "our" rather than "use vars".
- Corresponding to "our" usage, check for minimum Perl version
  5.006.
- Declare module dependencies.

- Remove $ENV{PERL_CORE} logic from test suite, which is no
  longer desired in the core.
- Convert test suite to use Test::More.
- Factor out watchdog code from test suite.
- In test suite, be consistent about using fully-qualified form
  of function names.
- Divide test suite into feature-specific scripts.
- Make ualarm timing test less vulnerable to delay-induced false
  failure, from Dave Mitchell.

update Math-Complex to CPAN version 1.58

* Declare package variables with "our" rather than "use vars".

* Corresponding to "our" usage, check for minimum Perl version
  5.006.

* Remove $ENV{PERL_CORE} logic from test suite, which is no
  longer desired in the core.

* In test suite, remove obsolete and now-incomplete handling of
  unavailability of Test::More.

* Declare module dependencies.

update release schedule document

perlunicode: Narrow verabitm lines so don't wrap

For 80 columns, this collection of spacing changes keeps all
lines in this pod from wrapping.

podcheck.t: sort --add_link output

The recently added --add_link option just appended lines to the end of
the data base file.  While this works, it means that the next time the
db is regen'd, that they will move around and show up in difference
listings unrelated to the original patch, thus potentially confusing
people.

podcheck.t: Reduce F<> message false positives

podcheck looks for things that might be file path references which
are enclosed in C<> instead of F<>.  But there are some things that
aren't paths that have slashes in them, like "origin/blead"  This
adds some more of these.

podcheck.t: White-space only changes

podcheck: Reduce fals positives for L<> message

podcheck looks for things like "See C<INSTALL>" and flags them as
perhaps L<> should have been used instead.  It was giving false
positives for things like "if you do x, you should see C<foo>".  This
catches those.

podcheck: Fix skip upstream cpan

Normally upstream cpan modules are not pod checked, but in the
one case where two had the same name, an error was generated
nonetheless.

podcheck.t: Fewer false positives for L<>

If there is a F<foo/bar> and a file named bar in the directory foo
exists, then assume that is what was meant, instead of raising
the possibility that L<> was meant.

podcheck.t: Suppress all L<> messages for config.pod

config.pod is a generated file that just adds C<> around all-caps words.
Until such time as it is made smarter to use L<> for links instead,
just suppress the messages that L<> might be better.

File::Find pod: fix link issue

README.macosx: Change C<> into F<>

perlapi: fix broken link, from sv.c

README.ce: Change some C<> into F<>

perlcygwin: Fix broken link

perlfunc: Fix some link issues

perlhack: Change some C<> into F<>

perlhaiku: Change some C<> into F<>

make_patchnum.pl: Change some C<> into F<>

perltodo: Change some C<> into F<>

perlrun: Fix some link issues

perlunifaq: Use L<> to actual link instead of C<>

perlxs: Fix broken link

podcheck.t: Fix typo

The match for the ">" that ends a C<> was misplaced, and should be
after looking for a file suffix.  This caused some messages to be missed

podcheck: Dashes often occur in file names

perlretut: Fix link problem

podcheck: Improve heuristics for L<>

podcheck.t uses heuristics to decide if something like a C<> should
instead be a L<>.  As noted in the comments, this adds some more checks
to eliminate some false positives

No need for D and d flags in pod.lst - source is fixed, and contains the target

As the edited perldelta is now always pod/perldelta.pod (instead of a file with
version numbers in it), we don't need a 'D' flag to mark it. As perldelta.pod
contains within itself the number of the version it refers to, we can use that
to infer the 'd' flag for the target filename.

This simplifies the release manager's tasklist for version bumping.

Record L<YAML::Syck> as a link to a known external module.

Previously it was seen as a broken link, and a "known issue" flagged against
perldelta, permitting podcheck.t to pass.

Update epigraphs.pod for 5.15.0

podcheck.t: update known issues

perldelta: note YAML::Syck as known issue

perldelta: put some things in C<> tags

perlhist: add 5.15.0 and 5.14.1 entries

Fix for slight inconsistency in perldelta

perldelta: removed confusing XSLoader commentary

perldelta: add Math::Trig to entry for Math::Complex

Update Module::CoreList for 5.15.0

perldelta: final draft

perldelta: additional pre-release amendments

update Changes with 5.15.0 references

update INSTALL with 5.15.0 references

tweak perlhack and update known POD issues

Update CUSTOMIZED in Maintainers.pl

Several cpan-upstream distributions had patches directly in blead
that do not match the upstream distribution.  These are all
now noted as CUSTOMIZED for future tracking.

Maintainers.PL: update Getopt::Long CUSTOMIZED

Revert autodie version bump

autodie has not changed from CPAN, though blead has locally patched
a single test file to avoid a platform specific failure.

This change keeps the original CPAN version numbering, as the
local test file patch is now tracked in Porting/Maintainers.pl

Add CUSTOMIZED for autodie and CGI

Improve tracking of cpan-upstream overrides

We have the CUSTOMIZED field in Maintainers.pl for indicating where we
expect blead to differ from a CPAN tarball.  This commit documents that
in perlhack and makes core-cpan-diff more aggressive about checking it
and reporting about it.

perldelta: pre-release partial cleanup

Talk to Leo about dev.perl.org, not Rafael

Create pod/perl5125delta.pod

Add the 5.12.4 epigraph to Porting/epigraphs.pod

Add 5.12.4 release date to pod/perlhist.pod

Add release date for 5.12.4 to lib/Module/CoreList.pm

Add Shigeya Suzuki’s new address to checkAUTHORS.pl

author email address change

perldelta template: who are -> who will be

perldelta: who are -> who will be

perl513*delta, perl5140delta: who are -> who will be

perldelta up to 5.12: who are -> who will be

perldelta updates for some build improvements and internal changes.

Correct the magic table in perlguts.pod following f34d15629da26eee.

Unfortunately this remains a manual step, as there's a strong distinction
between (100%) generated files that we need to ship, which the regen scripts are
allowed to wipe and replace, and any files subject to human editing, which they
aren't.

another perldelta typo

two perldelta fixes

while, not when

What are apple developer tools? Tools for developing apples? For
grafting, perhaps? :-)

Increase stylistic consistency in perldelta by adding C<> and F<>.

Correct the stated initial version of COW shared hash key scalars - the first
stable release they were in was 5.8.0, as they were added by Nick Ing-Simmons
in September 2002 in commit 1c846c1f6d96d2ca.

perldelta: Minor pod fixes

Commit a1e75797c204ade843c6eb8052cc5577af06c1d6 changed L<Shell>
to C<Shell>, but in reality there still is a Shell module accessible
from L<>, because L<> looks through all of CPAN.  podcheck.t has
to know that it's accessible, and that was done with commit
ce98560028e1830366c9182fe9dd70ec264fe6ee.

Also make the reference to Devel::DProf from C<> to L<> for symmetry.

And, a file reference should be enclosed in F<> and not C<>

Prefer the system gcc when searching for libraries such as -lm

40f026236b9959b7 added code to the Linux hints file to use gcc to locate
libraries such as -lm. However, if the user has their own gcc earlier in $PATH
than the system gcc, we don't want its libraries. So try to prefer the system
gcc.

Forgot to run keywords.pl

I didn’t realise adding a comment would require this.

Typo in podcheck.t

$ ./perl -I../lib --add_link foo
Unrecognized switch: --add_link  (-h will show valid options).

$ ./perl -I../lib porting/podcheck.t --add_link foo
Changed: /Users/sprout/Perl/perl.git/t/porting/known_pod_issues.dat

Deparse CORE::break

This is something 80e3f4adf22 missed.

Doc update for changes in 5.15.0 + tweaks

Remove unnecessary code and tests

This test was relying on bug #91844.  As that has been fixed (by com-
mit 3ed94dc, which erroneously claimed to fix #81944), this test is
not testing what it purports to test.

Also as a result of that bug fix, it doesn’t matter if the TEMP flag
is set, I can get rid of the SvTEMP_off.  The only effects the TEMP
flag can have now occur only when the refcount is 1, but since *_
holds a refcount and the mortals stack does as well, it’s going to be
2 unless someone undefs *_, in which case the callback won’t have any
reference to it.  If it tries to make a weak reference, doing so will
turn TEMP off.

perlfaq4: Use E<> instead of non-ASCII

We're trying to get rid of non-ASCII in pods.

podcheck.t: Suppress entity out of range messages

podchecker emits these messages if an E<> is above 255.  But these
actually work, so the messages are improper.  podchecker is not maintained.

podcheck: Make sure perltoc is known

perltoc is problematic because it is machine generated from constituent
pods.  This means that errors in them propgagate to it.  As a result,
commit e678c2947ab9ff776af461b393f3a3eecebab64a just skipped it.   But
this led to pods that link to it being flagged as having broken links.
This commit changes things so instead of skipping perltoc, it is added
to the list of pods that are known but are parsed only when there are
links to inside it (of which there are unlikely to be any), and then
only for the existence of such targets.

podcheck.t: Add --add_link option

L<> links in pod apply to anything that search cpan can find.  This
means that it applies to more things than podcheck can find in the Perl
distribution.  So, a list is kept in a data base of modules and man
pages that are known to exist, but aren't findable by podcheck.  This
list has to be maintained manually.  The new option makes adding to the
list easier.

The man page and instructions are somewhat tweaked as well.

podcheck.t: Add 'Shell' to list of known modules

Update Module::CoreList with v5.14.1 data

Correct a bug number in perldelta

Commit 3ed94dc0 fixed #91844, not #81944, as it claimed.

Allow ‘sub x :lvalue’ to apply to XSUBs and stubs

This was disabled in 5.12 (with a warning) by commit 885ef6f5, because
applying the attribute to a Perl sub isn’t effective: it does not mod-
ify the op tree accordingly.

But applying an attribute to an XSUB after the fact is perfectly
fine, and is the only way to do it (either with declarative syntax or
attributes.pm).  This commit restores the old behaviour of declarative
for XSUBs.  (attributes.pm never stopped working.)

Commit 885ef6f5 also stopped a declaration from applying the flag to
an undefined subroutine if it happens to have been assigned from else-
where.  It does not make sense to allow the :method attribute to be
applied to such a sub, but not :lvalue.

In pp_subst, use a mortal scalar for dstr, instead of SAVEFREESV().

Creation of the mortal can be done with newSVpvn_flags(), saving 1 function
call. Also, the mortal stack uses 1 pointer, whereas the save stack will use 2.
It doesn't matter that dstr is now marked SVs_TEMP, as both code paths already
gut it, stealing its SvPVX(). The SV head will happen to last a bit longer now,
to the next FREETMPS, instead of the the scope pop at the end of pp_subst or
pp_substcont.

Add the v5.12.4-RC2 and v5.14.1 epigraphs to Porting/epigraphs.pod

perldelta update + typo fixes

[perl #81944] Non-lvalue subs do not copy return values

return and leavesub see if they can cheat by not copying anything
marked TEMP, since presumably nothing else is using it.  That means
the return values of delete() and shift() are not copied.  Since @_
aliases to the caller’s variables, sometimes what is returned *is*
used elsewhere and still marked TEMP.  So cases like sub { return
delete $_[0] } ->($x) end up returning $x unchanged, instead of
copying it.

As mentioned in the ticket, the solution is to copy only if the refer-
ence count is 1.

This also allows me to simplify the lvalue-returning code without
spreading this bug further.  (pp_leavesublv currently avoids calling
sv_2mortal, in order not to set the TEMP flag.)

Stop lvalue subs from copying read-only scalars

They were only doing so in reference context (subroutine args,
for(...)).

Explicit return already worked, but only because I didn’t write it
well.  I’m in the process of trying to merge the two.

Test ref(\pos) and ref(\vec...)

Revert "pos in lvalue context now returns a PVMG instead of a PVLV."

This reverts commit 571f0e8653a532c34edde36e797ecba446978b1c.

I’m afraid I have to revert this, as it does not modify sv_reftype
accordingly, and doing so would add *more* complexity (the opposite
of what that commit was trying to achieve) and slow down ref() at run
time, by making it search for pos magic.

perldelta_template: No longer need to indent L<XXX>

The indentation was done to avoid podcheck.t tests failing for the
broken link to the placeholder XXX.  This is no longer necessary,
as
    1) perldelta_template itself is skipped by podcheck;
    2) podcheck now has been taught to not output a message for
       links to XXX in perldelta

podcheck.t: Special case perldelta

This adds a special case to the checks for perldelta to ignore the
placeholder links (that by necessity are broken).

podcheck.t: Ignore perldelta_template.pod

This file is not a pod, and was supposed to be ignored, but due to
an error in the logic that converts from native file system to
unix format, it was being processed.

podcheck.t: NAME ok to be used in multi perldeltas

The name of all perldeltas should be "perldelta".  So, set to -1 in the db
to indicate to ignore this check.

"who be" should be "who are" in perldelta template

"who be" should be "who are" in perldelta.pod

"who be" should be "who are" in perl*delta.pod

(5.13 and 5.14)

"who be" should be "who are" in perl*delta.pod

(up to 5.12)

perldelta: another CPAN breakage

Increase Pod::Perldoc’s version

Perl 5.14.1 is using the number 3.15_04.  What blead had until
recently was identical to 5.14.1, but now it has diverged, so
it needs a new number.