Test that SvPVutf8 works with magic vars

It didn’t until the previous commit.

[perl #108994] Stop SvPVutf8 from coercing SVs

In shouldn’t destroy globs or references passed to it, or try to
coerce them if they are read-only or incoercible.

I added tests for SvPVbyte at the same time, even though it was not
exhibiting the same problems, as sv_utf8_downgrade doesn’t try to
coerce anything.  (SvPVbyte has its own set of bugs, which I hope to
fix in fifthcoming commits.)

perldelta for read-only core globs

XS::APItest: Move $VERSION further up

so it can be changed without one having to search for it.

Increase $XS::APItest::VERSION to 0.35

perldelta for B::COP::stashflags

B.pm: Document stashflags

(actually just list in; you have to know the internals
to use these things anyway)

perldelta up to 675dc5e1a

Increase $File::Copy::VERSION to 2.23

not 2.22, as that was used by 5.13.10, but the only changes in 5.13.10
were reverted for 5.13.11+.

[perl #109104] File::Copy: Turn off ‘newline’ warnings

These warnings exist to catch file operations on unchomped file names.
But File::Copy should not be triggering them, otherwise it produces
warnings for every copy("foo/bar\n", "baz/bar\n"), with no (easy) way
to suppress the warning, as warnings are lexical.

I don’t know how to test this portably.

Remove use of study from splain

It only helps with very long strings.  It actually slows things down
slightly when used on short strings.

Increase $File::Glob::VERSION to 1.17

following commit ffa23acf6.

Add blank lines around Configure's duplicate extension diagnostic.

The diagnostic message containing the duplicate extension did not stand out,
particularly given that the immediately previous output is a very long line
of found extensions. So add blank lines on either to make it easier to
spot.

Pod-Perldoc is now maintained on CPAN

Remove from perlfunc.pod the summaries of what's new/obsoleted in Perl 5.

Perl 5.000 isn't *new*. These days, people migrating to current Perl 5 aren't
coming from Perl 4, so this section is redundant.

Purge references to --libpod, an abbreviation for --libpods

--libpods was removed from Pod::HTML in commit 3b49d8d9ac841d8e, and
references to it removed in commit 6c6fc3be9a6e74e4. However, that commit
missed the fact that 2 callers were not using the full name for the option,
which works because Getopt::Long defaults to accepting unambiguous
abbreviations.

Configure now aborts with a diagnostic if it detects a duplicate extension.

During the development process, if an extension module changes dual-life
status it can move between ext/, dist/ and cpan/. If one uses git pull in a
built tree across such a change, one can end up with the build tree in a
messy state from which the 'distclean' target can't recover. Configure will
spot both the old and new directories, and may wrongly categorise the
extension as non-XS, resulting in obscure build failures after miniperl is
build.

In the general case, the "old" directory might contain files the user wants
to keep, so it's not safe to automatically delete it. The only safe option
is to ask the user to resolve the problem, hence with this change, if
Configure detects such a situation it aborts with a diagnostic suggesting
what to do.

This isn't a problem that affects released tarballs.

In B.xs, use *pvf() functions to reduce the number of API calls.

This gives a small reduction in both source lines and object code size.

diagnostics doesn't need to search for perldelta.pod in pre-5.005 locations.

Pods are installed into privlib. Before Perl 5.005, privlib didn't contain
the version number in it, so 5.004 and 5.003 (etc) both installed to the same
directory. perldiag.pod differed in not-quite compatible ways, so hacks were
put in to (a) also install it in privlib with the version number in the
filename (b) to hard link it to archlib, which always did contain the version
number.

5.005 changed privlib to contain the version number, solving the underlying
problem (strictly commit bfb7748a896459cc, described here:
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/1998-07/msg00136.html )

Commit a841533b5cf319b3 (Oct 2009) removed the first installation hack,
Commit e8ea61279d90dbe9 (Jul 1998) removed the second.

Hence the code to search in the "hacked" locations is no longer needed,
and in the interests of clarity, should go.

podcheck.t: Add comment

locale.t: Add tests for fc()

locale.t: process in sorted order

for easier debugging, when debugging

locale.t: White space only

This indents code to account for a newly formed block around it,
reflowing things to fit into an 80 column window.

And it adjusts the indentation of a few other lines.

locale.t: Add tests for IN_UNI_8_BIT

These are more tests for commit b36bf33f6564c3e9a9ff131f4f3c9980b7a8af15

locale.t: white space only

Properly indent this statement

use locale; fc(""); shouldn't taint.

fc() brought to life its own version of #39028. fc(""), like
lc("") and friends, shouldn't taint the result.

Increase the fallback value of MAXPATHLEN

On systems without MAXPATHLEN or PATH_MAX defined (GNU/Hurd is an example
of such a system), set MAXPATHLEN to 4096 rather than 1024; this increase
creates parity with Linux.

Moving :mmap out of core binary into a module

stat.t: Use $>, not just $< for skipping

This is the paranoid version of just using $>.

[perl #108780] Make /foo$qr/ work under ‘no overloading’

This changes the code in pp_regcomp to use the underlying REGEXP
instead of the reference to it, when concatenating pieces to mark a
larger regular expression.  This makes /foo$qr/ work even under ‘no
overloading’.  It stopped working with commit a75c6ed6b.

Update IPC-Cmd to CPAN version 0.76

  [DELTA]

  Changes for 0.76    Mon Jan 30 11:30:53 GMT 2012
  =================================================
  * Make the empty arg stripping the default again,
    with option to override this behaviour.

  Changes for 0.74    Mon Jan 30 10:24:30 GMT 2012
  =================================================
  * Applied patch from WATANABE Hiroaki [RT #74470]
    "Empty string cannot be passed to command"
  * Resolved [RT #74373] reported by Randy Stauner
    "Compilation error when POSIX.pm fails to load"

Update MANIFEST

Increase $Term::ReadLine::VERSION to 1.08

Add Darin McBride to AUTHORS

[perl #108470] Make Term::ReadLine use AE if available

Term::ReadLine only allows the Tk event loop to be called during
a readline call. This should be updated to use AnyEvent which will
still work with Tk, as well as any other event loop the user may need.

With this patch, T::RL now uses AnyEvent if it is loaded, falling back
to Tk otherwise; so the Tk mode won't be affected.

T::RL::Stub has its own get_line. This does not honour the tkRunning
flag at all. If I remove it, it's fine. This patch does so.

Update IO-Compress to CPAN version 2.048

  [DELTA]

  2.048 29 January 2012

      * Set minimum zlib version to 1.2.0

      * IO::Compress test suite fails with Compress::Raw::Zlib 2.047 and zlib < 1.2.4
        [RT# 74503]

Update Compress-Raw-Zlib to CPAN version 2.048

  [DELTA]

  2.048 29 January 2012

      * Set minimum zlib version to 1.2.0

Update Compress-Raw-Bzip2 to CPAN version 2.048

  [DELTA]

  2.048 29 January 2012

      * No Changes

lc.t: Add tests for previously broken IN_UNI_8_BIT

This adds tests for commit b36bf33f6564c3e9a9ff131f4f3c9980b7a8af15

pp.c: Can grow scalar by less

The max expansion when a Latin1 character is folded and converted to
UTF-8 is '2' bytes per input byte, not the more general case.

pod updates for fc and \F

Implement the fc keyword and the \F string escape.

Along with the simple_casefolding and full_casefolding features.

fc() stands for foldcase, a sort of pseudo case (like lowercase),
which is used to implement Unicode casefolding. It maps a string
to a form where all case differences are erased, so it's a
locale-independent way of checking if two strings are the same,
regardless of case.

This functionality was, and still is, available through the
regular expression engine -- /i matches would use casefolding
internally. The fc keyword merely exposes this for easier access.

Previously, one could attempt to case-insensitively test two strings
for equality by doing

   lc($a) eq lc($b)

But that might get you wrong results, for example in the case of
\x{DF}, LATIN SMALL LETTER SHARP S.

Add the fc keyword in regen/feature.pl

It doesn't do anything yet.

utf8.h: Restore macro defn incorrectly trashed earlier

Commit 66cbab2c91fca8c9abc65a7231a053898208efe3 changed the definition
of IN_UNI_8_BIT, but in so doing, lost the 2nd line of the macro; and I
did not catch it.  Tests will be added shortly.

Add Windows 2008 to the list of Windows versions in perlport

feature.h: Mask hints in CURRENT_FEATURE_BUNDLE

Otherwise CURRENT_FEATURE_BUNDLE will end up including any hints added
later that happen to use higher bits.

This was causing autobox to turn off all features, causing failures
for Dist::Zilla::PluginBundle::AVAR.

I’m not adding tests for this, as such tests would need constant
tweaking in future perl developement.  What autobox is doing is
naughty and unsupported anyway.

[perl #109264] ->method(my(...)) forcing lvalue cx

A simple my($foo,$bar) list is flagged as an lvalue:

$ ./perl -Ilib -MO=Concise -e 'my($foo,$bar)'
7  <@> leave[1 ref] vKP/REFC ->(end)
1     <0> enter ->2
2     <;> nextstate(main 1 -e:1) v:{ ->3
6     <@> list vKPM/128 ->7
3        <0> pushmark vM/128 ->4
4        <0> padsv[$foo:1,2] vM/LVINTRO ->5
5        <0> padsv[$bar:1,2] vM/LVINTRO ->6
-e syntax OK

That 128 that the list op is the same flag as LVINTRO.

When a method call is compiled, the list op for the argument list is
itself converted into an entersub op.  That LVINTRO flag is never
turned off.  So foo->bar(my($foo,$bar)) becomes this:

$ ./perl -Ilib -MO=Concise -e 'foo->bar(my($foo,$bar))'
9  <@> leave[1 ref] vKP/REFC ->(end)
1     <0> enter ->2
2     <;> nextstate(main 1 -e:1) v:{ ->3
8     <1> entersub[t4] vKMS/LVINTRO,TARG ->9
3        <0> pushmark sM/128 ->4
4        <$> const[PV "foo"] sM/BARE ->5
5        <0> padsv[$foo:1,2] lM/LVINTRO ->6
6        <0> padsv[$bar:1,2] lM/LVINTRO ->7
7        <$> method_named[PV "bar"] ->8
-e syntax OK

This was rarely a problem until commit da1dff948 added lvalue check-
ing for method calls (a fifth bug fix in that commit not mentioned in
the commit message).

Calling the method will now result in ‘Can't modify non-lvalue subrou-
tine call’ unless the method has the :lvalue attribute.

Before that, this would only cause problems with lvalue methods:

$ perl -le '
    sub clear_queue:lvalue { warn "called"; undef }
    3==main->clear_queue(my ($id, $name))
'
called at -e line 2.
Can't return undef from lvalue subroutine at -e line 3.

Calling it with ($id, $name) was fine, and allowed undef to
be returned.

Perl_localize in op.c (which is called for my, our and local)
calls my() (aka Perl_my_attrs) on the list itself for my or our.
Perl_my_attrs was setting flags on the list, not just on its children.

So this commit modifies my_attrs not to set any flags on the list
op itself.

local() was not affected, as it goes through op_lvalue_flags instead
of my_attrs, and op_lvalue_flags doesn’t set flags on list ops (I
mean ops of type OP_LIST, not listops in general).  I added tests for
it anyway.

Update IO-Compress to CPAN version 2.047

  [DELTA]

  2.047 28 January 2012

      * Set minimum Perl version to 5.6

      * IO::Compress::Zip
        - In one-shot zip, set the Text Flag if "-T" thinks the file is a
          text file.
        - In one-shot mode, wrote mod time & access time in wrong order
          in the "UT" extended field.

Update Compress-Raw-Zlib to CPAN version 2.047

  [DELTA]

  2.047 28 January 2012

      * Allow flush to be called multiple times without any intermediate
        call to deflate and still return Z_OK.
        In the code below $status was Z_BUF_ERROR before this change.

          $def->flush(...);
          $status = $def->flush(...);

      * Added support for zlibCompileFlags

      * Set minimum Perl version to 5.6

Update Compress-Raw-Bzip2 to CPAN version 2.047

  [DELTA]

  2.047 28 January 2012

      * Set minimum Perl version to 5.6

Trim directory extension in pathify under EFS.

When Extened Filename Syntax (EFS) is in effect, 1fe570cc5e24ee
changed the age-old behavior of trimming the directory extension
off a directory filename when the path is in Unix syntax.  EFS
really has nothing to do with the necessity to trim .DIR;1 from
the name, so remove the conditional check for EFS from the
existing code.

This gets three more tests passing under EFS in vms/ext/filespec.t.

Fix bug #109206: ANCH_MBOL with while /.*/g

We had a fencepost error when ANCH_MBOL was enabled that meant we
did not "see" matches at the end of string. This fixes the problem
and adds tests.

accidentally used my work email for a commit so add it to checkAuthors.pl

fix variable not shared warnings coming from the use statement being executed too early

Increase $subs::VERSION to 1.01

checkAUTHORS.pl: Add another (old?) address for Juerd

[perl #21396] documentation error in subs.pm

Adjust skip counts in t/op/filetest_t.t.

Follow-up to 8db8f6b697e6f, where new tests were added without
changing the (implicit) skip count of 1.

Restore directory extension and version in fileify under EFS.

In df2786654 and 8a5aa89570, the traditional behavior of adding
the .DIR;1 onto a fileified directory spec was removed when
operating under Extended Filename Syntax.  Various scary comments
were added about its being a bug to add a type and version onto
a Unix-style path, but actually the CRTL appears to be perfectly
happy with, for example:

   stat('/foo/bar/baz.dir;1');

and without the extension, the home-grown rmdir() fails in the
case of a directory with no preceding path information.  E.g.,

   rmdir('foo');

was failing because there was no internal translation to foo.dir
before passing it to SYS$ERASE.

Moreover, even if there were something wrong with adding .DIR;1,
it has nothing to do with EFS.

filelify shouldn't always escape dots.

Follow-up to a9fac63d75d9222a73fbf511ca58ae1d66cbf9a7.  It turns
out that the CRTL doesn't handle the escapes when the path is in
Unix syntax (even though it requires them in native syntax).

  stat('foo/bar.baz.dir;1')

is ok, but

  stat('foo/bar^.baz.dir;1')

fails.  So skip the escaping if there is a slash anywhere in the
path.

ExtUtils::ParseXS: Changelog, version bump for dev release

Upgrade the version in ExtUtils::ParseXS to 3.13_01 for a development
release.

mymalloc isn't thread safe (rurban)

Allow prototypes (_@) and (_%)

Those will be equivalent to (_;@) and (_;%) ; since perlsub already
states that the semicolon is redundant before @ and % this is in
line with the existing documentation.

Show test names in output

Make blead upstream for warnings.pm

This isn’t even a dual-life module.  Why it has its own entry I don’t
know; but in any case it has to have blead for upstream, otherwise
cmp_version.t skips it.

Increase $warnings::VERSION to 1.13

Allow ${^WARNING_BITS} to turn off lexical warnings

Various magical modules copy hints from one scope to another.  But
copying ${^WARNING_BITS} doesn’t always copy the same hints.  If lexi-
cal warnings are not on at all, ${^WARNING_BITS} returns a different
value depending on the current value of $^W.  Setting ${^WARNING_BITS}
to its own value when $^W is true will stop $^W from being able to
control the warnings in the current compilation scope.  Setting
${^WARNING_BITS} to its own value when $^W is false causes even
default warnings to be suppressed.

This commit makes undef a special value that represents the default
state, in which $^W controls warnings.

pat.t: Test that . overloading gets passed qr ref

This is something that my sample patch in ticked #108780 (for
fixing /foo$qr/ under ‘no overloading’) would have broken had it
been applied.

stat.t: Skip test when running as root

Bump $Unicode::UCD::VERSION after commit c3e5bc54acca7110.

Update list of files for a minimal installation in INSTALL.

Update the minimal list of files for 5.15.7, and also for Debian's
perl-5.10.1 base package (which is what is in the Debian stable
distribution at present).  These lists do not include any explicit advice
about what to name those stripped down installations.

Correctly escape UTF-8 in hash keys in uninitialized warnings

Initialize buffer to an empty string to avoid spurious uninitialized warnings

Update known_pod_issues.dat for x2p/psed

Fix pod error in Unicode::UCD

Update DB_File to CPAN version 1.826

  [DELTA]

  1.826 25 Jan 2012

   * t/db-btree.t - fix use of "length @array"
     [RT ##74336]

Revert "Regen known_pod_issues.dat"

This reverts commit 5fa409a90f64110c5708f7141b376e9bdc54fbe2.

Resolved by update of Pod-Parser to version 1.51

Update Pod-Parser to CPAN version 1.51

  [DELTA]

  25-Jan-2012           Marek Rouchal                        <marekr@cpan.org>
  -----------------------------------------------------------------------------
  Version 1.51
  + CPAN#74271: Pod::Parser should not flag L<text|hyperlink>
    undo change CPAN#47473: L<alt text|URL> _is_ allowed (see discussion on
    pod-people and p5p); confirmed by blead change; references:
    http://code.activestate.com/lists/perl-pod-people/1690/
    http://code.activestate.com/lists/perl-pod-people/1393/
    http://perl5.git.perl.org/perl.git/commit/f6e963e4dd62b8e3c01b31f4a4dd57e47e104997
  + bump all module versions (except Pod::PlainText) to 1.51
  + update META.yml (thanks rjbs for the hint)

Make the display of the warning "Useless use of a constant (%s)" more robust

If the constant is a string (POK), we dump it using pv_pretty, to
properly escape non-printable characters. This also improves detection
of utf-8 encoding in the constant to be printed. Also, this patch
streamlines the tests for the type and value of that constant.

Potential backwards-compatibility issues : if the constant is a string
it will now be enclosed in double quotes, like this :
    Useless use of a constant ("a") in void context
instead of
    Useless use of a constant (a) in void context

CYG17 utf8 paths

Use cygwin_conv_path since cygwin-1.7 and support native utf-8 paths.

Update DB_File to CPAN version 1.825

  [DELTA]

  1.825 24 Jan 2012

    * t/db-btree.t - fix use of "length @array"
      [RT ##74336]

  and [perl #108970]

Increase $overload::VERSION to 1.18

In overload::AddrRef, use ‘no overloading’

This speeds this up by about 13 times.

[perl #108780] Make ‘no overloading’ work with qr//

Traditionally, overload::StrVal(qr//) has returned
Regexp=SCALAR(0xc0ffee), and later Regexp=REGEXP(0xc0c0a) when regexps
were made into first-class SVs.

When the overloading pragma was added in 5.10.1, qr// things were not
accounted for, so they would still stringify as (?-xism:) even with
‘no overloading’ (or as (?^:) under 5.14).

This commit makes the overloading pragma work with qr// things, so
that they stringify the same way as overload::StrVal; i.e., as
Regexp=REGEXP(0xbe600d).

Move amagic hint checking to new function

so that stringification will be able to use it, too.

Call FETCH once for stacked filetest ops

skip sigdispatch.t on cygwin RT#88814

[perl #77388] Make stacked -t work

Up till now, -t was popping too much off the stack when stacked with
other filetest operators.

Since the special use of _ doesn’t apply to -t, we cannot simply have
it use _ when stacked, but instead we pass the argument down from the
previous op.

To facilitate this, the whole stacked mechanism has to change.

As before, in an expression like -r -w -x, -x  and -w are flagged
as ‘stacking’ ops (followed by another filetest), and -w and -r are
flagged as stacked (preceded by another filetest).

Stacking filetest ops no longer return a false value to the next op
when a test fails, and stacked ops no longer check the truth of the
value on the stack to determine whether to return early (if it’s
false).

The argument to the first filetest is now passed from one op to
another.  This is similar to the mechanism that overloaded objects
were already using.  Now it applies to any argument.

Since it could be false, we cannot rely on the boolean value of the
stack item.  So, stacking ops, when they return false, now traverse
the ->op_next pointers and find the op after the last stacked op.
That op is returned to the runloop.  This short-circuiting is proba-
bly faster than calling every subsequent op (a separate function call
for each).

Filetest ops other than -t continue to use the last stat buffer when
stacked, so the argument on the stack is ignored.

But if the op is preceded by nothing other than -t (where preceded
means on the right, since the ops are evaluated right-to-left), it
*does* use the argument on the stack, since -t has not set the last
stat buffer.

The new OPpFT_AFTER_t flag indicates that a stacked op is preceded by
nothing other than -t.

In ‘-e -t foo’, the -e gets the flag, but not in ‘-e -t -r foo’,
because -r will have saved the stat buffer, so -e can just use that.

[rt.cpan.org #74289] Don’t make *CORE::foo read-only

newATTRSUB requires the sub name to be passed to it wrapped up in
a const op.

Commit 8756617677dbd allowed it to accept a GV that way, since
S_maybe_add_coresub (in gv.c) needed to pass it an existing GV not in
the symbol table yet (to simplify code elsewhere).

This had the inadvertent side-effect of making the GV read-only, since
that’s what the check function for const ops does.

Even if we were to call this a feature, it wouldn’t make sense as
implemented, as GVs for non-ampable (&-able) subs like *CORE::chdir
were not being made read-only.

This commit adds a new flag to newATTRSUB, to allow a GV to be passed
as the o parameter, instead of an op.  While this may look as though
it’s undoing the simplification in commit 8756617677dbd by adding
more code, the new code is still conceptually simpler and more
straightforward.

Since newATTRSUB is in the API, I had to add a new _flags variant.
(How did newATTRSUB get into the API to begin with?)

In adding a test, I also discovered that ‘used once’ warnings
were applying to these subs, which is obviously wrong.  Commit
8756617677dbd caused that, too, as it was relying on the side-effect
of newATTRSUB doing a GV lookup.

This fixes that, too, by turning on the multi flag in
S_maybe_add_coresub.

Fix bad pointer size in has_trailing_nul

point git-using installers to pod/perlgit

...this change was promised to Jim Keenan

perldelta for ~~ fix

Enforce Any ~~ Object smartmatch precedence

Increase $B::VERSION to 1.34

B.xs add missing B::COP::stashflags

Threaded perl added this field with 5.15.4 for utf8 stashes.

B.pm POD: revise 44f7f2d5b1 on IoFLAGS

IoFLAGS are not defined in perliol.h

More un-TODOs and two typos in vms/ext/filespec.t.

Two tests that actually pass under Extended Filename Syntax and
two that will now pass if we give them the correct expectations.

Handle directories with dots in fileify under EFS.

When Extended Filename Syntax (EFS) is in effect, it's legal for
directory names to contain dots that are not directory delimiters.
This commit escapes such dots when they are part of the file
portion of the fileified directory spec.

This gets two more of the vms/ext/filespec.t tests passing when
EFS is in effect.

perlpodspec: Remove outdated text

This warning against using alternate text with hyperlinks is no longer
valid, See http://code.activestate.com/lists/perl-pod-people/1393/

And relevant changes:

https://github.com/theory/pod-simple/commit/1e61e819debf9c7c23907d7bb9e37855665fd595
http://perl5.git.perl.org/perl.git/commit/f6e963e4dd62b8e3c01b31f4a4dd57e47e104997
https://github.com/theory/test-pod/commit/ae6a44894eda4fd09fb412d837efe543628cd7d6

sv.c:sv_utf8_encode: simplify code

sv_force_normal already croaks for read-only variables