>
> putting some trace statements in there, there's only 3 places where
> return_type is actually used:
> using Apache::Filter for ap_add_input_filter
> using Apache::RequestRec for ap_read_request
> using APR::IpSubnet for apr_ipsubnet_create
>
> i suppose we could probably get by without it, i'll work on getting rid of
> that line.
>
I think we could easily remove this line and put the package name for these
three functions in the map file. That would make much more sense to me. If
you agree to that, I will do it and you don't have to take care about it.
There is another thing that should going into the map files:
* TypeMap.pm:
#anything needed that mod_perl.h does not already include
#XXX: .maps should INCLUDE= these
my @includes = qw{
apr_uuid.h
apr_sha1.h
apr_md5.h
apr_base64.h
apr_getopt.h
apr_hash.h
apr_lib.h
apr_general.h
apr_signal.h
util_script.h
};
Your comment suggest it already...
* TypeMap.pm:
my %convert_alias = (
Apache__RequestRec => 'r',
Apache__Server => 'server',
Apache__Connection => 'connection',
APR__Table => 'table',
APR__UUID => 'uuid',
apr_status_t => 'status',
);
* WrapXS.pm
my %typemap = (
'Apache::RequestRec' => 'T_APACHEOBJ',
'apr_time_t' => 'T_APR_TIME',
'APR::Table' => 'T_HASHOBJ',
);
These two things would need a extention of the xxx_types.map files. I would
suggest simply append two addtional (optional) parameters for each line. The
first holds the name used in the typemap file (%typemap) and the second the
alias for the sv_convert.h file (%convert)
What do you think ?
Gerald
-------------------------------------------------------------
Gerald Richter ecos electronic communication services gmbh
Internetconnect * Webserver/-design/-datenbanken * Consulting
Post: Tulpenstrasse 5 D-55276 Dienheim b. Mainz
E-Mail: [EMAIL PROTECTED] Voice: +49 6133 925131
WWW: http://www.ecos.de Fax: +49 6133 925152
-------------------------------------------------------------
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
