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

Editorial: parseFloat/parseInt: fix unintentional normative change from #1602 #1634

Merged
merged 1 commit into from
Jul 19, 2019

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Jul 18, 2019

Thanks to @devsnek for discovering.

@ljharb ljharb requested review from devsnek, zenparsing and a team July 18, 2019 15:53
spec.html Show resolved Hide resolved
@devsnek
Copy link
Member

devsnek commented Jul 18, 2019

BTW, the bug was with parseX(undefined) and parseX(null), which should return NaN, not throw an error.

@leobalter
Copy link
Member

This is a nice caught and fix. I appreciate the work done here!

spec.html Show resolved Hide resolved
@ljharb ljharb force-pushed the trim_string_redux branch from dc8a979 to 4661220 Compare July 18, 2019 16:14
@ljharb ljharb self-assigned this Jul 18, 2019
@ljharb ljharb requested review from devsnek and leobalter July 18, 2019 16:26
spec.html Show resolved Hide resolved
spec.html Show resolved Hide resolved
@ljharb ljharb force-pushed the trim_string_redux branch from 4661220 to 503ff83 Compare July 18, 2019 22:55
@ljharb ljharb requested a review from leobalter July 18, 2019 22:55
@ljharb ljharb force-pushed the trim_string_redux branch from 503ff83 to a68d129 Compare July 19, 2019 15:55
@ljharb ljharb merged commit a68d129 into tc39:master Jul 19, 2019
@ljharb ljharb deleted the trim_string_redux branch July 19, 2019 15:56
@jmdyck
Copy link
Collaborator

jmdyck commented Jul 19, 2019

(Note re title: the unintentional change was in #1602, not #1629.)

@ljharb ljharb changed the title Editorial: parseFloat/parseInt: fix unintentional normative change from #1629 Editorial: parseFloat/parseInt: fix unintentional normative change from #1602 Jul 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants