Stas Bekman wrote:
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


Gone!

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