Update code style.

.github/workflows/continuous-integration.yml

@@ -28,6 +28,7 @@ jobs:
                   php-version: ${{ matrix.php-versions }}
                   extensions: ast,mbstring,xdebug
                   tools: pecl
+
             - name: Get Composer Cache Directory
               id: composer-cache
               run: echo "::set-output name=dir::$(composer config cache-files-dir)"

spec/loophp/phptree/Node/NaryNodeSpec.php

@@ -69,14 +69,11 @@ public function it_can_have_children(): void
 
     public function it_can_throw_an_error_when_capacity_is_invalid(): void
     {
-        $this->beConstructedWith(null);
+        $this->beConstructedWith();
 
         $this
             ->capacity()
-            ->shouldBeNull();
-
-        $this->shouldThrow(Exception::class)
-            ->during('add', [new NaryNode()]);
+            ->shouldBe(0);
     }
 
     public function it_can_use_a_different_traverser(): void

src/Builder/BuilderInterface.php

@@ -13,8 +13,6 @@ interface BuilderInterface
 {
     /**
      * @param iterable<mixed> $nodes
-     *
-     * @return NodeInterface|null
      */
     public static function create(iterable $nodes): ?NodeInterface;
 }

src/Builder/Random.php

@@ -39,8 +39,6 @@ public static function create(iterable $nodes): ?NodeInterface
 
     /**
      * @param array<mixed> $parameters
-     *
-     * @return \loophp\phptree\Node\NodeInterface
      */
     private static function createNode(array $parameters = []): NodeInterface
     {
@@ -60,11 +58,6 @@ static function ($parameter) {
         return new $class(...$parameters);
     }
 
-    /**
-     * @param \loophp\phptree\Node\NodeInterface $node
-     *
-     * @return \loophp\phptree\Node\NodeInterface
-     */
     private static function pickRandomNode(NodeInterface $node): NodeInterface
     {
         $pick = (int) random_int(0, $node->count());

src/Exporter/Graph.php

@@ -86,9 +86,6 @@ private function createVertexId(NodeInterface $node): string
         return sha1(spl_object_hash($node));
     }
 
-    /**
-     * @return \Fhaculty\Graph\Graph
-     */
     private function getGraph(): OriginalGraph
     {
         return $this->graph;

src/Exporter/Gv.php

@@ -178,8 +178,6 @@ private function findEdges(NodeInterface $node): iterable
     /**
      * Get the default GV file content.
      *
-     * @param string $attributes
-     * @param string $nodes
      * @param string $edges
      *   The edges.
      *

src/Exporter/Image.php

@@ -35,11 +35,7 @@ public function __construct()
     }
 
     /**
-     * @param NodeInterface $node
-     *
      * @throws Exception
-     *
-     * @return string
      */
     public function export(NodeInterface $node): string
     {
@@ -54,17 +50,12 @@ public function export(NodeInterface $node): string
 
     /**
      * Get the executable to use.
-     *
-     * @return string
      */
     public function getExecutable(): string
     {
         return $this->executable;
     }
 
-    /**
-     * @return string
-     */
     public function getFormat(): string
     {
         return $this->format;
@@ -73,8 +64,6 @@ public function getFormat(): string
     /**
      * Change the executable to use.
      *
-     * @param string $executable
-     *
      * @return \loophp\phptree\Exporter\Image
      */
     public function setExecutable(string $executable): self
@@ -85,8 +74,6 @@ public function setExecutable(string $executable): self
     }
 
     /**
-     * @param string $format
-     *
      * @return \loophp\phptree\Exporter\Image
      */
     public function setFormat(string $format): self
@@ -96,11 +83,6 @@ public function setFormat(string $format): self
         return $this;
     }
 
-    /**
-     * @param string $path
-     *
-     * @return string
-     */
     private function getConvertCommand(string $path): string
     {
         return sprintf(

src/Importer/MicrosoftTolerantPhpParser.php

@@ -20,29 +20,19 @@ final class MicrosoftTolerantPhpParser implements ImporterInterface
      * @param SourceFileNode $data
      *
      * @throws Exception
-     *
-     * @return NodeInterface
      */
     public function import($data): NodeInterface
     {
         return $this->parseNode($this->createNode(['label' => 'root']), $data);
     }
 
-    /**
-     * @param array $attributes
-     *
-     * @return AttributeNodeInterface
-     */
     private function createNode(array $attributes): AttributeNodeInterface
     {
         return new AttributeNode($attributes);
     }
 
     /**
-     * @param AttributeNodeInterface $parent
      * @param Node ...$astNodes
-     *
-     * @return NodeInterface
      */
     private function parseNode(AttributeNodeInterface $parent, Node ...$astNodes): NodeInterface
     {

src/Importer/NikicPhpAst.php

@@ -26,8 +26,6 @@ final class NikicPhpAst implements ImporterInterface
      * @param Node $data
      *
      * @throws Exception
-     *
-     * @return NodeInterface
      */
     public function import($data): NodeInterface
     {
@@ -36,21 +34,13 @@ public function import($data): NodeInterface
         return $this->parseNode($this->createNode(['label' => 'root']), $data);
     }
 
-    /**
-     * @param array $attributes
-     *
-     * @return AttributeNodeInterface
-     */
     private function createNode(array $attributes): AttributeNodeInterface
     {
         return new AttributeNode($attributes);
     }
 
     /**
-     * @param AttributeNodeInterface $parent
      * @param Node ...$astNodes
-     *
-     * @return NodeInterface
      */
     private function parseNode(AttributeNodeInterface $parent, Node ...$astNodes): NodeInterface
     {

src/Importer/NikicPhpParser.php

@@ -21,27 +21,18 @@ final class NikicPhpParser implements ImporterInterface
      * @param Node[] $data
      *
      * @throws Exception
-     *
-     * @return NodeInterface
      */
     public function import($data): NodeInterface
     {
         return $this->parseNode($this->createNode(['label' => 'root']), ...$data);
     }
 
-    /**
-     * @param array $attributes
-     *
-     * @return AttributeNodeInterface
-     */
     private function createNode(array $attributes): AttributeNodeInterface
     {
         return new AttributeNode($attributes);
     }
 
     /**
-     * @param \PhpParser\Node $astNode
-     *
      * @return array<int, Node>
      */
     private function getAllNodeChildren(Node $astNode): array
@@ -69,10 +60,7 @@ static function (string $subNodeName) use ($astNode): array {
     }
 
     /**
-     * @param AttributeNodeInterface $parent
      * @param Node ...$astNodes
-     *
-     * @return NodeInterface
      */
     private function parseNode(AttributeNodeInterface $parent, Node ...$astNodes): NodeInterface
     {

src/Importer/SimpleArray.php

@@ -38,10 +38,7 @@ private function createNode($data): AttributeNodeInterface
     }
 
     /**
-     * @param AttributeNodeInterface $parent
      * @param array ...$nodes
-     *
-     * @return \loophp\phptree\Node\NodeInterface
      */
     private function parseNode(AttributeNodeInterface $parent, array ...$nodes): NodeInterface
     {

src/Importer/Text.php

@@ -71,10 +71,7 @@ static function (string $match): string {
     }
 
     /**
-     * @param AttributeNodeInterface $parent
      * @param string ...$nodes
-     *
-     * @return \loophp\phptree\Node\NodeInterface
      */
     private function parseNode(AttributeNodeInterface $parent, string ...$nodes): NodeInterface
     {

src/Modifier/Apply.php

@@ -8,9 +8,6 @@
 use loophp\phptree\Traverser\PostOrder;
 use loophp\phptree\Traverser\TraverserInterface;
 
-/**
- * Class Apply.
- */
 class Apply implements ModifierInterface
 {
     /**
@@ -19,15 +16,12 @@ class Apply implements ModifierInterface
     private $apply;
 
     /**
-     * @var TraverserInterface|null
+     * @var TraverserInterface
      */
     private $traverser;
 
     /**
      * Apply constructor.
-     *
-     * @param callable $apply
-     * @param TraverserInterface|null $traverser
      */
     public function __construct(callable $apply, ?TraverserInterface $traverser = null)
     {

src/Modifier/Filter.php

@@ -26,9 +26,6 @@ class Filter implements ModifierInterface
 
     /**
      * Filter constructor.
-     *
-     * @param callable $filter
-     * @param TraverserInterface|null $traverser
      */
     public function __construct(callable $filter, ?TraverserInterface $traverser = null)
     {

src/Modifier/FulfillCapacity.php

@@ -22,8 +22,6 @@ class FulfillCapacity implements ModifierInterface
 
     /**
      * FulfillCapacity constructor.
-     *
-     * @param TraverserInterface|null $traverser
      */
     public function __construct(?TraverserInterface $traverser = null)
     {

src/Modifier/RemoveNullNode.php

@@ -22,8 +22,6 @@ class RemoveNullNode implements ModifierInterface
 
     /**
      * RemoveNullNode constructor.
-     *
-     * @param TraverserInterface|null $traverser
      */
     public function __construct(?TraverserInterface $traverser = null)
     {

src/Node/ABNode.php

@@ -40,8 +40,9 @@ public function add(NodeInterface ...$nodes): NodeInterface
 
             ksort($count);
 
-            $child = array_shift($count);
-            $child->add($node);
+            if (null !== $child = array_shift($count)) {
+                $child->add($node);
+            }
         }
 
         return $this;

src/Node/AttributeNode.php

@@ -6,27 +6,21 @@
 
 use loophp\phptree\Traverser\TraverserInterface;
 
-/**
- * Class AttributeNode.
- */
 class AttributeNode extends NaryNode implements AttributeNodeInterface
 {
     /**
      * @var array
      */
-    private $attributes = [];
+    private $attributes;
 
     /**
      * ValueNode constructor.
      *
      * @param array<int|string, mixed> $attributes
-     * @param int|null $capacity
-     * @param TraverserInterface|null $traverser
-     * @param NodeInterface|null $parent
      */
     public function __construct(
         array $attributes = [],
-        ?int $capacity = 0,
+        int $capacity = 0,
         ?TraverserInterface $traverser = null,
         ?NodeInterface $parent = null
     ) {

src/Node/AttributeNodeInterface.php

@@ -12,8 +12,6 @@ interface AttributeNodeInterface extends NaryNodeInterface
     /**
      * Get an attribute.
      *
-     * @param string $key
-     *
      * @return mixed
      *   The value of the attribute.
      */

src/Node/KeyValueNode.php

@@ -21,14 +21,11 @@ class KeyValueNode extends ValueNode implements KeyValueNodeInterface
      *
      * @param mixed|null $key
      * @param mixed|null $value
-     * @param int|null $capacity
-     * @param TraverserInterface|null $traverser
-     * @param NodeInterface|null $parent
      */
     public function __construct(
         $key,
         $value,
-        ?int $capacity = 0,
+        int $capacity = 0,
         ?TraverserInterface $traverser = null,
         ?NodeInterface $parent = null
     ) {

src/Node/MerkleNode.php

@@ -29,7 +29,6 @@ class MerkleNode extends ValueNode implements MerkleNodeInterface
      * MerkleNode constructor.
      *
      * @param mixed $value
-     * @param int $capacity
      * @param HasherInterface $hasher
      */
     public function __construct(

src/Node/MerkleNodeInterface.php

@@ -9,9 +9,6 @@
  */
 interface MerkleNodeInterface extends ValueNodeInterface
 {
-    /**
-     * @return string
-     */
     public function hash(): string;
 
     /**