Move `utils::FloatConversions` to `rand_core`

dhardy / rand

http://doc.rust-lang.org/rand

Other

2 stars 2 forks source link

Move `utils::FloatConversions` to `rand_core` #70

Closed pitdicker closed 6 years ago

pitdicker commented 6 years ago

It seems to me conversions to floats are fundamental enough to be part of rand_core. While you probably want some sort of distribution or range when using floats, these conversion functions are building blocks to provide a distribution or range.

dhardy commented 6 years ago

I don't see much point moving FP stuff into rand-core; it's only really aimed at cryptographic users and RNG implementations (and we already decided to remove next_f32 / next_f64 from Rng). Any disagreement or shall I close this?

pitdicker commented 6 years ago

Ok, restricting rand-core to only the minimum needed for RNG implementations seems like a good idea.