remove QRMatrix::flip()

Author: codemasher

src/Data/QRMatrix.php

@@ -282,18 +282,6 @@ public function setArea(int $startX, int $startY, int $width, int $height, bool
 		return $this;
 	}
 
-	/**
-	 * Flips the value of the module at ($x, $y)
-	 */
-	public function flip(int $x, int $y):self{
-
-		if(isset($this->matrix[$y][$x])){
-			$this->matrix[$y][$x] ^= $this::IS_DARK;
-		}
-
-		return $this;
-	}
-
 	/**
 	 * Checks whether the module at ($x, $y) is of the given $M_TYPE
 	 *
@@ -718,8 +706,13 @@ public function mask(MaskPattern $maskPattern):self{
 
 		foreach($this->matrix as $y => $row){
 			foreach($row as $x => $val){
-				if($mask($x, $y) && ($val & $this::M_DATA) === $this::M_DATA){
-					$this->flip($x, $y);
+				// skip non-data modules
+				if(($val & $this::M_DATA) !== $this::M_DATA){
+					continue;
+				}
+
+				if($mask($x, $y)){
+					$this->set($x, $y, ($val & $this::IS_DARK) !== $this::IS_DARK, $val);
 				}
 			}
 		}

tests/Data/QRMatrixTest.php

@@ -484,26 +484,6 @@ public function testSetLogoSpaceMaxSizeException():void{
 		(new QRCode($o))->addByteSegment('testdata')->getQRMatrix()->setLogoSpace(37, 37);
 	}
 
-	/**
-	 * Tests flipping the value of a module
-	 */
-	public function testFlip():void{
-		$this->matrix->set(20, 20, true, QRMatrix::M_TEST);
-
-		// cover checkType()
-		$this::assertTrue($this->matrix->checkType(20, 20, QRMatrix::M_TEST));
-		// verify the current state (dark)
-		$this::assertSame(QRMatrix::M_TEST_DARK, $this->matrix->get(20, 20));
-		// flip
-		$this->matrix->flip(20, 20);
-		// verify flip
-		$this::assertSame(QRMatrix::M_TEST, $this->matrix->get(20, 20));
-		// flip again
-		$this->matrix->flip(20, 20);
-		// verify flip
-		$this::assertSame(QRMatrix::M_TEST_DARK, $this->matrix->get(20, 20));
-	}
-
 	/**
 	 * Tests checking whether the M_TYPE of a module is not one of an array of M_TYPES
 	 */