On Tue, Jun 2, 2015 at 2:17 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Stefan Beller <sbeller@xxxxxxxxxx> writes: > >> No function uses the return value of get_remote_heads, so we don't want >> to confuse readers by it. >> >> Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> >> --- > > This is somewhat a sad change, as the returned value is designed to > be useful if caller wants to continue appending to the list. But there are no callers since like 2005. ;) (I did not exactly track it down, but even the last caller went away rather fast) > > Does it really "confuse" readers enough that it hurts to have a > return value? > Probably no. But I think this is just carrying around cruft we could avoid? > > Now such a caller has to tangle the list (the variable it gave the > function as the fourth argument) itself to find its tail. So you're saying if someone in the future really wants to append to that list, they don't find out to just return it again but rather do an O(n) operation? -- 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