Philippe M. Chiasson wrote:
The following ap_module_t fields are currently writable, and I can't think of a
good reason why they should be. As I am trying to finish the documentation for
Apache::Module, I'd like to suggest making the following fields read-only:


version
minor_version
module_index
name
next
cmds

+1

Patch below.

Index: xs/maps/apache_structures.map
===================================================================
RCS file: /home/cvs/modperl-2.0/xs/maps/apache_structures.map,v
retrieving revision 1.43
diff -u -I$Id -r1.43 apache_structures.map
--- xs/maps/apache_structures.map 13 Aug 2004 04:42:37 -0000 1.43
+++ xs/maps/apache_structures.map 17 Aug 2004 19:51:22 -0000
@@ -131,19 +131,19 @@
</server_addr_rec>


 <module>
-   version
-   minor_version
-   module_index
-   name
+<  version
+<  minor_version
+<  module_index
+<  name
 !  dynamic_load_handle
-   next
+<  next
 >  magic
 -  rewrite_args
 >  create_dir_config
 >  merge_dir_config
 >  create_server_config
 >  merge_server_config
-   cmds
+<  cmds
 >  register_hooks
 </module>



--
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to