On 5/16/2022 4:36 PM, Victoria Dye wrote: > Derrick Stolee via GitGitGadget wrote: >> -void ensure_full_index(struct index_state *istate) >> +void expand_to_pattern_list(struct index_state *istate, >> + struct pattern_list *pl) > > Hyper-nit: I don't think this is aligned (it's probably not worth fixing > unless you end up rerolling for something else). You're right. When I saw this I thought it was just the "off by one" issue when patches have a "+" in the beginning, but it's actually the backwards order. Thanks for catching it. -Stolee