Allow hex numbers and data units for block size argument by merkrafter · Pull Request #144 · sharkdp/hexyl
Thank you very much for your contribution!
This looks great on a first glance. Happy to review it in detail after the cleanup.
I used the
const_formatcrate to display the default block size to the user in a convenient and consistent way. However, using it required Rust 1.46.0+. Do you think this detail is worth raising the minimum required Rust version?
I didn't know this crate. That sounds really helpful. Bumping to Rust 1.46 sounds good. Since this only affects hexyl as a tool and not hexyl as a library, I'd be okay to bump it even further, in case that helps with other parts of the code. I don't think we have a lot of "customers" anyway: https://crates.io/crates/hexyl/reverse_dependencies