BurntSushi / cargo-benchcmp

A small utility to compare Rust micro-benchmarks.
The Unlicense
342 stars 21 forks source link

Allow running as either `cargo benchcmp` or `cargo-benchcmp` #20

Closed fitzgen closed 8 years ago

fitzgen commented 8 years ago

From #17:

@BurntSushi says:

You need to actually use cargo benchcmp, not cargo-benchcmp.

Yes, this is annoying and yes this should be in the README. It looks like this is Docopt's fault, and the error message is awful. :-(

@fitzgen says:

Ah because when run as cargo benchcmp you get two args and when run as cargo-benchcmp you get one? Would it be possible to detect which situation we're in before passing args to docopt and then do the Right Thing after that?

@BurntSushi says:

I'd wholeheartedly support that, yes. :-)