Thread (5 messages) 5 messages, 2 authors, 2016-06-15

Re: [PATCH] pretend_sha1_file(): Change return type from int to void

flat view

From: Johannes Schindelin <hidden>
Date: 2016-06-15 23:06:46

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help