Re: APR is inconsistent about pool behavior

> When we first talked about pools, that was the design that we agreed on:
> abort() when an alloc fails. svn_pool_create() should not take an argument,
> but should just use default_abort all the time. (for both client and server
> portions)

Yeah. I think this is the third time I've posted something about it.
For some reason this one won't stick in my head...

> As you point out: APR is not built to use any behavior other than "abort on
> alloc failure" (which I believe I pointed out back then, too). As a result,
> we pretty much *have* to use the abort design. However, that isn't bad at
> all... it works very well for us, too.

No, that's fine. But if all APR users want that behavior, wouldn't it
be nicer for it to be the default in APR? At the moment, you have to
take special pains to get that behavior; I'd think one would rather
have an abort function set in new pools by default, and provide a call
to override it, if the caller is willing to take the responsibility.
Received on Sat Oct 21 14:36:10 2006