Junio C Hamano, Thu, Mar 19, 2009 22:55:56 +0100: > Alex Riesen <raa.lkml@xxxxxxxxx> writes: > > @@ -91,6 +90,7 @@ static int show_tree(const unsigned char *sha1, const char *base, int baselen, > > if (!(ls_options & LS_NAME_ONLY)) { > > if (ls_options & LS_SHOW_SIZE) { > > if (!strcmp(type, blob_type)) { > > + unsigned long size = 0; > > sha1_object_info(sha1, &size); > > printf("%06o %s %s %7lu\t", mode, type, > > abbrev ? find_unique_abbrev(sha1, abbrev) > > Hmm, shouldn't you be checking the return value from sha1_object_info() > and skipping the printf() altogether instead? But then I cannot know the name of the failed tree entry. -- 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