On Tue, Dec 15, 2015 at 03:44:42PM -0800, Junio C Hamano wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > > > There already was strbuf_getline_crlf(), and I wanted a new name to > > be conservative. > > When I re-read the series, I realize that the existing one had > exactly the same semantics as strbuf_gets(), so I think no risk > would come from reusing that name. Let me try redoing the series > when I find time ;-) Heh. I just made up that name, not realizing it existed. Looks like it is fairly recent as part of builtin-am, which explains why I hadn't seen it yet. I agree it looks like it does what you want. Upon reading your first message I thought "eh, did I totally misunderstand the intent of the series?", but it looks like we are on the same page. :) -Peff -- 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