On 12/1/05, Greg Hudson <firstname.lastname@example.org> wrote:
> On Wed, 2005-11-30 at 17:48 -0600, email@example.com wrote:
> > Create a subpool to allocate SVN_STREAM_CHUNK_SIZE buffers. Just before
> > completing successfully, clean the subpool to free the 100kB buffer.
> I don't understand the need for this. If the function is being called
> in a loop, the loop should be using an iteration pool. If the function
> is not being called from a loop, 100K is noise.
This commit is a consequence of lundblad 'complaining' about the fact
that I left a buffer allocated in libsvn_subr/subst.c. That buffer was
about the same size. So, I looked where else the buffer was allocated
and disposed of, moving the allocation to a subpool if it wasn't
> If the functions return allocated information, that might make it worth
> doing large temporary allocations in a subpool. You didn't include any
> symbol names in your log message, so I can't easily tell whether you
> modified functions of that nature.
No, I didn't modify any of those.
Received on Thu Dec 1 00:10:41 2005