go ignores async code in letfn body

Description

The following snippet fails because <! is outside of go boundaries.

Async code in letfn body should be rewritten by the go macro, given all bound functions can be properly initialized when the body starts.

Environment

None

Activity

Show:
Nicola Mometto
October 16, 2018, 5:51 PM

Removing bad patch

Alex Miller
November 19, 2018, 5:42 PM

What's the status on this? 0001-ASYNC-221-handle-letfn-in-ioc+test.patch is the patch?

leonoel
November 19, 2018, 7:34 PM

No, the fix and the test are both wrong.
The fix doesn't properly establishes circular references and the test fails to check it because it shadows clojure.core's odd?/even?.

leonoel
November 19, 2018, 7:36 PM

Updated example to properly check circular references

Assignee

Unassigned

Reporter

leonoel

Labels

None

Approval

None

Patch

Code and Test

Priority

Major