gendx / lzma-rs

An LZMA decoder written in pure Rust
MIT License
129 stars 27 forks source link

Return a value in lambdas passed to Bencher::iter. #52

Closed gendx closed 4 years ago

gendx commented 4 years ago

Pull Request Overview

This pull request changes the benchmarks to always return the relevant value in Bencher::iter, rather than (). This makes sure that the relevant code isn't optimized away.

Testing Strategy

This pull request was tested by running cargo bench before and after the change. Fortunately, no difference was observed in the benchmarks that we currently have.

Supporting Documentation and References

N/A

TODO or Help Wanted

N/A