Bug #70965 [Ver->Csd]: `yield from` with a common iterator primes too much
Edit report at https://bugs.php.net/bug.php?id=70965&edit=1
ID: 70965
Updated by: bwoebi@php.net
Reported by: srice at fb dot com
Summary: yield from with a common iterator primes too much
-Status: Verified
+Status: Closed
Type: Bug
Package: Scripting Engine problem
Operating System: Linux
PHP Version: 7.0.0RC7
Assigned To: bwoebi
Block user comment: N
Private report: N
New Comment:
Automatic comment on behalf of bobwei9@hotmail.com
Revision: http://git.php.net/?p=php-src.git;a=commit;h=73a92964ffd80ae907314b8202ed99716e4ba918
Log: Fixed bug #70965 (yield from with a common iterator primes too much)
Previous Comments:
------------------------------------------------------------------------
[2015-11-24 23:14:32] bwoebi@php.net
You are right⦠re-opening, will fix tomorrowâ¦
------------------------------------------------------------------------
[2015-11-24 23:11:51] srice at fb dot com
Thanks for the quick response! To be clear, the issue isnt the null at the end, its that
int(2) never gets yielded. An element is skipped in the iterator.
I'll let you decide to reopen the bug or not (cause I'm not sure if its still fixed by
your other change), but I'm pretty sure the last two values shouldn't be
int(10) int(10).
------------------------------------------------------------------------
[2015-11-24 22:55:54] bwoebi@php.net
Just seen this now⦠I've just fixed this "accidentally" as a side effect by the
bugfix for #70904 :-)
The last two values should be int(10) int(10); this is the case because the validity of the calling
generator is not affected by the validity of the yielded generator. The values are just synchronized
between all generators. So the last value of the generators is actually the last value of the
subgenerator you yielded from. (which is why it's twice.)
------------------------------------------------------------------------
[2015-11-24 21:14:38] srice at fb dot com
I used a similar test case for both of them, but I actually worked around the linked bug in this
test case (notice the while statement is checking the inner generator for validity,
while the linked bug checks the outer generator for validity).
This one seems to be due to autopriming (autopriming the second generator advances the shared
iterator) while the other is due to the valid method returning the result of valid on the delegate
generator (which is not an assumption you can make, since the outer generator still might have
yields to execute).
------------------------------------------------------------------------
[2015-11-24 20:56:05] requinix@php.net
Looks similar to bug #70904.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=70965
--
Edit this bug report at https://bugs.php.net/bug.php?id=70965&edit=1
Thread (7 messages)