On Thu, Sep 07, 2023 at 03:59:31AM -0400, Jeff King wrote: > There are really two "factor outs" here: we pull the code from > cmd_repack() into a helper, and then the helper is also just a thin > wrapper around its "_1" variant. That latter part isn't needed yet, but > I can guess from your description that we'll eventually have the main > function dispatch to the "_1" helper for lists. Yeah... the "_1" variant looks ugly in isolation in this patch, but I think makes things cleaner in subsequent patches. Thanks, Taylor
