Re: [PATCH v5] use starts_with() instead of !memcmp()
On Fri, Mar 21, 2014 at 12:53 PM, Junio C Hamano wrote: > Eric Sunshine writes: > >> On Tue, Mar 18, 2014 at 9:18 PM, Quint Guvernator >> wrote: >>> Another version, this time very in line with the review and commentary of >>> Junio, Eric, and Michael. This version boasts a revamped commit message and >>> fewer but surer hunks changed. >> >> Explaining what changed in this version is indeed a courtesy to >> reviewers. Thanks. > > So, is that a "reviewed-by: Eric"? No, sorry. You and Peff were actively reviewing Quint's submissions, so I merely scanned them quickly without making a careful examination. I've been commenting upon so many GSoC submissions that it's hard to remember which is which, and upon reading "Another version, this time very in line with the review and commentary Junio, Eric, and Michael", I had to search the list archive to figure out why my name was listed. Hence, the suggestion from me that providing a link to the previous attempt is a welcome courtesy. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5] use starts_with() instead of !memcmp()
Eric Sunshine writes: > On Tue, Mar 18, 2014 at 9:18 PM, Quint Guvernator > wrote: >> Another version, this time very in line with the review and commentary of >> Junio, Eric, and Michael. This version boasts a revamped commit message and >> fewer but surer hunks changed. > > Explaining what changed in this version is indeed a courtesy to > reviewers. Thanks. So, is that a "reviewed-by: Eric"? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5] use starts_with() instead of !memcmp()
On Tue, Mar 18, 2014 at 9:18 PM, Quint Guvernator wrote: > Another version, this time very in line with the review and commentary of > Junio, Eric, and Michael. This version boasts a revamped commit message and > fewer but surer hunks changed. Explaining what changed in this version is indeed a courtesy to reviewers. Thanks. For bonus points, provide a link to the previous attempt, like this [1]. [1]: http://thread.gmane.org/gmane.comp.version-control.git/244292 > Thanks again for the guidance. > > Quint Guvernator (1): > use starts_with() instead of !memcmp() > > builtin/apply.c| 4 ++-- > builtin/for-each-ref.c | 2 +- > builtin/mktag.c| 2 +- > builtin/patch-id.c | 10 +- > connect.c | 4 ++-- > imap-send.c| 6 +++--- > remote.c | 2 +- > 7 files changed, 15 insertions(+), 15 deletions(-) > > -- > 1.9.0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html