sfackler/rust-postgres-range

No containment for f64/f32?

dovahcrow opened this issue · 1 comments

By looking into the code, it seems like trait Normalizable is only implemented for i32/i64 and time. So whats the rationale behind to use Normalizable and make it only work on discrete data type?

I don't think there's any reason to not implement it for floats.

I honestly think it might make sense to remove Normalizable entirely. If I remember correctly it was originally there to make the Rust type behave like Postgres with respect to bound normalization, but I don't really know why that's an interesting thing to do.