Add Louis Strous to AUTHORS

Confusing comment typo in Deparse

Clean: Remove confusing comment from a test

As per:

  Re: [PATCH 2/2] Clean: Actually use HvUSEDKEYS() instead of HvKEYS()
  Sun, 15 May 2011 20:29:05 +0000
  Message-Id: <2a28ed668b6641e6b5f6e0fb4e5374b4-mfwitten@gmail.com>

the text in question was introduced along with the test in this commit:

  $ c=58da6fbcb8d595318b667d7d7cc8739f5feb15f3
  $ git log -1 $c
  commit 58da6fbcb8d595318b667d7d7cc8739f5feb15f3
  Author: Max Maischein <corion@corion.net>
  Date:   Mon Jun 1 14:18:42 2009 +0200

      Add benchmark test for keys() on empty hashes (RT26188)

I believe the text in question is an allusion to the code introduced in
the parent commit:

  $ git log -1 $c^
  commit 900ac0519e4b408fd93443b14b734cc330c59698
  Author: Max Maischein <corion@corion.net>
  Date:   Sun May 31 23:50:12 2009 +0200

      Fix RT26188, speed up keys() on empty hash

namely the code starting at line 2132 of
900ac0519e4b408fd93443b14b734cc330c59698:hv.c:

   /* At start of hash, entry is NULL.  */
   ...
  if (HvKEYS(hv) || (flags & HV_ITERNEXT_WANTPLACEHOLDERS)) {
      ...
  }
  ...
  return entry;

It is best just to remove the text in question, as it is of no
real use.

Signed-off-by: Michael Witten <mfwitten@gmail.com>

Clean: Actually use HvUSEDKEYS() instead of HvKEYS()

This:

  commit 8aacddc1ea3837f8f1a911d90c644451fc7cfc86
  Author: Nick Ing-Simmons <nik@tiuk.ti.com>
  Date:   Tue Dec 18 15:55:22 2001 +0000

      Tidied version of Jeffrey Friedl's <jfriedl@yahoo.com> restricted hashes
       - added delete of READONLY value inhibit & test for same
       - re-tabbed

      p4raw-id: //depot/perlio@13760

essentially deprecated HvKEYS() in favor of HvUSEDKEYS(); this is
explained in line 144 (now 313) of file `hv.h':

  /*
   * HvKEYS gets the number of keys that actually exist(), and is provided
   * for backwards compatibility with old XS code. The core uses HvUSEDKEYS
   * (keys, excluding placeholdes) and HvTOTALKEYS (including placeholders)
   */

This commit simply puts that into practice, and is equivalent to running
the following (at least with a35ef416833511da752c4b5b836b7a8915712aab
checked out):

  git grep -l HvKEYS | sed /hv.h/d | xargs sed -i s/HvKEYS/HvUSEDKEYS/

Notice that HvKEYS is currently just an alias for HvUSEDKEYS:

  $ git show a35ef416833511da752c4b5b836b7a8915712aab:hv.h | sed -n 318p
  #define HvKEYS(hv) HvUSEDKEYS(hv)

According to `make tests':

  All tests successful.

README.macosx - clarify requirement of apple developer tools

README.macosx - remove Carbon reference

README.macosx - cleanup/update copy, remove 'MacPerl'

Cleanup README.macosx

- Direct people for README.macosx straight away
- Make synopsis copy shorter and clearer

Update AUTHORS file (add myself)

Add Nuno Cavalho’s other e-mail address

add given when to perfunc

buildtoc: make dying clearer

print a big "ABORTED" if it dies.

buildtoc: make a warning clearer

make it clear that the cannot find '=head1 NAME' warning isn't fatal,
and split it over two lines ofr clarity with long filenames

