wikipeg 4.0.1 → 4.0.2

package/VERSION

@@ -1 +1 @@
-4.0.1
+4.0.2

package/lib/compiler/language/javascript.js

@@ -225,7 +225,7 @@ let javascript = {
     if (opts.params.length) {
       keyParts = keyParts.concat(opts.params);
     }
-    const storeRefs = opts.storeRefs.filter(function(part) {
+    let storeRefs = opts.storeRefs.filter(function(part) {
       return part !== '';
     }).map(function(part) {
       return '  ' + part;

package/lib/peg.js

@@ -5,7 +5,7 @@ var arrays  = require("./utils/arrays"),
 
 var PEG = {
   /* WikiPEG version (uses semantic versioning). */
-  VERSION: "4.0.1",
+  VERSION: "4.0.2",
 
   GrammarError: require("./grammar-error"),
   parser:       require("./parser"),

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wikipeg",
-	"version": "4.0.1",
+	"version": "4.0.2",
 	"description": "Parser generator for JavaScript and PHP",
 	"license": "MIT",
 	"homepage": "https://gerrit.wikimedia.org/r/plugins/gitiles/wikipeg/",
@@ -40,7 +40,7 @@
 		"url": "https://gerrit.wikimedia.org/r/wikipeg"
 	},
 	"devDependencies": {
-		"eslint": "8.57.0",
+		"eslint": "8.31.0",
 		"jasmine-node": "3.0.0"
 	},
 	"engines": {

package/src/DefaultTracer.php

@@ -5,31 +5,31 @@ namespace Wikimedia\WikiPEG;
 use InvalidArgumentException;
 
 class DefaultTracer implements Tracer {
-	protected int $indentLevel = 0;
+	private $indentLevel = 0;
 
-	public function trace( array $event ): void {
+	public function trace( $event ) {
 		switch ( $event['type'] ) {
-			case 'rule.enter':
-				$this->log( $event );
-				$this->indentLevel++;
-				break;
+		case 'rule.enter':
+			$this->log( $event );
+			$this->indentLevel++;
+			break;
 
-			case 'rule.match':
-				$this->indentLevel--;
-				$this->log( $event );
-				break;
+		case 'rule.match':
+			$this->indentLevel--;
+			$this->log( $event );
+			break;
 
-			case 'rule.fail':
-				$this->indentLevel--;
-				$this->log( $event );
-				break;
+		case 'rule.fail':
+			$this->indentLevel--;
+			$this->log( $event );
+			break;
 
-			default:
-				throw new InvalidArgumentException( "Invalid event type {$event['type']}" );
+		default:
+			throw new InvalidArgumentException( "Invalid event type {$event['type']}" );
 		}
 	}
 
-	protected function log( array $event ) {
+	private function log( $event ) {
 		print str_pad(
 			'' . $event['location'],
 			20
@@ -40,7 +40,7 @@ class DefaultTracer implements Tracer {
 			. "\n";
 	}
 
-	protected function formatArgs( ?array $argMap ): string {
+	private function formatArgs( $argMap ) {
 		if ( !$argMap ) {
 			return '';
 		}

package/src/PEGParserBase.php

@@ -2,31 +2,29 @@
 
 namespace Wikimedia\WikiPEG;
 
-use stdClass;
-
 abstract class PEGParserBase {
-	protected static ?stdClass $FAILED = null;
-	protected static ?stdClass $UNDEFINED = null;
-	protected int $currPos;
-	protected int $savedPos;
-	protected string $input;
-	protected int $inputLength;
-	protected array $options;
-	/** @var array */
+	protected static $FAILED;
+	protected static $UNDEFINED;
+	protected $currPos;
+	protected $savedPos;
+	protected $input;
+	protected $inputLength;
+	protected $options;
 	protected $cache;
 
 	/** @var array<int,array{line:int,column:int,seenCR:bool}> */
-	protected array $posDetailsCache;
-	protected int $maxFailPos;
-	protected array $maxFailExpected;
+	protected $posDetailsCache;
+	protected $maxFailPos;
+	protected $maxFailExpected;
 
 	/** @var array Associative arrays of expectation info */
 	protected $expectations;
 
 	/** @var Expectation[] */
-	private array $expectationCache;
+	private $expectationCache;
 
-	protected Tracer $tracer;
+	/** @var Tracer */
+	protected $tracer;
 
 	public function __construct() {
 		if ( !self::$FAILED ) {
@@ -37,8 +35,7 @@ abstract class PEGParserBase {
 		}
 	}
 
-	/** @return mixed */
-	protected function traceCall( callable $parseFunc, string $name, array $argNames, array $args ) {
+	protected function traceCall( $parseFunc, $name, $argNames, $args ) {
 		$argMap = [];
 		foreach ( $args as $i => $argValue ) {
 			$argMap[$argNames[$i]] = $argValue;
@@ -50,7 +47,7 @@ abstract class PEGParserBase {
 			'location' => $this->computeLocation( $startPos, $startPos ),
 			'args' => $argMap
 		] );
-		$result = $parseFunc( ...$args );
+		$result = call_user_func_array( $parseFunc, $args );
 		if ( $result !== self::$FAILED ) {
 			$this->tracer->trace( [
 				'type' => 'rule.match',
@@ -68,11 +65,11 @@ abstract class PEGParserBase {
 		return $result;
 	}
 
-	protected function text(): string {
+	protected function text() {
 		return substr( $this->input, $this->savedPos, $this->currPos - $this->savedPos );
 	}
 
-	protected function location(): LocationRange {
+	protected function location() {
 		return $this->computeLocation( $this->savedPos, $this->currPos );
 	}
 
@@ -102,7 +99,7 @@ abstract class PEGParserBase {
 		);
 	}
 
-	public static function charAt( string $s, int $byteOffset ): string {
+	public static function charAt( $s, $byteOffset ) {
 		if ( !isset( $s[$byteOffset] ) ) {
 			return '';
 		}
@@ -120,7 +117,7 @@ abstract class PEGParserBase {
 		return $char;
 	}
 
-	public static function charsAt( string $s, int $byteOffset, int $numChars ): string {
+	public static function charsAt( $s, $byteOffset, $numChars ) {
 		$ret = '';
 		for ( $i = 0; $i < $numChars; $i++ ) {
 			$ret .= self::consumeChar( $s, $byteOffset );
@@ -128,7 +125,7 @@ abstract class PEGParserBase {
 		return $ret;
 	}
 
-	public static function consumeChar( string $s, int &$byteOffset ): string {
+	public static function consumeChar( $s, &$byteOffset ) {
 		if ( !isset( $s[$byteOffset] ) ) {
 			return '';
 		}
@@ -146,10 +143,6 @@ abstract class PEGParserBase {
 		return $char;
 	}
 
-	/**
-	 * @param mixed $value
-	 * @return mixed
-	 */
 	public static function &newRef( $value ) {
 		return $value;
 	}
@@ -172,8 +165,7 @@ abstract class PEGParserBase {
 		while ( $p < $pos ) {
 			$ch = self::charAt( $this->input, $p );
 			if ( $ch === "\n" ) {
-				if ( !$details['seenCR'] ) {
-					$details['line']++;
+				if ( !$details['seenCR'] ) { $details['line']++;
 				}
 				$details['column'] = 1;
 				$details['seenCR'] = false;
@@ -193,7 +185,7 @@ abstract class PEGParserBase {
 		return $details;
 	}
 
-	protected function computeLocation( int $startPos, int $endPos ): LocationRange {
+	protected function computeLocation( $startPos, $endPos ) {
 		if ( $endPos > $this->inputLength ) {
 			$endPos--;
 		}
@@ -210,7 +202,7 @@ abstract class PEGParserBase {
 		);
 	}
 
-	protected function fail( int $expected ) {
+	protected function fail( $expected ) {
 		if ( $this->currPos < $this->maxFailPos ) {
 			return;
 		}
@@ -242,7 +234,7 @@ abstract class PEGParserBase {
 		return $expanded;
 	}
 
-	private function buildMessage( array $expected, ?string $found ): string {
+	private function buildMessage( $expected, $found ) {
 		$expectedDescs = [];
 
 		foreach ( $expected as $info ) {
@@ -259,9 +251,7 @@ abstract class PEGParserBase {
 		return "Expected " . $expectedDesc . " but " . $foundDesc . " found.";
 	}
 
-	protected function buildException(
-		?string $message, ?array $expected, ?string $found, LocationRange $location
-	): SyntaxError {
+	protected function buildException( $message, $expected, $found, $location ) {
 		if ( $expected !== null ) {
 			sort( $expected );
 			$expected = array_unique( $expected );
@@ -281,7 +271,7 @@ abstract class PEGParserBase {
 		);
 	}
 
-	protected function buildParseException(): SyntaxError {
+	protected function buildParseException() {
 		$char = self::charAt( $this->input, $this->maxFailPos );
 		return $this->buildException(
 			null,
@@ -294,7 +284,7 @@ abstract class PEGParserBase {
 	protected function initialize() {
 	}
 
-	protected function initInternal( string $input, array $options ) {
+	protected function initInternal( $input, $options ) {
 		$this->currPos = 0;
 		$this->savedPos = 0;
 		$this->input = $input;
@@ -309,6 +299,5 @@ abstract class PEGParserBase {
 		$this->initialize();
 	}
 
-	/** @return mixed */
-	abstract public function parse( string $input, array $options = [] );
+	abstract public function parse( $input, $options = [] );
 }

package/src/SyntaxError.php

@@ -3,9 +3,9 @@
 namespace Wikimedia\WikiPEG;
 
 class SyntaxError extends \Exception implements \JsonSerializable {
-	public array $expected;
-	public ?string $found;
-	public LocationRange $location;
+	public $expected;
+	public $found;
+	public $location;
 
 	/**
 	 * @param string $message
@@ -13,7 +13,7 @@ class SyntaxError extends \Exception implements \JsonSerializable {
 	 * @param string|null $found
 	 * @param LocationRange $location
 	 */
-	public function __construct( string $message, array $expected, ?string $found, LocationRange $location ) {
+	public function __construct( string $message, array $expected, $found, LocationRange $location ) {
 		parent::__construct( $message );
 		$this->expected = $expected;
 		$this->found    = $found;

package/src/Tracer.php

@@ -3,5 +3,5 @@
 namespace Wikimedia\WikiPEG;
 
 interface Tracer {
-	public function trace( array $event ): void;
+	public function trace( $event );
 }