Skip to content

Change "foreach" statement behavior #1034

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dstogov wants to merge 24 commits into from
Closed

Change "foreach" statement behavior #1034

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
4638f7b
Change "foreach" statement behavior (this is just a PoC yet)
dstogov Jan 28, 2015
dd2a36a
Use GET_OP1_ZVAL_PTR_DEREF() (IS_TMP_VAR and IS_CONST can't be IS_REF…
dstogov Jan 28, 2015
92e90c0
Fixed operand destruction in case of exceptions in iterator
dstogov Jan 28, 2015
61e7391
Fixed temporary variable re-allocation pass
dstogov Jan 28, 2015
eef80c5
Fixed foreach by reference iteration over constant array
dstogov Jan 28, 2015
994d435
Merge branch 'master' into foreach
dstogov Jan 28, 2015
2705d17
Merge branch 'master' into foreach
dstogov Jan 29, 2015
10a3260
New test
dstogov Jan 29, 2015
15a23b1
Reimplement iteration magic with HashTableIterators (see https://wiki…
dstogov Jan 29, 2015
721fc9e
Added new test
dstogov Jan 29, 2015
621fd74
Merge branch 'master' into foreach
dstogov Jan 30, 2015
4c5b385
More careful iterators update.
dstogov Jan 30, 2015
5aa9712
Implement consistent behavior for foreach by value over plain object
dstogov Jan 30, 2015
cc4b7be
Make internal function, operation on array passed by reference, to pr…
dstogov Jan 30, 2015
08302c0
Make array_splice() to preserve foreach hash position
dstogov Jan 30, 2015
fb2d079
API cleanup
dstogov Jan 30, 2015
b37f1d5
Fixed test name
dstogov Jan 30, 2015
5406f21
Reduced alghorithms complexity
dstogov Jan 30, 2015
1e41295
Generalize HashTableIterator API to allows its usage without iinvolve…
dstogov Jan 31, 2015
6c168e5
Merge branch 'master' into foreach
dstogov Feb 2, 2015
9d17a42
Merge branch 'master' into foreach
dstogov Feb 5, 2015
79b8bb1
Merge branch 'master' into foreach
dstogov Feb 9, 2015
7a50a32
Merge branch 'master' into foreach
dstogov Feb 11, 2015
b00f3bd
Merge branch 'master' into foreach
dstogov Feb 12, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
New test
  • Loading branch information
@dstogov
dstogov committed Jan 29, 2015
commit 10a3260b1f16b6075fd8140f673dfef4d5efea91
13 changes: 13 additions & 0 deletions Zend/tests/foreach_007.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
--TEST--
Foreach by reference and inserting new element when we are already at the end
--FILE--
<?php
$a = [1];
foreach($a as &$v) {
echo "$v\n";
$a[1]=2;
}
?>
--EXPECT--
1
2