On Fri, 5 Oct 2007, Pierre Habouzit wrote: > > - strbuf_grow(buf, len); > + /* only grow if not in place */ > + if (strbuf_avail(buf) + buf->len < len) > + strbuf_grow(buf, len - buf->len); Umm. This is really ugly. The whole point of strbuf's was that you shouldn't be doing your own allocation decisions etc. So why do it? Wouldn't it be much better to have a strbuf_make_room() interface that just guarantees that there is enough room fo "len"? Otherwise, code like the above would seem to make the whole point of a safer string interface rather pointless. The above code only makes sense if you know how the strbuf's are internally done, so it should not exists except as internal strbuf code. No? Linus - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
