Hi,

thanks for digging into this.  Overall, the patch makes sense, I'm just
wondering about this:

On Sun, May 22, 2016 at 12:46:27AM -0400, Selva Nair wrote:
> Other changes:
> - Use minimal initial allocation size so that the realloc code path
>   gets exercised (2 or more connections will cause realloc).

I can see that this is useful for testing, but do we want this in production?

It won't make a difference for most usages ("only one connection active"),
but then, the extra memory use for 10 is minimal, and realloc() *can* fail,
so just keeping at 10 won't need the realloc() path generally...

Dunno, just thinking loud.

gert

-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Attachment: signature.asc
Description: PGP signature

Reply via email to