Skip to content

Comments

ext/pcntl: various refactorings#21282

Draft
devnexen wants to merge 3 commits intophp:masterfrom
devnexen:pcntl_refactoring
Draft

ext/pcntl: various refactorings#21282
devnexen wants to merge 3 commits intophp:masterfrom
devnexen:pcntl_refactoring

Conversation

@devnexen
Copy link
Member

No description provided.

pcntl_alarm() accepts a zend_long (signed) but passes it to alarm(),
which takes an unsigned int. Negative values silently wrap to large
unsigned values, scheduling an alarm far in the future instead of
raising an error.
deduplicated max signals computation by using globals
since it is casted to zend_long, better using add_assoc_long_ex due to
loss of precision with add_assoc_double_ex
@devnexen devnexen changed the title Pcntl refactoring ext/pcntl: various refactorings Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant