search

Tarsnap / tarsnap

Command-line client code for Tarsnap.
https://tarsnap.com
Other
864 stars 60 forks source link

statstape_printlist(): refactor #577

Closed gperciva closed 11 months ago

gperciva commented 1 year ago

I think I'd like to rebase #576 on top of this refactoring.

Adding a separate statstape_printlist_item() is overkill for merely allowing --list-archives --null, but there's some future work that would be greatly simplified by this. (See upcoming private email in an hour.)

If you like that plan and we're going to have statstape_printlist_item() eventually, I think the git history will make more sense to add it sooner rather than later.

gperciva commented 1 year ago

This is prep for #576; once this refactoring is done, it's trivial to allow --list-archives --null.

You probably won't want to merge this by itself, preferring to do #576 all at once. But I figured there was no harm in separating the refactoring from the new feature, just in case you wanted to merge this one first.