Re: [Chicken-hackers] [PATCH] Fix #1620 by ignoring captured state of replaced variables

[Peter Bex]

On Wed, Jul 03, 2019 at 02:54:24PM +0300, megane wrote:
> Regarding the capturing, the capture test is still there:
> 
> >      (when (and value (not global))
> >        (when (eq? '##core#variable (node-class value))
> > -        (let* ((name (first (node-parameters value)))
> > -               (nrefs (db-get db name 'references)) )
> > +        (let ((name (first (node-parameters value))) )
> >            (when (and (not captured)
>                           ^^^^^^^^^^^^^^
>                           Here.
> I got the impression you wanted to remove this.

You're right, good catch!  That was an oversight on my part, I only
removed the captured check of the other variable.  I hope this makes
things faster in more cases.  I can make and test a new patch, but don't
know when I'll get around to it.  Possibly in the weekend.

Cheers,
Peter

signature.asc
Description: PGP signature