Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:116283
MIME-Version: 1.0
References: <17c9ff9e9da.10b3e7f1c1280030.5141627710284677362@pmmp.io>
In-Reply-To: <17c9ff9e9da.10b3e7f1c1280030.5141627710284677362@pmmp.io>
Date: Thu, 21 Oct 2021 09:44:42 +0200
Message-ID: <CAF+90c8AcwPvVHfDh2X-xvRqhBOkn=f_0EF+TkuuRxeAvhpMGg@mail.gmail.com>
To: "Dylan K. Taylor" <dktapps@pmmp.io>
Cc: internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000004cd10305ced812ba"
Subject: Re: [PHP-DEV] Add ReflectionFunctionAbstract::isAnonymous()
From: nikita.ppv@gmail.com (Nikita Popov)

--0000000000004cd10305ced812ba
Content-Type: text/plain; charset="UTF-8"

On Thu, Oct 21, 2021 at 1:12 AM Dylan K. Taylor <dktapps@pmmp.io> wrote:

> Hi all,
>
> Given the addition of Closure::fromCallable() and the upcoming first-class
> callable syntax in 8.1, it seems slightly problematic that there's no
> simple way to tell by reflection if a Closure refers to an anonymous
> function or not. ReflectionFunctionAbstract::isClosure() (perhaps somewhat
> misleadingly) returns whether the closure is literally a \Closure instance,
> so it's not useful for this purpose.
>
> The only way to do this currently (that I know about) is to check if the
> name of the function contains "{closure}", which is a bit unpleasant and
> depends on undocumented behaviour.
>
> I'm proposing the addition of ReflectionFunctionAbstract::isAnonymous(),
> which would fill this use case, and may be able to offer an implementation.
>

Sounds reasonable to me!

Nikita

--0000000000004cd10305ced812ba--