Skip to content
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.

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

Safari & BigInt #4848

Closed
MartyIX opened this issue Sep 19, 2019 · 6 comments
Closed

Safari & BigInt #4848

MartyIX opened this issue Sep 19, 2019 · 6 comments
Labels
data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript

Comments

@MartyIX
Copy link

MartyIX commented Sep 19, 2019

Hi,

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt is not supported on Safari supposedly. However, https://developer.apple.com/safari/technology-preview/release-notes/ mentions BigInt in Release 57 which was released AFAIK.

I'm not a Mac user. Maybe somebody can elaborate on what the current state of BigInt support in Safari is.

Regards,
M.

@queengooborg queengooborg added the data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Sep 21, 2019
@stof
Copy link
Contributor

stof commented Oct 3, 2019

I suspect that these release notes are generated from a commit history. They might have a partial implementation of BigInt behind some flag for now.
https://bugs.webkit.org/show_bug.cgi?id=179001 is their tracking bug for that, and it still has a bunch of unresolved dependencies.

@queengooborg
Copy link
Collaborator

Safari 13 doesn't support BigInt objects as per manual testing (and I have not found any flags that can be enabled that would enable its support, so I'm thinking it's potentially a compilation flag). As such, I will be closing this issue.

@cvkline
Copy link

cvkline commented Feb 25, 2020

I see lots of updates to BigInt support in the Safari TP release notes, but for some reason even as of Version 101 which is the latest, they do not appear to work.

const foo = 12n
< SyntaxError: No identifiers allowed directly after numeric literal
BigInt
< ReferenceError: Can't find variable: BigInt
BigInt(5)
< ReferenceError: Can't find variable: BigInt

@stof
Copy link
Contributor

stof commented Feb 26, 2020

Well, I guess they generate the release notes based on some commit log or on closed issues, without taking into account flags which make the feature unavailable.

@sideshowbarker
Copy link
Collaborator

@vinyldarkscratch given #4848 (comment) maybe this issue should actually be closed? (Of course anyone can still comment here after it’s closed.)

@queengooborg
Copy link
Collaborator

Sorry, good catch! I must have forgotten to hit “Close” before... 🤦‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript
Projects
None yet
Development

No branches or pull requests

6 participants
@sideshowbarker @MartyIX @stof @queengooborg @cvkline and others