On Tue, Sep 05, 2023 at 04:36:46PM -0400, Taylor Blau wrote: > To reduce the complexity of the already quite-long `cmd_repack()` > implementation, extract out the parts responsible for deleting redundant > packs from a geometric repack out into its own sub-routine. Makes sense. Again, I'm happy to see some of these large functional pieces of cmd_repack() moved into sub-functions. Even if we never call them twice, IMHO it is a readability improvement to give them meaningful names. -Peff
