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

ng/eval: possible race triggered by eval/testdata/method{1,2} #158

Open
sbinet opened this issue Dec 11, 2017 · 1 comment
Open

ng/eval: possible race triggered by eval/testdata/method{1,2} #158

sbinet opened this issue Dec 11, 2017 · 1 comment

Comments

@sbinet
Copy link
Collaborator

sbinet commented Dec 11, 2017

with go1.10beta1, running the following failed:

$> go test -v -count=3 ./eval
=== RUN   TestExprs
--- PASS: TestExprs (0.00s)
[...]
    --- FAIL: TestPrograms/method1 (0.00s)
    	eval_test.go:212: 25: ng eval panic: no method pool entries available for Read, func([]uint8) (int, error)
    --- FAIL: TestPrograms/method2 (0.00s)
    	eval_test.go:212: 13: ng eval panic: no method pool entries available for Read, func([]uint8) (int, error)
[...]
@crawshaw
Copy link
Member

I am actively working on this. The method pool is a failed experiment that needs to go. I just keep getting distracted by yaks to shave when I try to fix this.

My intention is to use the Go-generating backend to construct method types usable that can be loaded as plugins when a methodik is defined. The one issue is how to use named types as part of the type (or method parameters). Restricting methodiks to struct types solves the former, but the latter is tricky.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants