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.

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

Jump to bottom

Editorial: mathematical operations on BigInts #1723

Closed
jmdyck opened this issue Oct 7, 2019 · 2 comments
Closed

Editorial: mathematical operations on BigInts #1723

jmdyck opened this issue Oct 7, 2019 · 2 comments

Comments

@jmdyck
Copy link
Collaborator

jmdyck commented Oct 7, 2019

5.2.5 Mathematical Operations begins:

This specification makes reference to two kinds of numeric values:
[Number and Mathematical value]

This clause should presumably be updated to acknowledge BigInts.

(Of the specific operations defined, I think modulo is the only one that the spec currently applies to BigInt values.)
@ljharb
Copy link
Member

ljharb commented Oct 7, 2019

cc @caiolima @littledan

@jmdyck jmdyck mentioned this issue Oct 7, 2019
Merged
@bakkot
Copy link
Contributor

bakkot commented Jan 10, 2021

I believe #2007 comprehensively resolved this.

@bakkot bakkot closed this as completed Jan 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ljharb @bakkot @jmdyck