Oops, I just saw this PR after I fixed this in #617.

I'm also supporting --long[=N] which man zstd says is required for --adapt to work.

@digint Is there anything I can do to get this merged?

Is there anything I can do to get this merged?

Are you also supporting --long[=N] which man zstd says is required for --adapt to work?

(See my PR for hints and maybe other improvements too?)

@HaleTom I believe --long is not actually required for --adapt; the wording on the man page is unfortunately just confusing- it is saying "you may combine this with --long". What it requires is non-single threaded mode; you can check this with cat /dev/zero | zstd --adapt --single-thread and see it spits back an error but cat /dev/zero | zstd --adapt -v > /dev/null works, as you can see via the -v.

This PR does not support --long though, as it's a separate feature. I'm fine with either PR being merged depending on if @digint is fine with --long or not.

I'm likewise fine with either PR being accepted.

I think that mine's (#617) neater because:

• I was able to make it neater based on your excellent existing work after I found it :)
• It allows for both --long (future proofing) and --adapt.

@digint Is there anything I can do to get this merged?

I really hope to get some time next week to work on btrbk, and especially also to look at this MR (and @HaleTom 's).
I'm afraid there's so much going on currently, I can't promise anything right now...