Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118147
MIME-Version: 1.0
References: <2b35605f-8da8-46b1-aec3-00bd1bfe47fd@www.fastmail.com>
 <7513519.rnE6jSC6OK@arnaud-t490> <CAC5V6gjk6T_XupP8aRr0TYEH9BCoaEJUr737St5c4YKUb02O5Q@mail.gmail.com>
 <5584889.DvuYhMxLoT@arnaud-t490>
In-Reply-To: <5584889.DvuYhMxLoT@arnaud-t490>
Date: Fri, 1 Jul 2022 10:12:33 +0200
Message-ID: <CAC5V6ghfa=ND6kjOdeSxB6xGKmmAjj-FfLov-zbo6A4_3CQxmw@mail.gmail.com>
To: internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] [RFC] Short Closures 2, aka auto-capture take 3
From: guilliam.xavier@gmail.com (Guilliam Xavier)

On Thu, Jun 30, 2022 at 7:37 PM Arnaud Le Blanc <arnaud.lb@gmail.com> wrote:
>
> On jeudi 30 juin 2022 18:29:44 CEST Guilliam Xavier wrote:
> > Ah? Sorry, I had interpreted
> > https://github.com/php/php-src/pull/8330/files#diff-85701127596aca0e597bd796
> > 1b5d59cdde4f6bb3e2a109a22be859ab7568b4d2R7318-R7320 as "capture the
> > *minimal* set of variables for *both* arrow functions and short closures",
> > but I was wrong?
>
> No, you are right, the PR changes arrow functions too. But in the RFC we
> decided to not touch the arrow functions for now.

Ah, I see that you have updated the PR indeed:
https://github.com/php/php-src/pull/8330/commits/5bb0a1c8d032666079db5dab94b4b22b2afa9dac
(and thanks for the test).

PS: so the link I gave before is now outdated ^^' I should have given
https://github.com/php/php-src/pull/8330/commits/9dec265adba44dcf9d2cadc05dd5ad842fc4ae66#diff-85701127596aca0e597bd7961b5d59cdde4f6bb3e2a109a22be859ab7568b4d2R7318-R7320

-- 
Guilliam Xavier