On 12/16/2020 5:27 PM, Elijah Newren via GitGitGadget wrote: > +struct commit_list *reverse_commit_list(struct commit_list *list) > +{ > + struct commit_list *next = NULL, *current, *backup; > + for (current = list; current; current = backup) { > + backup = current->next; > + current->next = next; > + next = current; > + } > + return next; > +} > + I followed the discussion about the variable names in the earlier thread, and I certainly did not meant to stir a controversy. I think this approach of deduplicating but keeping an exact copy is the best solution. Thanks, -Stolee