Move $func_args into $self.

Move $RETVAL_code into $self.

Move $gotRETVAL into $self.

Move $deferred into $self.

Eliminate 'our' $printed_name

Replace it with a 'my' variable declared inside INPUT_handler().
Permissible, because $printed_name is explicitly passed to subs
called from within INPUT_handler() and only there.

Move $thisdone and $retvaldone to $self.

Add $Packprefix to $self.

Move $ret_type into $self.

$pname posed interpolation problems.

More globals in $self

Move $Prefix, $processing_arg_with_types, $proto_in_this_xsub,
$scope_in_this_xsub, $interface, $interface_macro,
$interface_macro_set, $ProtoThisXSUB, $ScopeThisXSUB into $self.

Move more globals into the global $self

Move $proto_re, $Overload, $errors, $Fallback, $hiertype,
$WantPrototypes, $WantVersionChk, $WantLineNumbers,
$filepathname, $dir, $filename, $BLOCK_re, $lastline, $lastline_no
into $self. $proto_re not used within ParseXS.pm, so remove it.
$FH did not play well with $self inside angle-bracket operator
(while loop). $Package could not be transformed either, probably
due to problems in its usage as ${Package} inside HEREdocs.

Declare global instance as a stepping stone to an OO ExtUtils::ParseXS

Declare 'our' variable $self and move the first package global ($ProtoUsed) into it.

More and better tests

Provide messages for some tests in 001-basic.t.  Add file
t/004-nolinenumbers.t to test case where line numbers are not
requested.

Eliminate sub ProtoString(), which was used in only one location.

Move list of XS keywords to Constants.pm.

Remove the wistful comment about "use strict -- one of these days ..."

That day has come!

Change name of sub errors() to sub report_error_count()

... to distinguish it better from 'our' variable $errors.
Adjust t/002-more.t and lib/ExtUtils/xsubpp accordingly.

Indentation fix

Change arguments to output_init() to single hashref

Also move definitions of some utility subs to end of file.

Since we don't yet have a way around the 'eval EXPR' problem.  We'll have to
de-reference that hashref once inside the sub.

Change arguments to generate_init() to single hashref.

Since we don't yet have a way around the 'eval EXPR' problem.  We'll have to
de-reference that hashref once inside the sub.

Inline comment about difficulties which 'eval qq()' will pose

Typographical cleanup only.

Improve test coverage, extract function for better testing

Add files to test EU::PXS::Utilities::map_type(), valid_proto_string(),
process_typemaps().  Extract code for processing a single typemap file
into its own sub, thereby permitting more focused testing.  Eliminate some
unreachable branches and conditions in process_typemaps().

On the basis of coverage analysis, eliminate unreachable code.
Reformat some code for readability (shorter line length).

Explicitly pass $hiertype to map_type()

Also adjust all (3) calls to map_type() to suit. We can do this
because $hiertype is read, but not written to, within map_type().
map_type() is now fully encapsulated and can be extracted from the
main package and placed in EU::PXS::Utilities.

Eliminate last 'local' variables.

In two cases, guarantee that generate_init() is explicitly passed
all 4 arguments. With this commit, ExtUtils::ParseXS for the first
time ever runs completely under 'use strict;'.

More variable scope fixes

Eliminate last instances of $name. Eliminate the debugging
messages I was using to detect $name and its effects. Begin to
replace the 'local' variables in 3 subroutines, as these are
the final remaining obstacles to use strict.

Refine debugging messages to only identify CRITICAL messages.

STDERR debugging messages around the last suspected '' global variables.

Rename $name_printed to $printed_name

... so that it's better distinguished from instances of $name.

Rename two more instances of '$name'.

Reclassify approx 7 variables as either 'our' or 'my'.

Thanks to dagolden for diagnosis of problem with 'our' variables.

Add a little POD

More variable renaming and comments

1. Rename another instance of $name to $iface_name to further reduce
confusion.
2. Write inline comments at two locations where $name appears to be
previously undeclared global.
3. Correct some indentation.

More variable and code shuffling to improve maintainability

1. Move creation of read filehandle closer to its first point of use.

2. Move code for population of %targetable into subroutine and place that sub,
make_targetable, in EU::PXS::Utilities (for the time being).

3. Rename one instance of $name to $len_name to (start to) reduce confusion.

Note: Munged by Steffen to fix problem with order of chdir() and open()
calls.

Move code handling processing of typemap files to Utilities.pm.

Change $ALIAS from 'my' to 'our'

This is not because I think it really should be changed but because
declaring it as 'my' produces change in resulting C code -- albeit
probably a desirable change. Include summary of debugging as inline
comment.

Change ValidProtoString() to valid_proto_string()

and move its definition to Utilities.pm

Revise some of the placements into EU::PXS::Constants.

