search

BurntSushi / cargo-benchcmp

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

Include speedup in output #30

Closed Luthaf closed 7 years ago

Luthaf commented 7 years ago

Fix #29.

I did a bit of yak shaving on the path by included the Cargo.lock in the repo. It was in gitignore but already committed. I can revert this if you prefer.

I also needed a workaround for a bug in the second_law crate. It should work on Travis and Appveyor, but to run the tests locally one need to set OUT_DIR to target/debug/<whatever>/<whatever>/<whatever>/.

The example output in the README are not updated yet. Do you want me to do this?

BurntSushi commented 7 years ago

Thanks so much for doing this! And double-thanks for the (hilarious) second_law workaround. I want to rip that dependency out soon, but haven't gotten a chance to do it.

The example output in the README are not updated yet. Do you want me to do this?

Yes please. :-)

This otherwise looks great.

BurntSushi commented 7 years ago

Fantastic, thank you!