Use chill.ClosureCleaner to remove outer$ refs on Functions #273

Closed
wants to merge 1 commit into
from

Projects

None yet

1 participant

@johnynek
Collaborator
@johnynek
Collaborator

Looks like there are serialization errors going on here with this change. Need to look more carefully at what is going on. Candidates: maybe ClosureCleaner needs to be Serializable. Maybe there is some breakage with the ClosureCleaner itself. Maybe chill is having problem serializing some resulting data.

@johnynek
Collaborator

Actually, it looks like ClosureCleaner is either a no-op or busted now, so let's hold off here. Watch this issue:

twitter/chill#21

Once this is good, we can reconsider.

@johnynek
Collaborator

I am starting to think this closure cleaner thing is only useful for a repl.

Closing until we get someone working on that or we prove otherwise.

@johnynek johnynek closed this Jul 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment