Add method to generate the prototype lookup hash

Akin to the methods in the parent commit that generate the lookup hash
for TYPEMAP/INPUT/OUTPUT, this method will generate the lookup has for
typemap's prototypes as required by ExtUtils::ParseXS at this time.

Methods to get typemaps in ParseXS compatible format

Essentially, these extra methods export the typemap information in the
format expected by ExtUtils::ParseXS at this time. Down the road, we may
want EU::PXS to simply pass around an ExtUtils::Typemaps object.

Reduce code duplication by using the same prototype regexp

Teach EU::Typemaps about the _ prototype

I am in a hurry and this makes things correct, but really, this should
maybe use EU::ParseXS' regular expression.

Bump ExtUtils::Typemaps version

Just to make sure there really will never be a confusion or
collision with the similarly named module that has been living
on CPAN.

Simplify "remove" logic after previous refactoring

Hash lookup for output sections

More O(1) for everyone.
Also fixes a tiny issue with an error message from the inputmap
conversion.

Convert input sections to hash lookup

Die, O(n), die!

Start refactoring EU::Typemaps

Use a hash lookup for avoiding collisions.

Remove note about being untested

This code is actually in production use.

update copyright

Add a trailing 's' to the name of EU::Typemap

This is to prevent collisions on case-insensitive file systems with
lib/typemap.

Extract typemap-related code from ExtUtils::ParseXS

This was available from CPAN a while ago. The plan is to use it to make
the typemap parsing and manipulation saner and rip the parser from
ExtUtils::ParseXS' dead claws.

Cleanup of make_targetable

Fix function name in documentation

Remove unnecessary Data::Dumper loading

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

This reverts commit cf8fc7eed929846d31d02d565b4d5f4480a18d69,
reinstating the move of the prototype check to XS code.

Start documenting the EU::ParseXS changes

Documentation: maintainers & copyright

Adding Jim Keenan and myself to the list of maintainers. Sorry, Jim!
Update copyright and prettify a note.

Documentation fix

Be more precise about the return values of process_typemaps().

Rename some ExtUtils::ParseXS 'constants'

Make them have properly descriptive, camel case names.

Reapply the patch that tought EU::ParseXS about _ protos

Even including a test!

Ignore new ExtUtils::ParseXS files in lib

Re-apply ExtUtils::ParseXS portion of 1e8125c62127

Which had to be reverted as 634b52b92f42a6bb93c549
for a big rebase.

Re-apply typo fixes from c4a6f826b3676d1fdbd99

These had to be reverted for the ExtUtils::ParseXS rebase of Jim
Keenan's strictification work.

Originally reverted as 8e19e0c1f155990dc120199.

Re-apply EU::ParseXS portion of c33e8be1506a75e

This is the unused-arg stuff in tests.
Originally reverted as 836ae07757f3a1a32212.

A bit more documentation of make_targetable().

Add documentation on the hashrefs produced by process_typemaps().

Add documentation and additional tests

... exploring relationship between typemaps and \%targetable.

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.