Shuffle more functions and variables around

Move sub C_string() to Utilities.pm. Eliminate some 'my' variables
inside process_file() that can be handled equally well by %args elements.

Move EU::ParseXS::CountLines to its own file to simplify editing

Implement use strict 'subs' and 'refs'

With two understandable exceptions for use strict 'refs'.

Correct ambiguous regex capture syntax.

Tighten scope of more variables

Move $pname from global to 'our'. Move $clean_func_name from global
to 'my'. Declare our $bal in separate statement.

Move about 9 global variables into 'our' status.

Lexicalize more variables

Move 6 more variables out of 'our' status and into 'my' status
inside process_file().

Continue renaming confusing variables

Remove an unused variable called $line_no (which should reduce
confusion with @line_no). Rename another $line_no within
Warn() to $warn_line_number. %targetable becomes 'my' variable
inside subroutine.

EU::ParseXS Version bump

Also nuke one more unused variable and fix a typo.

Start to remove unused variables

Also identify 'our' variables which are used only within the scope of
process_file() and which therefore can be changed into 'my' variables.

Move TidyType() to Utilities::tidy_type()

Create test file for it.

Change TrimWhitespace() to trim_whitespace()

and place it in ParseXS::Utilities.  Write a file to test it.
Update MANIFEST.

Move POD for internal sub from ParseXS.pod to Utilities.pm

Create file to test standard_typemap_locations().
Update MANIFEST.

Separate POD file and ExtUtils::ParseXS::Utilities

Create separate file to hold POD and introduce
ExtUtils::ParseXS::Utilities to hold non-object-oriented functions
refactored out of ExtUtils::ParseXS; first instance:
standard_typemap_locations().

Continue tidying up the code, range lines 1000-end

Uncuddle all '} elsif' and '} else' statements.
Switch to 'use Exporter' rather than 'require' (for consistency).

Change from 'use vars' to 'our', which was introduced in Perl 5.6.0.

Begin to uncuddle 'elsif' and 'else'. Rationalize indentation. Completed thru line 1000.

Eliminate whitespace before final semicolon.

Change all hard tabs to 4-spaces. Eliminate trailing whitespace.

Rename EU::PXS test files to include digits

Revert "Moved prototype check to XS code, allowing the .pm file change from yesterday"

This reverts commit d16d4b10300b35e2d4d858a3e23cff975bc713fe. It
requires the _ prototype support in ExtUtils::ParseXS and we need to
temporarily disable that. Will be reapplied later.

Revert EU::ParseXS part of 1e8125c621275d18c74bc8dae3bfc3c03929fe1e

This is in order to be able to apply Jim Keenan's work on
ExtUtils::ParseXS. This change will be reapplied later.

Revert the EU::ParseXS part of c4a6f826b3676d1fdbd9972a9d0a8f11c02d003d

This is in order to apply Jim Keenan's work to EU::PXS. It is planned to
re-apply this change later.

Revert EU::ParseXS part of c33e8be1506a75e393304af89d64e3f46e0ca7cb

This is necessary in order to apply Jim Keenan's work on
ExtUtils::ParseXS. The plan is to reapply this change later.

... also bump version.

Add an email alias for James Keenan

