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. -- 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