coding style: fixes in code

Author: dg

src/PhpGenerator/Helpers.php

@@ -231,7 +231,7 @@ public static function isIdentifier($value): bool
 
 	public static function isNamespaceIdentifier($value, bool $allowLeadingSlash = false): bool
 	{
-		$re = '#^' . ($allowLeadingSlash ? '\\\\?' : '') . Helpers::PHP_IDENT . '(\\\\' . Helpers::PHP_IDENT . ')*\z#';
+		$re = '#^' . ($allowLeadingSlash ? '\\\\?' : '') . self::PHP_IDENT . '(\\\\' . self::PHP_IDENT . ')*\z#';
 		return is_string($value) && preg_match($re, $value);
 	}
 

tests/PhpGenerator/ClassType.from.anonymous.phpt

@@ -15,7 +15,7 @@ require __DIR__ . '/../bootstrap.php';
 
 abstract class Class1
 {
-	function func1()
+	public function func1()
 	{
 	}
 }
@@ -26,7 +26,7 @@ $res[] = ClassType::from(new class {
 	private $b;
 
 
-	function a()
+	public function a()
 	{
 	}
 
@@ -37,7 +37,7 @@ $res[] = ClassType::from(new class {
 });
 
 $res[] = ClassType::from(new class extends Class1 {
-	function a()
+	public function a()
 	{
 	}
 });

tests/PhpGenerator/ClassType.from.php71.phpt

@@ -16,17 +16,17 @@ require __DIR__ . '/../bootstrap.php';
 /** */
 class ClassA
 {
-	function func1(A $a, ?B $b, ?C $c = null, D $d = null, E $e, ?int $i = 1, ?array $arr = [])
+	public function func1(A $a, ?B $b, ?C $c = null, D $d = null, E $e, ?int $i = 1, ?array $arr = [])
 	{
 	}
 
 
-	function func2(): ?stdClass
+	public function func2(): ?stdClass
 	{
 	}
 
 
-	function func3(): void
+	public function func3(): void
 	{
 	}
 }

tests/PhpGenerator/ClassType.from.phpt

@@ -22,7 +22,7 @@ require __DIR__ . '/../bootstrap.php';
  */
 interface Interface1
 {
-	function func1();
+	public function func1();
 }
 
 interface Interface2
@@ -32,7 +32,7 @@ interface Interface2
 abstract class Class1 implements Interface1
 {
 	/** @return Class1 */
-	function func1()
+	public function func1()
 	{
 	}
 
@@ -65,7 +65,7 @@ class Class2 extends Class1 implements Interface2
 	}
 
 
-	final function func2()
+	final public function func2()
 	{
 	}
 }

tests/PhpGenerator/ClassType.inheritance.phpt

@@ -16,7 +16,7 @@ class A
 	private $c;
 
 
-	function foo()
+	public function foo()
 	{
 	}
 }
@@ -29,7 +29,7 @@ class B extends A
 	private $f;
 
 
-	function bar()
+	public function bar()
 	{
 		return 3;
 	}

tests/PhpGenerator/GlobalFunction.phpt

@@ -12,7 +12,7 @@ require __DIR__ . '/../bootstrap.php';
 /** global */
 function func(stdClass $a, $b = null)
 {
-};
+}
 
 
 $function = GlobalFunction::from('func');

tests/PhpGenerator/Helpers.comments.phpt

@@ -20,6 +20,6 @@ Assert::same("/**\n * A\n * B\n * C\n */\n", Helpers::formatDocComment("A\nB\nC\
 
 Assert::same('', Helpers::unformatDocComment(''));
 Assert::same('', Helpers::unformatDocComment("/**  */\n\r\t"));
-Assert::same('@var string', Helpers::unformatDocComment(" /** @var string */ "));
+Assert::same('@var string', Helpers::unformatDocComment(' /** @var string */ '));
 Assert::same('@var string', Helpers::unformatDocComment("/**\n * @var string\n */"));
 Assert::same("A\nB\nC", Helpers::unformatDocComment("/**\n * A\n * B\n * C\n */\n"));

tests/PhpGenerator/Helpers.dump().phpt

@@ -35,7 +35,7 @@ Assert::same("'He\\llo'", Helpers::dump('He\llo'));
 Assert::same('\'He\ll\\\\\o \\\'wor\\\\\\\'ld\\\\\'', Helpers::dump('He\ll\\\o \'wor\\\'ld\\'));
 Assert::same('[]', Helpers::dump([]));
 
-Assert::same("[\$s]", Helpers::dump([new PhpLiteral('$s')]));
+Assert::same('[$s]', Helpers::dump([new PhpLiteral('$s')]));
 
 Assert::same('[1, 2, 3]', Helpers::dump([1, 2, 3]));
 Assert::same("['a', 7 => 'b', 'c', '9a' => 'd', 'e']", Helpers::dump(['a', 7 => 'b', 'c', '9a' => 'd', 9 => 'e']));
@@ -59,17 +59,17 @@ Assert::equal(new Test, eval('return ' . Helpers::dump(new Test) . ';'));
 
 class Test2 extends Test
 {
-	private $c = 4;
 	public $d = 5;
+	private $c = 4;
 
 
-	function __sleep()
+	public function __sleep()
 	{
 		return ['c', 'b', 'a'];
 	}
 
 
-	function __wakeup()
+	public function __wakeup()
 	{
 	}
 }
@@ -83,13 +83,13 @@ class Test3 implements Serializable
 	private $a;
 
 
-	function serialize()
+	public function serialize()
 	{
 		return '';
 	}
 
 
-	function unserialize($s)
+	public function unserialize($s)
 	{
 	}
 }

tests/PhpGenerator/Method.returnTypes.phpt

@@ -25,15 +25,15 @@ namespace
 
 	interface A
 	{
-		function testClass(): \A\Foo;
+		public function testClass(): \A\Foo;
 
-		function testScalar(): string;
+		public function testScalar(): string;
 	}
 
-	$method = Method::from(A::class .'::testClass');
+	$method = Method::from(A::class . '::testClass');
 	Assert::same('A\Foo', $method->getReturnType());
 
-	$method = Method::from(A::class .'::testScalar');
+	$method = Method::from(A::class . '::testScalar');
 	Assert::same('string', $method->getReturnType());
 
 	// generating methods with return type declarations

tests/PhpGenerator/Method.variadics.phpt

@@ -22,7 +22,7 @@ interface Variadics
 	function bar($foo, array &...$bar);
 }
 
-$method = Method::from(Variadics::class .'::foo');
+$method = Method::from(Variadics::class . '::foo');
 Assert::true($method->isVariadic());
 
 $method = Method::from(Variadics::class . '::bar');