Repository navigation
Regression: Generator::current()/::key() string-concat nested operand — returns true not yielded value (Zend/zend_generators.c) #13899
Description
Category
stdlib / language · php-src-strict · regression (nested operand in string concatenation)
Problem
After advancing a generator with next(), Generator::current() and Generator::key() must return the yielded value and integer key. When either method is a nested operand inside string concatenation with var_export(..., true), the VM returns true instead of the value/key.
Standalone method calls work; dot-concat form fails. Same failure class as pointer builtins (#13830 next(), closed #13811 prev()).
php-src reference
Zend/zend_generators.c—zim_Generator_current,zim_Generator_key
Repro (verified 2026-06-30)
test/repro/maintainer_gap_generator_current_concat_simple.php
test/repro/maintainer_gap_generator_key_concat.php
php bin/vm.php test/repro/maintainer_gap_generator_current_concat_simple.php
php bin/vm.php test/repro/maintainer_gap_generator_key_concat.php| Check | Zend | VM |
|---|---|---|
$g->next(); $g->next(); 'val='.var_export($g->current(), true) |
val=30 |
val=true |
$g->next(); 'key='.var_export($g->key(), true) |
key=1 |
key=true |
Standalone $g->current() / $g->key() |
correct | correct |
Scope (PHP-in-PHP first)
| Layer | Path |
|---|---|
| VM | Generator object handlers, method dispatch return typing |
| Compiler | nested-call / concat operand lowering if mis-binding return type |
| JIT | after VM green |
Done when (php-src-strict)
- Both repros exit 0 on VM
- Generator methods return correct types in concat/nested contexts
- JIT/AOT agree when in scope
Verification
php bin/vm.php test/repro/maintainer_gap_generator_current_concat_simple.php
php bin/vm.php test/repro/maintainer_gap_generator_key_concat.phpLinks
- Epic Epic: Self-host critical path — compiler compiles itself (M3→M5) #1492 · sibling pointer-builtin concat issues (Regression: next() string-concat nested operand — returns true not element (ext/standard/array.c) #13830, Regression: prev() string-concat nested operand — returns true not element (ext/standard/array.c) #13811)
Activity
PurHur commentedon Jun 30, 2026
Maintainer run 2026-06-30: added committed repro scripts referenced in this issue body:
test/repro/maintainer_gap_generator_current_concat_simple.php— FAIL (val=true, expectedval=30)test/repro/maintainer_gap_generator_key_concat.php— PASS on current master (key concat path may differ from current())Verified:
Sibling pointer-builtin concat regressions filed/tracked: #13903 (
prev), #13912 (next, re-#13830).PurHur commentedon Jun 30, 2026
claim: cursor-agent-lane-a — batch (issues #13899)
Add Generator concat regression guards for #13899. (#13944)
PurHur commentedon Jun 30, 2026
Merged via #13944.
Status: Repro was already green on master — root fix is #13910 (
siblingMethodCallFeedsVarExportConsumerfor hoisted MethodCall siblings invar_exportconcat). This PR adds maintainer repro scripts + PHPUnit guards.Verification: