Skip to content

Error when using reflect.Select #420

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
Closed
dave opened this issue Mar 12, 2016 · 4 comments
Closed

Error when using reflect.Select #420

dave opened this issue Mar 12, 2016 · 4 comments
Labels
bug

Comments

@dave
Copy link
Contributor

dave commented Mar 12, 2016

The following test works correctly when running natively (printing "Received on chan multi"). However when run via gopherjs, it errors with "Uncaught Error: Native function not implemented: reflect.rselect":

https://play.golang.org/p/p9yfcLB5t8
@dmitshur dmitshur added the bug label Mar 13, 2016
@dmitshur
Copy link
Member

According to https://github.com/gopherjs/gopherjs/blob/master/doc/packages.md, reflect package is supposed to be fully supported, so this is a bug. Thanks for reporting.

@neelance
Copy link
Member

@davelondon Thanks for reporting. It was not supported initially but should have been by now. Somehow it slipped through the cracks. I've taken care of it now.

@neelance
Copy link
Member

Hmm, the test of the reflect package passes now, but your test case does not. Looking into it...

@neelance neelance reopened this Mar 13, 2016
@neelance
Copy link
Member

Works now. :-)

dmitshur added a commit to gopherjs/gopherjs.github.io that referenced this issue Mar 14, 2016
@dmitshur
Regenerate.
5d6ad58 
go generate github.com/gopherjs/gopherjs.github.io/...

Includes fix for gopherjs/gopherjs#420.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@neelance @dave @dmitshur