Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:128505
Precedence: list
MIME-Version: 1.0
Date: Tue, 19 Aug 2025 09:32:22 +0200
Message-ID: <CANBC5np5gFoG2vy572K_rV5df58qBJdQuKsmYdHiprA2mLPWUg@mail.gmail.com>
Subject: [PHP-DEV] [RFC] Add clamp function
To: internals@lists.php.net
Content-Type: multipart/alternative; boundary="0000000000000c9b61063cb2dc17"
From: kylekatarnls@gmail.com (Kyle Katarn)

--0000000000000c9b61063cb2dc17
Content-Type: text/plain; charset="UTF-8"

Hello,

I took a look at thinkverse <https://github.com/thinkverse> work on
https://wiki.php.net/rfc/clamp and on the open issues (handling NAN and
other comparable values not being integer nor float).

This new implementation should resolve those issues:

https://github.com/php/php-src/pull/19434

If you allow me, I would update the RFC to add this new implementation
listed on it, remove the "Open issues" section if you don't see any with
the new implementation and would change the status of this RFC to draft to
reflect the fact that there is again an open PR for it.

Thanks,

--0000000000000c9b61063cb2dc17
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Hello,<div><br></div><div>I took a look at=C2=A0<a href=3D=
"https://github.com/thinkverse" target=3D"_blank" style=3D"color:rgb(145,15=
2,161);font-family:-apple-system,BlinkMacSystemFont,&quot;Segoe UI&quot;,&q=
uot;Noto Sans&quot;,Helvetica,Arial,sans-serif,&quot;Apple Color Emoji&quot=
;,&quot;Segoe UI Emoji&quot;;font-size:14px;font-weight:600;box-sizing:bord=
er-box;background-color:rgba(0,0,0,0);text-decoration-line:none">thinkverse=
</a>=C2=A0work on=C2=A0<a href=3D"https://wiki.php.net/rfc/clamp" target=3D=
"_blank">https://wiki.php.net/rfc/clamp</a>=C2=A0and on the open issues (ha=
ndling NAN and other comparable values not being integer nor float).</div><=
div><br></div><div>This new implementation should resolve those=C2=A0issues=
:</div><div><br></div><div><a href=3D"https://github.com/php/php-src/pull/1=
9434" target=3D"_blank">https://github.com/php/php-src/pull/19434</a></div>=
<div><br></div><div>If you allow me, I would update the RFC to add this new=
 implementation listed on it, remove the &quot;Open issues&quot; section if=
 you don&#39;t see any with the new implementation and would change the sta=
tus of this RFC to draft to reflect the fact that there is again an open PR=
 for it.</div><div><br></div><div>Thanks,=C2=A0</div></div>

--0000000000000c9b61063cb2dc17--