Does anyone know the reasoning behind this commit and the changes it introduced? https://github.com/JetBrains/kotlin/commit/5773594412660af11357ac9adc6ffcb45138b840

@ilya.gorbunov shall know

Great will wait for his reply then 🙂 thanks @elizarov

The reason is that

ClosedRange.contains

is a virtual method, which can be implemented differently in inheritors. We've strived to reimplement

contains

extensions such that they delegated the check to the

contains

member. It turned out that could lead to some weird results related to rounding, as it was mentioned in the issue https://youtrack.jetbrains.com/issue/KT-18938. We're still to discuss what should be the correct behavior in that case.

@ilya.gorbunov is there a way to participate in this discussion?

I think it would be ok to post arguments in that issue comments.

A more interesting case is

3.0 in 1..4

. I, frankly, believe it should be

false

(added that to the issue)

Great, thank you for your explanation 🙂