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
signature.asc
Description: PGP signature