Hi,

does the attached patch make sense, or do I misunderstand something?

The two tests search for "val", but print about "key". This makes the
debug print indistinguishable from the tests just before, where a lookup
for "key" is done.

Peter
-- 
Allen Gewalten zum Trutz sich erhalten.
 
SUSE LINUX Products GmbH
Research & Development
Index: tables/apr_tables.c
===================================================================
--- tables/apr_tables.c	(revision 540097)
+++ tables/apr_tables.c	(working copy)
@@ -725,7 +725,7 @@
 	    abort();
 	}
 	if (!apr_pool_is_ancestor(apr_pool_find(val), t->a.pool)) {
-	    fprintf(stderr, "apr_table_mergen: key not in ancestor pool of t\n");
+	    fprintf(stderr, "apr_table_mergen: val not in ancestor pool of t\n");
 	    abort();
 	}
     }
@@ -794,7 +794,7 @@
 	    abort();
 	}
 	if (!apr_pool_is_ancestor(apr_pool_find(val), t->a.pool)) {
-	    fprintf(stderr, "apr_table_addn: key not in ancestor pool of t\n");
+	    fprintf(stderr, "apr_table_addn: val not in ancestor pool of t\n");
 	    abort();
 	}
     }

Attachment: pgpUVRZ3CGMeF.pgp
Description: PGP signature

Reply via email to