[perl #78074] Make it explicit that symtab manipulation is not supported

Restore a croak I deleted by mistake

RT72688 - Benchmark.t test 24 bug when the testing system is under load.

release_managers_guide: replace =item with =head3

The main body of the Release Manager's Guide is currently just a long list
of steps, each with a '=item *'. Replace these with
    =head3 some description
so that its easier to navigate around.

Also do a few minor structural fixups along the way (eg splitting or
merging individual =item sections).

Also a few typos and bits fixed as I noticed them.

release_managers_guide: add BLEAD-POINT/-FINAL

the Release Manager's Guide talks about three different release types,
RC, BLEAD and MAINT. This is a bit ambiguous for a major release
(e.g. 5.14.0): is it the first MAINT or the last BLEAD?

So, split BLEAD into two:

BLEAD-FINAL - a final release on a blead branch (e.g. 5.14.0)
BLEAD-POINT - a point release on a blead branch (e.g. 5.15.0, 5.15.1)

Then update the whole document to use these tags consistently.

release_managers_guide: remove SNAPSHOT

Now that we make regular blead releases, we no longer need to make
snapshot releases. So remove the instructions for SNAPSHOT that
litter release_managers_guide.pod.

given wasn't calling set magic [RT#84526]

TODO tests for RT#84526 - given needs to handle magical TARG

Revert "Partially revert 4155e4fe"

This reverts commit 7f586e41ad4ecd904c7d8dbe1ddb0f9410484bac.

Revert "silence do_curse() compiler warning"

This reverts commit 82336099d393c4ac04507e58e4402ba9c413f791.

Ignore pod/*.{html,man}

Signed-off-by: Michael Witten <mfwitten@gmail.com>

waitpid doesn't work with WIFSTOPPED

> Quoth Emmanuel Rodriguez:
>> I'm see a strange behavior in the fonction WIFSTOPPED($?) when
>> waitpid($pid, WUNTRACED) is invoked and that the child process
>> receives a stop signal. Under this conditions one would expect that
>> WIFSTOPPED($?) would return a true value, but that's not what is
>> happening. A similar program written in C will have the same macro
>> return true instead of false under the same conditions.
>>
>> I can reproduce this with the default perl provided by Ubuntu 9.10 and
>> OS X 10.6. Which lets me guess that this is not a distro related bug.
>
> This is a documentation error.  POSIX.pod incorrectly claims that
> you can pass the value of $? to WIFEXITED and its relatives.
> You must use the raw status value from ${^CHILD_ERROR_NATIVE} instead.

And here's the patch.  Note that perlvar.pod gets it right already.

/Bo Lindbergh

[perl #90898] B::Deparse failure on glob()

Change d1bea3d changed the way that glob() is converted into
a CORE::GLOBAL::glob call, by preserving the OP_GLOB node, so
that it could still do overloading.

But B::Deparse was never updated to account for the change to
the op tree.

[perl #90160] U* gives ‘U0 mode on an empty string’

This is a regression in 5.10 caused by change 23966/08ca2aa38a29,
which added a bit of faulty logic. It was treating U* in the middle of
a pack template as equivalent to U0, if the input string was empty.

update Math-Complex to CPAN version 1.57

* Add copy constructor and arrange for it to be called
  appropriately, problem found by David Madore and Alexandr
  Ciornii.

* Correctly format polarwise when a numeric format specifier is
  given, problem found by TomC.

* More stable great_circle_direction algorithm, problem found
  by Daniel Burr.

update Time-HiRes to CPAN version 1.9722

- Fix broken linkage on Windows with gcc 3.4 seen with ActivePerl,
  report from Christian Walde [rt.cpan.org #61648], fix derived
  from Vincent Pit.
- Jump through hoops to avoid compiler warnings.

[perl #89896] Locale::Maketext test failure
when environment has variable containing unbalanced brackets

Patch is attached to escape brackets in the value of the environment
variable being used in the test.

In Perl_regdupe_internal() eliminate npar, which is assigned to but never used.

It has been unused since 28d8d7f41ab202dd restructured the regexp dup code.

In S_regpiece(), only declare parse_start if conditional code needs it.

gcc 4.6.0 warns about variables that are set but never read, and unless
RE_TRACK_PATTERN_OFFSETS is defined, parse_start is never read. So avoid
declaring or setting it if it's not actually going to be used later.

In Perl_safesyscalloc(), only declare total_size if conditional code needs it.

gcc 4.6.0 warns about variables that are set but never read, and for most
combinations of conditional compilation options, total_size is never read.
So avoid declaring or setting it if it's not actually going to be used later.

In pp_warn and pp_die, eliminate pv, which is assigned to but never read.

Eliminate C variables unused since 4d0fbddde6c5dcb9 refactored HvFILL()

3 functions had C variables previously used to track whether the number of
hash chains have any entries. 4d0fbddde6c5dcb9 refactored the hash
implementation to calculated this on demand, instead of tracking changes to
it on hash updates. That change missed eliminating those variables, as
gcc prior to 4.6.0 didn't actually warn that they weren't used, because
(technically) they aren't unused - they are assigned to, but never read.

gcc (at least 4.3.2 and 4.6.0) generates identical object code with/without
this change, implying that its optimiser correctly eliminates the code.
Other optimisers may be similar, in which case there's no runtime saving from
this change.

mktables: Put off removing the \N{BELL} conflict

Unicode 6.0 re-used the name BELL for a character other than what it
was traditionally for.  There is code in mktables to change the
treatment for 5.15, so that it wouldn't be forgotten.  But I'm thinking
that we might want to wait til 5.18 to change, and so I just changed it
to 5.17 instead.

[perl #89662] PATCH to perlfunc.pod: select fix

I think it's about time--by say 20 years--that we stop passing around
filehandles as strings, especially if we don't use Symbol::qualify_to_ref()
to fix the package.  Plus it isn't strict-safe.  All fixed now.

I am somewhat concerned about the final comment regarding read
and sysread, as I don't know whether PerlIO encoding translation
issues affect this picture.

--tom

[perl #89660] PATCH to perlfunc.pod: three forgotten prototypes, unforgotten

Improved perlpodstyle docs from tchrist

An editing pass on perlop.pod from tchrist

Subject: [perl #89490] PATCH: perlop.pod

perllol doc updates from tchrist.

This patch is aagainst the "blead du moment".  It fixes various
things, but does not address the "this version" != 5.14 and the
perlio issues.  I believe that these at least should be addressed
as soon as possible (I don't know whether that means 5.14;
probably not).  It has to either say the particular version that
applies.  We can't keep releasing things that say this version:
they are not trustable.

--tom

Update the version of Darwin we know the by_BE locale to be broken on

Add Vladimir Timofeev to AUTHORS

Fix for [perl #90106] 5.14.0-RC2 ODBM_File failures on Ubuntu natty [multiarch]

Ubuntu has juggled lib paths so we get to play games to catch up with
them.

S_sv_unmagicext_flags wasn't declared static either

S_mg_findext_flags wasn't declared static

In S_doparseform(), don't force the pattern to a string. This resolves #79532

Previously S_doparseform() was using SvPV_force(), because the pattern had to
be forced to a string, because the compiled format was stored in the string's
buffer. Now that the compiled format is stored in the magic struct, this isn't
necessary.

Additionally, removing the call to SvPV_force() removes the need to hack with
the SvREADONLY() flag in pp_formline.

Store the compiled format in mg_ptr instead of after SvCUR() - fixes RT #89218

Formats are compiled down to a sequence of U32 opcodes in doparseform().
Previously the block of opcodes was stored in the buffer of SvPVX() after
the raw string by extending the buffer, and calculating the first U32 aligned
address after SvCUR(). A flag bit on the scalar was set to signal this hackery,
tested with SvCOMPILED()

The flag bit used happened to be the same as one of the two used by to signal
Boyer-Moore compiled scalars. The assumption was that no scalar can be used for
both. Unfortunately, this isn't quite true.

Given that the scalar is alway upgraded to PVMG to add PERL_MAGIC_fm magic,
to clear the cached compiled version, there's no extra memory cost in using
mg_ptr in the MAGIC struct to point directly to the block of U32 opcodes. The
test for "is there a compiled version" can switch to mg_find(..., PERL_MAGIC_fm)
returning a pointer, and the use of a flag bit abolished.

Retain SvCOMPILED() and SvCOMPILED_{on,off}() as compatibility for XS code on
CPAN - the first is always 0, the other two now no-ops.

S_doparseform() should return void, not OP*, as it should use Perl_die not DIE

a1b950687051c32e added an error condition in S_doparseform() but used DIE(...)
to report it. DIE is defined as C<return Perl_die>, which acts as a hint to the
compiler about the control flow [as Perl_die() never returns], but also forces
the return type to be OP *. Whilst this is appropriate for pp functions, it's
not for S_doparseform() - a1b950687051c32e had to change the return type to OP*
and return NULL, just to appease DIE(). Hence use Perl_die() instead, remove
return statements, and remove the didn't-return-NULL (dead) code from
pp_formline.

perlreapi: nits

op_reg_common.h: Fix comment

perluniintro: revise text on blocks vs scripts

perlop: Add explanation of \c

perluniprops: specify exact chars in PerlSpace

perlrecharclass: Nits

perlrecharclass: Move table

The table makes more sense moved; some accompanying wording cleanup.

perlrecharclass: Move text about \N

This should come right after the text about '.', as they are related.

perlrecharclass: Extra leading zero in code points

regcomp.sym: Add comment

perlop: Clarify that only ASCII brackets nest

perluniprops: AHex and PosixXDigit are identical

charnames: pod nit

perlfunc: Refer to if.pm from "use"

regcomp.c: change comment wording, from TomC

perlxs: Fix pod errors

perlunicode: Fix pod error

perlport: Fix pod errors

pod/perlfaq2: Fix pod errors

handy.h: Link moved to perlhacktips

README.os2: Fix pod errors

pod/perlinterp.pod: Fix broken link

ext/Devel-Peek/Peek.pm: Fix broken links

dist/Cwd/lib/File/Spec/Win32.pm: Fix broken link

version/Internals.pod: Fix broken links

README.os2: fix broken link

FieldHash.pm: Fix broken links

perlsyn: fix broken link to this

The X<> have spaces between them which was causing the =head to
generate a different anchor than expected

perlop.pod: Fix broken link

The reason there are links broken to this is that the X<>
were part of the heading, and the spaces between them are
significant

perlfaq4: Fix broken link

perlxs: Fix broken link

perlthrtut: fix broken links

perlhack: Fix broken link

perlguts: Fix broken link

README.os2: Fix broken links

Internals.pod: Fix broken link

overload.pm: Fix broken link

perlform: Revise link

The link doesn't have to be to the exact text, can just name the function,
and doing so is clearer.

configpm: Fix broken links

perlsub.pod: Fix broken link

perlguts: Link to internal doesn't need pod name

perlfaq9: Add links to man pages

perlfaq8: Add links to man pages

perlebcdic.pod: Add man page section number to link

perldata.pod: Fix broken link

mathoms.c: pod: fix broken link

vmsish.pm: Fix broken link

Attribute-Handlers: Fix broken link

README.cygwin: Fix broken link

Internals.pod: Fix broken links