Hi,

ludo@chbouib.org (Ludovic Courtès) writes:

> For 1.8, I'm pretty much inclined to commit a similar patch, i.e., where
> `module-make-local-var!' and `scm_m_define' are copied from HEAD.  This
> would break code that does things like:
>
>   (define foo (begin (set! foo 1) (+ foo 1)))
>
> but I think it's reasonable to break such code (which relies on
> non-R5RS-compliant behavior anyway), especially given the performance
> gain we get in return.  What do you think?

I committed the attached patch to 1.8.

I added an "Incompatible Changes" category in `NEWS'.  This sounds a bit
frightening, but the description hopefully emphasizes the
conservativeness of the change.  Please do improve the wording or
presentation if you find it inappropriate.

Thanks,
Ludovic.