Remove unneeded to_f64 funciton

[younies]

No description provided.

[robertbastian]

Actually why not use ratio.get_ratio().to_f64()?

[younies]

Actually why not use ratio.get_ratio().to_f64()?

either way, we need to have the feature of "big-int", also, get_ratio() should consume the object.

and what is the point of have IcuRatio if we are going to get_ratio() for all the operations?

as in #4841.

[robertbastian]

get_ratio() should consume the object

You'll need both, checking .is_reciprocal() should not consume.

and what is the point of have IcuRatio if we are going to get_ratio() for all the operations?

That you can define things like impl Mul<SiPrefix>, which you cannot for BigRational

[robertbastian]

.

[robertbastian]

This is why there should also be a -> &BigRational method, this clone is not necessary because to_f64 doesn't need ownership. However, given that this is in-crate, you can use test.result.0.to_f64()

[younies]

You'll need both, checking .is_reciprocal() should not consume.

I prefer that these functions to be implemented for IcuRatio