Stefan Beller <sbeller@xxxxxxxxxx> writes: > 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. ;) That is not a good excuse when it comes to the API design to deal with linked lists in a bog-standard way, though. Many callers of commit_list_insert() may ignore its return values, but even when the last caller stopped caring it, that does not necessarily mean the feature can be removed. -- 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