Remove more assignments in conditionals (while each to foreach)
authorSam Reed <reedy@users.mediawiki.org>
Sat, 29 Oct 2011 02:04:27 +0000 (02:04 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Sat, 29 Oct 2011 02:04:27 +0000 (02:04 +0000)
includes/MagicWord.php
includes/diff/DairikiDiff.php

index e327971..136aac9 100644 (file)
@@ -754,7 +754,7 @@ class MagicWordArray {
         */
        function parseMatch( $m ) {
                reset( $m );
-               while ( list( $key, $value ) = each( $m ) ) {
+               foreach ( $m as $key => $value ) {
                        if ( $key === 0 || $value === '' ) {
                                continue;
                        }
index c935eee..7f618fc 100644 (file)
@@ -371,7 +371,7 @@ class _DiffEngine {
                                }
                                $matches = $ymatches[$line];
                                reset( $matches );
-                               while ( list( , $y ) = each( $matches ) ) {
+                               foreach ( $matches as $y ) {
                                        if ( empty( $this->in_seq[$y] ) ) {
                                                $k = $this->_lcs_pos( $y );
                                                assert( $k > 0 );
@@ -379,7 +379,7 @@ class _DiffEngine {
                                                break;
                                        }
                                }
-                               while ( list ( , $y ) = each( $matches ) ) {
+                               foreach ( $matches as $y ) {
                                        if ( $y > $this->seq[$k -1] ) {
                                                assert( $y < $this->seq[$k] );
                                                // Optimization: this is a common case:
@@ -493,7 +493,7 @@ class _DiffEngine {
                        // Use the partitions to split this problem into subproblems.
                        reset( $seps );
                        $pt1 = $seps[0];
-                       while ( $pt2 = next( $seps ) ) {
+                       foreach ( $seps as $pt2 ) {
                                $this->_compareseq ( $pt1[0], $pt2[0], $pt1[1], $pt2[1] );
                                $pt1 = $pt2;
                        }