On 06/03/2015 11:20 PM, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > >> On Wed, Jun 03, 2015 at 11:51:43AM -0700, Junio C Hamano wrote: >> >>> Jeff King <peff@xxxxxxxx> writes: >>> >>>> On Wed, Jun 03, 2015 at 03:51:59PM +0200, Michael Haggerty wrote: >>>> >>>>> NULL_SHA1 is used to indicate of "invalid SHA-1" throughout our code >>>> >>>> s/of/an/ ? >>> >>> Also possibly s/invalid SHA-1/invalid ref/? >> >> I thought it was trying to express that we use the null sha1 as a >> sentinel value throughout the code, no matter if the value came from a >> ref or otherwise. > > Yeah, an invalid object name, not limited to refs, is correct. > > Thanks. Thanks for all the comments. Taking them into consideration, I suggest changing the last commit message to --8<----------------------------------------------------------------- read_loose_refs(): treat NULL_SHA1 loose references as broken NULL_SHA1 is used to indicate an "invalid object name" throughout our code (and the code of other git implementations), so it is vastly more likely that an on-disk reference was set to this value due to a software bug than that NULL_SHA1 is the legitimate SHA-1 of an actual object. Therefore, if a loose reference has the value NULL_SHA1, consider it to be broken. Signed-off-by: Michael Haggerty <mhagger@xxxxxxxxxxxx> --8<----------------------------------------------------------------- Since the only comments were about this one commit message, I won't send an updated patch series to the mailing list unless you request it. You can also get the patches from my GitHub account [1], branch "for-each-ref-errors". Michael [1] https://github.com/mhagger/git -- Michael Haggerty mhagger@xxxxxxxxxxxx -- 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