search

rust-av / ssimulacra2_bin

Binary interface for the ssimulacra2 Rust port
BSD 2-Clause "Simplified" License
40 stars 10 forks source link

Use clap for arg handling #2

Closed BuyMyMojo closed 1 year ago

BuyMyMojo commented 1 year ago

The main point of this PR was to add clap for args but I added a little more verbosity to the error handling and changed how it prints the final result too.

If you have any perfences for clap implamentation just ask! :)

shssoichiro commented 1 year ago

I intentionally avoided including clap in order to minimize the number of dependencies. However, I've already had users ask for things like y4m support which would require specifying items such as color matrices and transfer characteristics... for which clap would be very helpful. So I'm not opposed to adding it here, since the work is already done.

BuyMyMojo commented 1 year ago

Glad to have been of help! 👍