Mark H Weaver <m...@netris.org> writes: > This is a _preliminary_ patch. In particular: > > * The compiler does not yet handle (capture-lexical-environment) > (which uses a new tree-il type). > > * The lexical environment object is currently non-opaque list structure. > > * I deliberately avoided reindenting eval.scm so that the non-whitespace > changes would be evident, to make review easier. > > * I wouldn't be surprised if `primitive-local-eval' does the wrong thing > if (current-module) is different from what it was when the associated > `primitive-eval' was called.
I don't know what the right thing would be (the simplest would likely be to just save the current module along with the rest) and in my previous use of local-eval, I had no idea what would happen in this case. But since my application seemed to work as expected (likely because of not changing modules), I left worrying about this alone. -- David Kastrup
