azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix incorrect Double.toLong conversion for infinite values in JS

Browse Source 
#KT-33225 Fixed
This commit is contained in:
Ilya Gorbunov
2019-08-08 22:38:10 +03:00
parent 8bf61a12df
commit ebb59d51d2
2 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
libraries/stdlib/js-ir/runtime/longjs.kt
+5 -5
View File
@@ -335,13 +335,13 @@ internal fun Long.shiftRightUnsigned(numBits: Int): Long {
internal fun fromInt(value: Int) = Long(value, if (value < 0) -1 else 0)
/**
* Returns a Long representing the given value, provided that it is a finite
* number. Otherwise, zero is returned.
* @param {number} value The number in question.
* @return {!Kotlin.Long} The corresponding Long value.
* Converts this [Double] value to [Long].
* The fractional part, if any, is rounded down.
* Returns zero if this `Double` value is `NaN`, [Long.MIN_VALUE] if it's less than `Long.MIN_VALUE`,
* [Long.MAX_VALUE] if it's bigger than `Long.MAX_VALUE`.
*/
internal fun fromNumber(value: Double): Long {
if (value.isNaN() || !value.isFinite()) {
if (value.isNaN()) {
return ZERO;
} else if (value <= -TWO_PWR_63_DBL_) {
return MIN_VALUE;
libraries/stdlib/js-v1/src/js/long.js
+5 -3
View File
@@ -94,13 +94,15 @@ Kotlin.Long.fromInt = function(value) {
/**
* Returns a Long representing the given value, provided that it is a finite
* number. Otherwise, zero is returned.
* Converts this number value to `Long`.
* The fractional part, if any, is rounded down.
* Returns zero if this `Double` value is `NaN`, `Long.MIN_VALUE` if it's less than `Long.MIN_VALUE`,
* `Long.MAX_VALUE` if it's bigger than `Long.MAX_VALUE`.
* @param {number} value The number in question.
* @return {!Kotlin.Long} The corresponding Long value.
*/
Kotlin.Long.fromNumber = function(value) {
if (isNaN(value) || !isFinite(value)) {
if (isNaN(value)) {
return Kotlin.Long.ZERO;
} else if (value <= -Kotlin.Long.TWO_PWR_63_DBL_) {
return Kotlin.Long.MIN_VALUE;