Update CPANPLUS to CPAN version 0.9107

  [DELTA]

  Changes for 0.9107      Tue Jul 12 15:22:14 2011
  ================================================
  * Add support for v-strings and x.y.z versions, using the version module.
  * Resolve [rt.cpan.org #69420] "type checking for CPANPLUS::Backend::RV::new is incorrect"
  * Resolve [rt.cpan.org #69203] "custom sources ignores timeout config option"

I, for one, welcome our new DTrace overlord

regenerate known_pod_issues

Document and test the phase-change probe

Load modules only after setting up @INC

Make Porting/makerel create smaller archives with 7z and advdef

It automatically will use these to create archives ~5% smaller,
otherwise falls back to gzip and bzip2.

Also, document in Porting/release_managers_guide.pod.

buildtoc, known_pod_issues

This just does the general bookkeeping necessary when a new pod
(like the one in the previous commit) is added.

New document for perldtrace

New test file that exercises Perl's DTrace support

Add a phase-change DTrace probe

Factor out a PERL_SET_PHASE macro

This is the first step in adding a dtrace probe for global phase change

A not X for was_lvalue_sub in embed.fnc.

7b70e8177801df4e142684870ce037d584f72e7b was my (wrong) suggestion,
and it made the symbol only visible when PERL_CORE was defined,
which it isn't in List::Util.

Storable: Restore 5.6 compatibility

Storable: Restore 5.8.2 compatibility

Fix a Storable test to work in 5.8.9-

This test only passes in 5.10.0+ because of a nasty bug in Hash::Util,
or perhaps Internals::SvREADONLY.

lock_hash is supposed to lock values as well as keys, but it doesn’t
always in 5.10.0+, leading me to believe that it was the right func-
tion to use.  (Fixing that is for another commit.)

use a flag to signal a stacking filetests instead of peeking at the next op.

Preparation for the codegeneration changes where the next op isn't accessible.

Add tests for wrapping formats in B objects

In sv.c, remove comments with structure sizes from bodies_by_type[].

On balance, the "typical" sizes seem to be more of a liability than an asset,
as they (by necessity) can only be correct for one particular combination of
UV size, pointer size, NV alignment, etc, which means more likely than not
they *aren't* correct for the platform you're building on. And that's for the
best case scenario, that the comments are updated if the structures are
updated.

Export Perl_was_lvalue_sub

As Craig Berry pointed out, the multicall API uses the PUSHSUB macro,
so anything it calls must be exported.

Add support for Unicode's Script_Extension property

This property is an improved version of Script.

mktables: Suppress certain expected debug msgs

mktables has a mode for quicker debugging to skip processing some
input files.  When this is done, it is to be expected that the output
will be incomplete, so don't bother warning in this case.

Unicode::UCD: nits in comments and pod

perlunicode: nits

perlrecharclass: nit

mktables: Move test to added class member

Early Unicode releases didn't specify all possible property values,
but newer ones do, with some exceptions.  As an error check, on a newer
relese, a warning is raised if an unexpected property value is raised.
This member allows properties to be handled in new releases that
Unicode doesn't list the property values for.  (Each property value
corresponds to a mapping.)  The previous test that had a hard-coded
value is amended to use the member's value instead.

mktables: New method on properties to delete tables

This method will be used in a future commit.

perlgit: C<<...>> -> C<< ... >>

Squelch warning in deparse.t

B::Deparse misdeparses qq(@,)

This commit changes the signature of stash_variable() in B::Deparse.
stash_variable() takes $cx value additionally to know whether or not
it is in the interpolation context.

Punctuation arrays such as @* are usually interpolated only if the
name is enclosed in braces @{*}, but the arrays @_, @+, @- are
interpolated even without braces.

related issue: #93990

perlref: update bareword bracketing discussion, mention new auto-dereferencing

Update Pod-Simple to CPAN version 3.17

  [DELTA]

  2011-07-09   David E. Wheeler <david@justatheory.org>
  * Release 3.17

  Documented tertiary methods. Patch from Shawn H Corey.

  Added "backlink" option to Pod::Simple::XHTML. Thanks to Marc
  Green/Google Summer of Code for the pull request.

  Typos fixed in Pod::Simple::HTMLBatch. Reported by Shawn H Corey.

  Fixed quoting of value returned by a "strip_verbatim_indent()"
  code reference so that regex meta characters are properly escaped.

  Added "anchor_items" option to Pod::Simple::XHMTL. This allows
  text items (which are output as <dt> elements) to have IDs that
  can be referenced in the "#" part of a URL. Marc Green/Google
  Summer of Code.

  Added "recurse" option to Pod::Simple::Search. It's enabled by
  default; disable it to turn off recursion into subdirectories.
  Marc Green/Google Summer of Code.

  Added documentation to clarify the behavior of the "content_seen"
  method. Thanks to Olaf Alders for the pull request.

Update perldoc.pod with URL support

Add crude support to perldoc for viewing PODs via HTTP(s)

Propagate (non-)lvalue context through nested calls

Before this commit, this code would fail:

$foo = "foo";
sub foo :lvalue{ return index "foo","o" }
sub bar :lvalue { foo }
$x = bar;

(It would fail for ‘return $]’ as well.  Whether it’s a PADTMP or a
read-only scalar makes no difference.)

foo would think it was being called in true lvalue context, because
the entersub op that called it (in bar) was marked that way, bar being
an lvalue sub as well.

The PUSHSUB macro in cop.h needed to be modified to account for
dynamic, or indetermine, context (i.e., indeterminable at compile
time).  This happens when an entersub op is an argument to return or
the last statement in a subroutine.  In those cases it has to propa-
gate the context from the caller.

So what we now do is this: Both lvalue and in-args flags are turned on
for an entersub op when op_lvalue is called with OP_LEAVESUBLV as the
type.  Then PUSHSUB copies into the context stack only those flags
that are set both on the current entersub op and in the context stack
for the previous sub call.

perlrecharclass: slight reword

S_return_lvalues: collapse duplicated code

S_return_lvalues: microöptimisation

This jumps over an if() condition when we know it’s going to be true.

Free temps on recursive scalar lvalue sub exit

See the thread starting at
<83877DD1-4624-4497-B784-0F8DA25DB4A8@cpan.org>.

I still don’t know why this does not apply to list context.

As with most lvalue sub fixes, this involves deleting code.