Skip to content

Loading…

Array mutations can fail when arguments to splice are a string #2062

Closed
kevinpschaaf opened this Issue · 0 comments

1 participant

@kevinpschaaf

Repro here: http://jsbin.com/faqetabada/2/edit?html,output

Should just parseInt on args before notifying splices.

@kevinpschaaf kevinpschaaf added the p1 label
@kevinpschaaf kevinpschaaf self-assigned this
@kevinpschaaf kevinpschaaf referenced this issue from a commit
@kevinpschaaf kevinpschaaf Allow array API's accept string & negative args. Fixes #2062.
Brings the API more in line with native splice, etc.
2ef0ae0
@kevinpschaaf kevinpschaaf closed this issue from a commit
@kevinpschaaf kevinpschaaf Allow array API's accept string & negative args. Fixes #2062.
Brings the API more in line with native splice, etc.
7e2ceeb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.