On Tue, Mar 9, 2021 at 2:51 PM Derrick Stolee <stolee@xxxxxxxxx> wrote: > > On 3/8/2021 7:09 PM, Elijah Newren via GitGitGadget wrote:> + goto simple_cleanup; /* collect_renames() handles some of cleanup */ > > + > > +cleanup: > > + /* > > + * Free now unneeded filepairs, which would have been handled > > + * in collect_renames() normally but we're about to skip that > > + * code... > > + */ > > "but we're about to skip that code" > > Haven't you skipped it already, earlier in the above part? This should > instead say "but we skipped that code", right? Oh, indeed. I wonder if I moved that comment around and didn't update it. Anyway yes, I'll fix it.
