-
Notifications
You must be signed in to change notification settings - Fork 36
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
Philippe/fix/176 plus #204
Conversation
Further List proxy work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few missing tests to write, .sort()
behaviour is a bit off, and I think using array_copy_func
in PyProxyHandler.cc
can lead to incorrect behaviour on a number of methods, particularly in the case where the method takes a callback and the callback mutates the array. See my comments for more details.
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
Co-authored-by: Caleb Aikens <caleb@distributive.network>
assert str(type(e)) == "<class 'pythonmonkey.SpiderMonkeyError'>" | ||
assert str(e).__contains__("TypeError: forEach: At least 1 argument required, but only 0 passed") | ||
|
||
#map |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Responded above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Enhances the Proxy that enables Python List to JS Array conversion behavior, when lists are used in an array context
closes #176
closes #201