Repository navigation
Regression: atan2()/pow()/fmod()/intdiv()/base_convert()/log() arity — LogicException not ArgumentCountError (ext/standard, re-#21964) #21982
Description
Category
Regression · php-src-strict · stdlib arity (pillar 4; sibling of #21964)
Problem
#21964 covers strpos/implode/count/… wrong-arity throwing LogicException instead of ArgumentCountError. The same bug remains for several math/base_convert builtins: under-arity throws LogicException from hand-rolled argc checks.
| Repro | Zend 8.2+ | VM (2026-07-21) |
|---|---|---|
atan2(1) |
ArgumentCountError |
LogicException |
pow(2) |
ArgumentCountError |
LogicException |
fmod(5) |
ArgumentCountError |
LogicException |
intdiv(5) |
ArgumentCountError |
LogicException |
base_convert('10', 2) |
ArgumentCountError |
LogicException |
log() |
ArgumentCountError |
LogicException |
php-src reference
- php/php-src
ext/standard/math.c—atan2/pow/fmod/intdiv/logarginfo - php/php-src
ext/standard/basic_functions.stub.php—base_convert
PHP implementation target
ext/standard/{atan2,pow,fmod,intdiv,base_convert,log}.php(and JITcall()mirrors) — replaceLogicExceptionargc guards withArgumentCountErrormatching Zend messages- Reuse the same pattern as the Regression: strpos()/implode()/preg_match()/json_encode()/count()/abs()/defined()/constant()/gettype()/settype()/uniqid() arity — LogicException not ArgumentCountError (ext/standard, re-#21756) #21964 / Regression: addcslashes() arity error throws LogicException — Zend ArgumentCountError (ext/standard/string.c) #21756 fix; no new C
Repro
./script/docker-exec.sh -- bash -lc 'php bin/vm.php test/repro/maintainer_gap_math_arity_logicexception.php'Done when
- Each under-arity call in the repro throws
ArgumentCountError(class name match) - Compliance
.phptguard for at leastpow/intdiv/base_convert/logzero-/under-arity - php-src-strict; no php-compiler-strict shortcut
- Coordinate with Regression: strpos()/implode()/preg_match()/json_encode()/count()/abs()/defined()/constant()/gettype()/settype()/uniqid() arity — LogicException not ArgumentCountError (ext/standard, re-#21756) #21964 if still open — do not duplicate work if a worker already widened that PR
Activity
PurHur commentedon Jul 22, 2026
claim: PHP Compiler STD Lib — starting this run
Pre-triage gates (host):
php script/bootstrap-inventory.php --check→ OK 6331/6331make bootstrap-selfhost-link(docker-exec) — still running; will paste resultImplementing ArgumentCountError for atan2/pow/fmod/intdiv/base_convert/log under-arity (sibling of closed #21964).
PurHur commentedon Jul 22, 2026
Merged in #22057
Verification: VM + JIT repro green;
./script/phpunit.sh --filter issue21982_math_arity→ OK (4 tests). AOT compile currently red on master for HelloWorld too (StringOffsetRuntimezExt / htmlspecialchars bridge) — not introduced by this change; JIT covers the arity path.Stdlib: math/base_convert under-arity throws ArgumentCountError (#21982…
chore: drop stale maintainer_gap repros for closed Jul 22 issues (#22255