BurntSushi / quickcheck

Automated property based testing for Rust (with shrinking).
The Unlicense
2.4k stars 149 forks source link

prevent stack overflows on some broken shrinkers #139

Closed oli-obk closed 7 years ago

oli-obk commented 8 years ago

cc #133

this is obviously a breaking change due to the new Eq bound... It might be too constraining

BurntSushi commented 7 years ago

I think it's too constraining. The fact that this even passed tests suggests we have insufficient test coverage, since I would think that this would prevent one from using f32/f64.