Christian Couder <christian.couder@xxxxxxxxx> writes: > Unfortunately strbuf_add() calls strbuf_grow() itself which is not > needed as we have already called strbuf_grow() to avoid repeated > reallocation. Why is it unfortunate? If the current allocation is sufficient, it is a cheap no-op, isn't it (if not, we should make it so)?