Skip to content

Extend wrapped JS function coverage in core runtime types #12263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+674 −52
Open

Extend wrapped JS function coverage in core runtime types #12263

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
133 changes: 133 additions & 0 deletions sdk/src/org.graalvm.webimage.api/src/org/graalvm/webimage/api/JSNumber.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,4 +119,137 @@ public boolean equals(Object that) {
public int hashCode() {
return javaDouble().hashCode();
}

@JS.Coerce
@JS(value = "return isFinite(number);")
public native static boolean isFinite(JSNumber number);

@JS.Coerce
@JS(value = "return isFinite(number);")
public native static boolean isFinite(Number number);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically, not every Number data type is autocoerced to JSNumber (i.e. JavaScript Number). There are many Number subtypes in Java for which a proxy will be created on the JavaScript side.

I guess isFinite returns true in all non-Number cases in JavaScript (?), but is this the intent?
Maybe a double number is what we want in the signature here.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar comment for isInteger, isNaN, isSafeInteger, parseInt and parseFloat.


@JS.Coerce
@JS(value = "return Number.isInteger(number);")
public native static boolean isInteger(JSNumber number);

@JS.Coerce
@JS(value = "return Number.isInteger(number);")
public native static boolean isInteger(Number number);

@JS.Coerce
@JS(value = "return isNaN(number);")
public native static boolean isNaN(JSValue number);

@JS.Coerce
@JS(value = "return isNaN(number);")
public native static boolean isNaN(Number number);

@JS.Coerce
@JS(value = "return Number.isSafeInteger(number);")
public native static boolean isSafeInteger(JSValue number);

@JS.Coerce
@JS(value = "return Number.isSafeInteger(number);")
public native static boolean isSafeInteger(Number number);

@JS.Coerce
@JS(value = "return parseFloat(number);")
public native static float parseFloat(Number number);

@JS.Coerce
@JS(value = "return parseFloat(number);")
public native static float parseFloat(String number);

@JS.Coerce
@JS(value = "return parseInt(number);")
public native static int parseInt(Number number);

@JS.Coerce
@JS(value = "return parseInt(number);")
public native static int parseInt(String number);

@JS.Coerce
@JS(value = "return parseInt(number, radix);")
public native static int parseInt(String number, int radix);

@JS.Coerce
@JS(value = "return Number.EPSILON;")
public native static double EPSILON();

@JS.Coerce
@JS(value = "return Number.MAX_SAFE_INTEGER;")
public native static double MAX_SAFE_INTEGER();

@JS.Coerce
@JS(value = "return Number.MAX_VALUE;")
public native static double MAX_VALUE();

@JS.Coerce
@JS(value = "return Number.MIN_SAFE_INTEGER;")
public native static double MIN_SAFE_INTEGER();

@JS.Coerce
@JS(value = "return Number.MIN_VALUE;")
public native static double MIN_VALUE();

@JS.Coerce
@JS(value = "return Number.NaN;")
public native static double NaN();

@JS.Coerce
@JS(value = "return Number.NEGATIVE_INFINITY;")
public native static double NEGATIVE_INFINITY();

@JS.Coerce
@JS(value = "return Number.POSITIVE_INFINITY;")
public native static double POSITIVE_INFINITY();

@JS.Coerce
@JS(value = "return this.toExponential();")
public native String toExponential();

@JS.Coerce
@JS(value = "return this.toExponential(fractionDigits);")
public native String toExponential(int fractionDigits);

@JS.Coerce
@JS(value = "return this.toFixed();")
public native String toFixed();

@JS.Coerce
@JS(value = "return this.toFixed(digits);")
public native String toFixed(int digits);

@JS.Coerce
@JS(value = "return this.toLocaleString();")
public native String toLocaleString();

@JS.Coerce
@JS(value = "return this.toLocaleString(locales);")
public native String toLocaleString(String locales);

@JS.Coerce
@JS(value = "return this.toLocaleString(locales, options);")
public native String toLocaleString(String locales, JSObject options);

@JS.Coerce
@JS(value = "return this.toPrecision();")
public native String toPrecision();

@JS.Coerce
@JS(value = "return this.toPrecision(precision);")
public native String toPrecision(int precision);

@Override
@JS.Coerce
@JS(value = "return this.toString();")
public native String toString();

@JS.Coerce
@JS(value = "return this.toString(radix);")
public native String toString(int radix);

@JS.Coerce
@JS(value = "return this.valueOf();")
public native double valueOf();
}
Loading