Hi Eli,
> My comments below.
thanks for the review. I've incorporated all your comments, see the
attached patch for the result.
> This could be improved as follows:
>
> @cindex thunk
> Return a @dfn{thunk} for evaluating the @var{forms}. A thunk is a
> closure (@pxref{Closures}) that evaluates the @var{forms} in the
> lexical environment present when @code{thunk-delay} had been called.
Done (but I've moved the cindex before the defmac).
> > address@hidden had been called.
>
> "had been called" or "will be called"?
"Had", I think, but I've rephrased that as follows:
A thunk is a closure (@pxref{Closures}) that inherits the lexical
environment of the address@hidden call.
> > @defun thunk-force thunk
> > Force @var{thunk} to perform the evaluation
> > of the forms specified to the address@hidden that created the
> > thunk.
>
> "to the thunk-delay" or "in the thunk-delay" (or "for the thunk-delay")?
I hope "in" is the right preposition?
> Quoting in Texinfo is ``like this'' (I guess you didn't use the Emacs
> Texinfo mode, or typed `C-q "' here.)
It was a pilot error - I had hit the key twice to get the "right"
quoting :-P - fixed.
> > address@hidden
> > +(f 12)
> > address@hidden "Calculating 1 plus 2 times 12"
> > +25
>
> This 25 should have @result{} before it, right?
Right. I also removed the quotes in the "print" line.
Thanks,
Michael.
Attachment:
0001-Add-macros-thunk-let-and-thunk-let.patch
Description: Text Data