Junio C Hamano <gitster@xxxxxxxxx> writes: > Is this sufficient? > If you are going to do something in 2/2 that relies on the format of > this line being correct (as opposed to "noop" or "#" that can have > any garbage on the remainder of the line), wouldn't you want to at > least check $sha1 is sensible? That's also something that I was wondering, I wrote about it in the 0/2 part of this patch, I wanted some opinion about it. If there is no opposition on the subject, I will have it ready for the v3 of the patch. Quote of that part of the 0/2 for more clarity: Galan Rémi <remi.galan-alfonso@xxxxxxxxxxxxxxxxxxxxxxx> writes: > For the 'drop' command, maybe instead of just doing the same thing as > noop, checking if the SHA-1 that supposedly follow does exist could be > a good idea. -- 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