Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121388
MIME-Version: 1.0
References: <a7ff0df5-bf67-4d94-8d13-7771e11fea47@daniil.it>
In-Reply-To: <a7ff0df5-bf67-4d94-8d13-7771e11fea47@daniil.it>
Date: Wed, 18 Oct 2023 14:00:23 +0100
Message-ID: <CAEKnhAHF9WYOGZCLFMvKd2tyrCgZHrcEfi7Nf538BTSV+JM6eQ@mail.gmail.com>
To: Daniil Gentili <daniil@daniil.it>
Cc: internals@lists.php.net
Content-Type: multipart/alternative; boundary="0000000000009c32cd0607fd3948"
Subject: Re: [PHP-DEV] Deprecate posix_times
From: bukka@php.net (Jakub Zelenka)

--0000000000009c32cd0607fd3948
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi,

On Tue, Oct 17, 2023 at 6:39=E2=80=AFPM Daniil Gentili <daniil@daniil.it> w=
rote:

> Hi everyone,
>
> I would like to submit an RFC (and related PR) to deprecate posix_times
> for PHP 8.4, removing it for PHP 9.
>

The times() is defined by POSIX and I don't see a reason why it couldn't be
exposed even though there is a better alternative. I think posix extension
should just expose posix defined functions and it is thus a low level
extension.


>
> There are multiple reasons to deprecate this function:
>
> 1)
>
> The values which times(2) returns are expressed as the number of clock
> ticks elapsed since the process was started: the frequency of the
> aforementioned clock varies from platform to platform, and PHP does not
> expose the sysconf(2) function, which is what should be used to fetch
> the clock frequency, invoking `sysconf(_SC_CLK_TCK)`.
>
>
I think the only thing that is needed is to expose _SC_CLK_TCK as
POSIX_SC_CLK_TCK and add a note to docs about it in the same way how this
is documented for the C counterpart.

Cheers

Jakub

--0000000000009c32cd0607fd3948--