Mostly adding tests and documentation.

Rename sub print_preprocessor_statements() to
analyze_preprocessor_statements().  Modify interface to map_type() and
re-work tests as needed.  Wrote documentation in Utilities.pm for
process_single_typemap(), map_type(), standard_XS_defs(),
assign_func_args(), analyze_preprocessor_statements().  Write tests
in t/109-standard_XS_defs.t, t/110-assign_func_args.t.

Move several functions to Utilities.pm

Namely: check_conditional_preprocessor_statements(), Warn(),
blurt() and death() to Utilities.pm.

Add file to test blurt, death and Warn.  (death() is not
really testable yet due to hard-coded 'exit'.)  ovid++, rjbs++,
xdg+++ for assistance in developing tests."

Change check_cpp() to check_conditional_preprocessor_statements()

... to be more self-documenting.

Also: Explicitly pass $self to Warn(), blurt() and death().
Add skeletons of files to test functions moved (or planned to be
moved) to Utilities.pm.

Was able to move $Packid into $self.

Note that I got build failures when I attempted to pull $Package into $self.

The distributions below because during various 'eval EXPR'
statements, I got a 'Global variable $Package ...' error.

cut -d '/' -f 7,8,9,10,11 research/problematic.builds.20100410.txt
D/DB/DBURDICK/BoostGraph/Boost-Graph-1.4.tar.gz
D/DE/DEEMON/Crypt-OpenSSL-SMIME-0.02.tar.gz
D/DL/DLOWE/Text-Tmpl-0.33.tar.gz
D/DM/DMAKI/Data-Valve-0.00010.tar.gz
D/DO/DORMANDO/Perlbal-XS-HTTPHeaders-0.20.tar.gz
E/ED/EDPRATOMO/Algorithm-Permute-0.12.tar.gz
J/JH/JHI/String-Approx-3.26.tar.gz
K/KA/KARMAN/Search-Tools-0.45.tar.gz
K/KI/KILINRAX/HTML-Strip-1.06.tar.gz
L/LA/LAIRDM/Algorithm-SVM-0.13.tar.gz
M/MA/MARKSMITH/HTTP-HeaderParser-XS-0.20.tar.gz
M/MC/MCEGLOWS/Search-ContextGraph-0.15.tar.gz
M/MH/MHX/Convert-Binary-C-0.74.tar.gz
M/MH/MHX/Tie-Hash-Indexed-0.05.tar.gz
M/MI/MIKEWONG/Table-ParentChild-0.05.tar.gz
M/MS/MSERGEANT/CDB_File-0.96.tar.gz
M/MS/MSERGEANT/IO-KQueue-0.34.tar.gz
P/PA/PAJAS/XML-LibXML-1.70.tar.gz
P/PM/PMQS/Filter-1.37.tar.gz
S/SA/SALVA/Compress-PPMd-0.11.tar.gz
S/SA/SAPER/Net-Pcap-0.16.tar.gz
S/SA/SAPER/Net-RawIP-0.25.tar.gz
S/SG/SGOELDNER/MonetDB-CLI-MapiXS-0.03.tar.gz
S/SM/SMUELLER/Math-SymbolicX-FastEvaluator-0.01.tar.gz
S/SM/SMUELLER/Parse-ExuberantCTags-1.01.tar.gz
S/SP/SPIDB/Net-ext-1.011.tar.gz
T/TJ/TJMATHER/Crypt-OpenSSL-DSA-0.13.tar.gz

Refactor some code into sub set_cond()

Eliminate instances of calling subs with '&'.

Add vim modeline entries.

Move sub print_preprocessor_statements() into Utilities.pm.

Refactor some code into sub assign_func_args().

Don't have to undef if it's never been assigned to.

Move several variables found inside the PARAGRAPH loop

... closer to the point where each is first used.

Move $orig_args closer to where it's used

Move $prepush_done closer to where it's used

Move $func_header inside while loop closer to where it's used

Move two heredocs into Utilities::standard_XS_defs().

Change @outlist and %only_C_inlist into corresponding references

... in anticipation of upcoming refactorings,

Add messages to 3 tests which lacked them.

Move 4 elements in %args into $self

It was not yet possible to move $args{'s'} into $self because
of a quotemeta problem.

No evident need for 'for' loop in assignment from $args{filename}.

Explicitly pass $self to check_cpp()

Also explicitly return $self from print_preprocessor_statements().

Refactor code inside process_file() into print_preprocessor_statements()

Change \@BootCode to $BootCode_ref

to facilitate more refactoring of code into (somewhat) encapsulated
subs.

Eliminate some synthetic variables

and move declarations of some lexicals closer to the point where
they are actually used.

Typographical cleanup

Mostly elimination of trailing whitespace.

Add $newXS to $self

Keeping fingers crossed!

Move $proto into $self;

Add $Module_cname to $self.

Move %defaults to $self.

Move @line to $self.

Move @line_no, @Attributes, %outargs, @XSStack to $self.

Move %lengthof, %XsubAliases, %XsubAliasValues, %Interfaces.

To do: Eliminate confusion between these attributes: interface
(scalar) interfaces (scalar) Interfaces (hash)

Add \%in_out and \%argtype_seen to $self.

Move 9 'our' hashes and arrays into $self

For now, bypassing \@line, \%defaults.

Move $var_num into $self.

Write inline comment explaining current status of 'our' variables.

Move $Interfaces into $self, but as $self->{interfaces}

... to avoid confusion with %Interfaces. (But note we already have
$self->{interface}. This will have to be fixed.)

Move $DoSetMagic, $XsubAliases, into $self.

The latter is called 'xsubaliases', the change of casing
anticipating confusion with %XsubAliases.

Tidy up lists of 'our' variables.

Move $condnum, $cond, into $self. $ALIAS ran into eval EXPR problems.

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"