Jeff King <peff@xxxxxxxx> writes: > On Wed, Sep 16, 2015 at 03:54:50PM -0700, Junio C Hamano wrote: > >> Jeff King <peff@xxxxxxxx> writes: >> >> >> Is this conversion correct? This seems to me that the caller wants >> >> to create an IMAP folder name immediately under the root hierarchy >> >> and wants to have the leading slash in the result. >> > >> > Ugh, you're right. This is the "other" style Eric mentioned earlier. >> > >> > This looks like the only one in the patch (there are many that did not >> > check buf.len at all, but if we assume they were not invoking undefined >> > behavior before, then they are fine under the new code). >> >> Yes, I should have said that earlier to save one roundtrip. >> >> Thanks for working on this. > > For my re-roll, I've just omitted changing that caller. I think we can > leave it as-is; it is not worth trying to introduce a new helper for the > one site. Yup, I think the decision is sensible. Thanks. -- 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