Re: [RFC] Add num_available_processors
| From: | Gina P. Banyard | Date: | Tue, 17 Jun 2025 09:59:47 +0000 |
| Subject: | Re: [RFC] Add num_available_processors | ||
| References: | 1 2 3 4 5 6 7 8 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-127696@lists.php.net to get a copy of this message | ||
On Sunday, 15 June 2025 at 19:32, Daniel Kesselberg <mail@danielkesselberg.de> wrote:
> Hi,
>
> Thanks for all your feedback on the RFC.
>
> I've updated the RFC to incorporate most of your feedback:
> https://wiki.php.net/rfc/num_available_processors
>
> 1) The limitation, that the CPU affinity mask is ignored
This is something you need to decide on as the RFC author.
> 2) The naming discussion
I don't care much about this topic, but possibly adding the sys_ prefix makes the most sense to
"soft namespace" the function.
> 3) Function availability on unsupported platforms
The function should not be available if it is unsupported,
as this how every other conditionally supported feature is handled.
> 4) Return type
Arguably, this should always return an integer.
Which if my understanding is correct, when you make it conditionally available then it cannot fail.
> How do we continue? ;) I see there are various ideas how to approach it,
> is that something you would vote (let's do a or b) on, or how does that
> work?
Small RFCs should only have a single yes/no vote.
As the RFC author, you can (and should) be opinionated.
Best regards,
Gina P. Banyard
PS: Minor list etiquette remainder, please do not top post, and only keep the relevant things while
quoting.