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

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>

--
--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Attachment: signature.asc
Description: OpenPGP digital signature



Reply via email to