You are viewing a plain text version of this content. The canonical link for it is here.
Posted to modperl-cvs@perl.apache.org by ge...@apache.org on 2004/03/02 05:12:20 UTC
cvs commit: modperl-2.0/src/modules/perl modperl_config.c
geoff 2004/03/01 20:12:20
Modified: src/modules/perl modperl_config.c
Log:
clarify comments
Revision Changes Path
1.77 +7 -4 modperl-2.0/src/modules/perl/modperl_config.c
Index: modperl_config.c
===================================================================
RCS file: /home/cvs/modperl-2.0/src/modules/perl/modperl_config.c,v
retrieving revision 1.76
retrieving revision 1.77
diff -u -r1.76 -r1.77
--- modperl_config.c 14 Feb 2004 01:38:05 -0000 1.76
+++ modperl_config.c 2 Mar 2004 04:12:20 -0000 1.77
@@ -67,11 +67,14 @@
*
* unfortunately, there is no set of apr functions to do this for us -
* apr_compress_table would be ok, except it always merges mulit-valued
- * keys into one, regardless of the merge flag, which is no good - we
- * need separate entries, not a single comma-delimted entry.
+ * keys into one, regardless of the APR_OVERLAP_TABLES flag. that is,
+ * regardless of whether newer entries are set or merged into existing
+ * entries, the entire table is _always_ compressed. this is no good -
+ * we need separate entries for existing keys, not a single (compressed)
+ * entry.
*
- * fortunately, the logic here is simple - first, (re)set the base (parent)
- * table where a PerlSetVar entry exists in the child (add) configuration.
+ * fortunately, the logic here is simple. first, (re)set the base (parent)
+ * table where a PerlSetVar entry exists in the add (child) configuration.
* then, just overlay the PerlAddVar configuration into it.
*/