Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:122398
Precedence: bulk
MIME-Version: 1.0
References: <CAK7R4azyh4Yi6NjYFLwD_9BMeH9TD8YmTxQ46cn96Z7cCVy-eA@mail.gmail.com>
 <A813DBD2-219F-404C-AB67-AD5ECA0124C2@gmail.com>
In-Reply-To: <A813DBD2-219F-404C-AB67-AD5ECA0124C2@gmail.com>
Date: Fri, 16 Feb 2024 18:16:43 +0100
Message-ID: <CAK7R4aysYd6VYCjoD4zz5voKyYJQ1WWBRAjGL5db061uDZ1_NA@mail.gmail.com>
Subject: Re: [PHP-DEV] int|float for sleep? sleep(0.1) => sleep 0.1 seconds
To: Claude Pache <claude.pache@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: hans@loltek.net (Hans Henrik Bergan)

On Fri, 16 Feb 2024 at 09:36, Alexandru P=C4=83tr=C4=83nescu <drealecs@gmai=
l.com> wrote:
>
>
> On Fri, Feb 16, 2024 at 10:18=E2=80=AFAM Hans Henrik Bergan <hans@loltek.=
net> wrote:
>>
>> Can we make sleep accept int|float?
>> Made a PR: https://github.com/php/php-src/pull/13401
>>
>> For years when I wanted to sleep for 0.1 seconds, it annoyed me that I c=
ouldn't do
>> `sleep(0.1);`
>> instead I had to do
>> `usleep(figure out how many microseconds there are in 0.1 seconds and pu=
t it here);`
>>
>
> Based on previous discussions https://externals.io/message/111448
> it seems the desired way was for a RFC to be created for this change, as =
it involves some small changes that break BC.
>
> There is also the PR done by Michael Vo=C5=99=C3=AD=C5=A1ek from 3.5 year=
s ago where some technical discussions took place:
> https://github.com/php/php-src/pull/5961
>
> Regards,
> Alex
>

Interesting indeed. I implemented some improvements from that PR now
(higher precision sleep and fixing the windows 192-issue),
and got Michael Vo=C5=99=C3=AD=C5=A1ek's attention :)
also applied for RFC Karma.

On Fri, 16 Feb 2024 at 14:20, Claude Pache <claude.pache@gmail.com> wrote:
>
> Hi,
>
>
> FWIW Python's `time.sleep` also works like this: https://docs.python.org/=
3/library/time.html#time.sleep
>
>
> Python also implements the following:
>
> If the sleep is interrupted by a signal and no exception is raised by the=
 signal handler, the sleep is restarted with a recomputed timeout.
>
>
> I think we should also implement that. As a consequence, `sleep(...)` wil=
l always return `0`.
>
> If we =E2=80=9Cfix=E2=80=9D sleep(), let=E2=80=99s =E2=80=9Cfix=E2=80=9D =
it completely.
>
> =E2=80=94Claude


it seems we already have a function for that, it's called `time_sleep_until=
()`,
quoting the documentation
>    Note: All signals will be delivered after the script wakes up.

in contrast to normal sleep() that is signal interruptible.
Sorry, I'm not interested in making sleep() uninterruptible, at least
not until after I get int|float accepted upstream.