[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Implement `capture-lexical-environment' in evaluator

From: David Kastrup
Subject: Re: [PATCH] Implement `capture-lexical-environment' in evaluator
Date: Wed, 14 Dec 2011 10:08:19 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (gnu/linux)

Mark H Weaver <address@hidden> 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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]