Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:130540
Precedence: list
MIME-Version: 1.0
Date: Thu, 02 Apr 2026 22:15:42 +0200
To: Jakub Zelenka <bukka@php.net>
Cc: Valentin Udaltsov <udaltsov.valentin@gmail.com>, PHP internals list
 <internals@lists.php.net>
Subject: Re: [PHP-DEV] Re: [RFC] Polling API
In-Reply-To: <CAEKnhAFqcskpHr-JDDk6rjFX7qQgWnP73EgKRcm4pYqdXPbuiA@mail.gmail.com>
References: <CAEKnhAFZXqWH20Cy2g6-0c2V1Vd4w-N0M4xefzt6tTEp90+LqQ@mail.gmail.com>
 <CAEKnhAF2bqkNcHnXGa66JhGBsQRD3KE1tBarXzPJ9jf_7Od0Bw@mail.gmail.com>
 <CAPyeZ4Ou5XMyAqGRP_MYHGb3hLyu5F508tE7cVEsHMA5VbSH6A@mail.gmail.com>
 <CAEKnhAFqcskpHr-JDDk6rjFX7qQgWnP73EgKRcm4pYqdXPbuiA@mail.gmail.com>
Message-ID: <126fb4b28fde5cb46753d22edd17ab01@bastelstu.be>
Content-Type: text/plain; charset=UTF-8;
 format=flowed
Content-Transfer-Encoding: 8bit
From: tim@bastelstu.be (=?UTF-8?Q?Tim_D=C3=BCsterhus?=)

Hi

Am 2026-03-31 15:23, schrieb Jakub Zelenka:
> There is actually no internal API for hooks so it would require using
> object handlers (which is from the user space point of view just 
> __get). It
> means no stubs declaration either. We also don't have an internal 
> policy
> where to use hooks and where methods. So there are still lots of 
> blockers
> to start using it in core.

While I agree with the conclusion that using hooks internally is 
complicated and there is no real policy there, Valentin was actually 
making a good point: We could use regular `readonly` properties for 
those cases where we're just exposing a “constructor parameter”. 
Specifically:

- Context::getBackend() -> public readonly Backend $backend;
- Watcher::getHandle() -> public readonly Handle $handle;

Potentially Watcher::$data could also be a regular (non-readonly) 
property, or does `modifyData()` contain additional logic that is not 
just “store the value somewhere”?

There is precedent for this kind of regular `readonly` property 
referring to an input object in ext/random with Randomizer::$engine: 
https://github.com/php/php-src/blob/32c1931f18109655bc074dd5cda3248b838de636/ext/random/random.stub.php#L139

Best regards
Tim Düsterhus