Merge pull request #7 from localheinz/fix/each

CloCkWeRX · web-flow · commit a335da940185 · 2023-04-08T21:43:32.000+09:30
Fix: Do not use `each()`
diff --git a/Text/Diff/Engine/native.php b/Text/Diff/Engine/native.php
@@ -192,15 +192,15 @@ function _diag ($xoff, $xlim, $yoff, $ylim, $nchunks)
                 }
                 $matches = $ymatches[$line];
                 reset($matches);
-                while (list(, $y) = each($matches)) {
+                foreach ($matches as list(, $y)) {
                     if (empty($this->in_seq[$y])) {
                         $k = $this->_lcsPos($y);
                         assert($k > 0);
                         $ymids[$k] = $ymids[$k - 1];
                         break;
                     }
                 }
-                while (list(, $y) = each($matches)) {
+                foreach ($matches as list(, $y)) {
                     if ($y > $this->seq[$k - 1]) {
                         assert($y <= $this->seq[$k]);
                         /* Optimization: this is a common case: next match is

Original file line number	Diff line number	Diff line change
`@@ -192,15 +192,15 @@ function _diag ($xoff, $xlim, $yoff, $ylim, $nchunks)`
`192`	`192`	`}`
`193`	`193`	`$matches = $ymatches[$line];`
`194`	`194`	`reset($matches);`
`195`		`- while (list(, $y) = each($matches)) {`
	`195`	`+ foreach ($matches as list(, $y)) {`
`196`	`196`	`if (empty($this->in_seq[$y])) {`
`197`	`197`	`$k = $this->_lcsPos($y);`
`198`	`198`	`assert($k > 0);`
`199`	`199`	`$ymids[$k] = $ymids[$k - 1];`
`200`	`200`	`break;`
`201`	`201`	`}`
`202`	`202`	`}`
`203`		`- while (list(, $y) = each($matches)) {`
	`203`	`+ foreach ($matches as list(, $y)) {`
`204`	`204`	`if ($y > $this->seq[$k - 1]) {`
`205`	`205`	`assert($y <= $this->seq[$k]);`
`206`	`206`	`/* Optimization: this is a common case: next match is`