Skip to content

[8.6] Add clamp function#554

Open
kylekatarnls wants to merge 5 commits intosymfony:1.xfrom
kylekatarnls:feature/clamp
Open

[8.6] Add clamp function#554
kylekatarnls wants to merge 5 commits intosymfony:1.xfrom
kylekatarnls:feature/clamp

Conversation

@kylekatarnls
Copy link

@kylekatarnls kylekatarnls commented Nov 19, 2025

RFC: https://wiki.php.net/rfc/clamp_v2
Original implementation: php/php-src#19434
Documentation: php/doc-en#4814

@nicolas-grekas
Copy link
Member

That would be for PHP 8.6, isn't it?

@kylekatarnls
Copy link
Author

That would be for PHP 8.6, isn't it?

Correct, moving it.

@kylekatarnls kylekatarnls force-pushed the feature/clamp branch 2 times, most recently from 5bbf49c to d88b259 Compare November 19, 2025 09:53
@kylekatarnls kylekatarnls changed the title [8.5] Add clamp function [8.6] Add clamp function Nov 19, 2025
@kylekatarnls kylekatarnls force-pushed the feature/clamp branch 3 times, most recently from bead94d to c9033e5 Compare November 19, 2025 10:45
@kylekatarnls kylekatarnls marked this pull request as ready for review November 19, 2025 18:05
@kylekatarnls kylekatarnls reopened this Dec 7, 2025
@kylekatarnls
Copy link
Author

kylekatarnls commented Dec 19, 2025

php/php-src#19434 is merged (will land in PHP 8.6)

kylekatarnls and others added 3 commits December 24, 2025 16:58
Co-authored-by: Alexandre Daubois <2144837+alexandre-daubois@users.noreply.github.com>
Copy link
Contributor

@Ayesh Ayesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a very minor nitpick.

@kylekatarnls kylekatarnls requested a review from Ayesh January 8, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants