On Wed, Feb 17, 2016 at 5:19 PM, Junio C Hamano <[email protected]> wrote: > Jeff King <[email protected]> writes: >> On Wed, Feb 17, 2016 at 05:11:50PM -0500, Eric Sunshine wrote: >>> On Wed, Feb 17, 2016 at 1:06 PM, Karthik Nayak <[email protected]> >>> wrote: >>> > From: Jeff King <[email protected]> >>> > >>> > We don't do any post-processing on the resulting strbufs, so it is >>> > simpler to just use string_list_split, which takes care of removing >>> > the delimiter for us. >>> > >>> > Written-by: Jeff King <[email protected]> >>> >>> Perhaps Peff can give his sign-off... >> >> Ah, right. I usually sign-off when sending to the list, so the version >> he pulled from GitHub didn't have it. >> >> Definitely: >> >> Signed-off-by: Jeff King <[email protected]> >> >> And I don't think "Written-by" was doing much here, anyway; I am already >> the author by the From header at the top. :) > > ;-). > > So, is everybody happy with this round?
Yes, v6 looks good to me and is: Reviewed-by: Eric Sunshine <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
