On Tue, 20 Nov 2001, Stas Bekman wrote:
 
> exactly for the same reason that addn() should be used. 
> apr_table_overlay  uses setn() which will loose all pairs with the same 
> key but the last one added.
> 
> apr_table_overlay:
>   *  for (i = 0; i < barr->nelts; ++i) {
>   *      if (flags & APR_OVERLAP_TABLES_MERGE) {
>   *          apr_table_mergen(a, belt[i].key, belt[i].val);
>   *      }
>   *      else {
>   *          apr_table_setn(a, belt[i].key, belt[i].val);
>              ^^^^^^^^^^^^^^^
>   *      }
>   *  }

that is not apr_table_overlay(), that is apr_table_overlap().
and the current version of overlap uses addn().

i think we should be using apr_table_overlay() just as 1.x does, no?



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

Reply via email to