Hi Tobias, On 2015-10-06 14:15, Tobias Klauser wrote: > prented_sha1_file() always returns 0 and its only callsite in > builtin/blame.c doesn't use the return value, so change the return type > to void. While this commit message is technically correct, it would appear that there are some things left unsaid. Is there a problem with the current code that is solved by not returning 0? If so, could you add it to the commit message? And in particular, change the oneline appropriately? Ciao, Johannes -- 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