Skip to content
This repository has been archived by the owner on Mar 19, 2019. It is now read-only.

JVM-backed arrays don't work with map with 2 args #632

Closed
jessitron opened this issue Jun 5, 2017 · 1 comment
Closed

JVM-backed arrays don't work with map with 2 args #632

jessitron opened this issue Jun 5, 2017 · 1 comment
Labels
bug

Comments

@jessitron
Copy link
Contributor

JS Arrays support map with a fn with two args (element and index). This does not work when we return our own imitation-Array from the JVM
@kipz kipz added the bug label Jun 5, 2017
@kipz kipz self-assigned this Jun 5, 2017
kipz added a commit that referenced this issue Jun 5, 2017
@kipz
Ensure index & original array are passed to map fn #632
a3f91c2
ddgenome added a commit that referenced this issue Jun 5, 2017
@ddgenome
Merge pull request #633 from atomist/issue-632
003d180 
Ensure index & original array are passed to map fn #632
@kipz
Copy link
Contributor

kipz commented Jun 6, 2017

Fixed in master. Thanks for reporting @jessitron

@kipz kipz closed this as completed Jun 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jessitron @kipz