railt / sdl

src/Frontend/Builder.php

Doc Comments +1 added lines, -1 removed lines

@@ -243,7 +243,7 @@
 
     /**
      * @param Readable $readable
-     * @param iterable|RuleInterface[] $rules
+     * @param RuleInterface $rules
      * @return iterable|array[]
      * @throws \Railt\Io\Exception\ExternalFileException
      */

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 use Railt\Io\Readable;
 use Railt\Parser\Ast\RuleInterface;
-use Railt\Parser\Exception\UnexpectedTokenException;
 use Railt\Parser\Exception\UnrecognizedTokenException;
 use Railt\SDL\Exception\InternalException;
 use Railt\SDL\Exception\SyntaxException;

Spacing +3 added lines, -3 removed lines

@@ -140,7 +140,7 @@ 
      */
     private function bootInterceptors(DeferredStorage $deferred, TypesStorage $types): Factory
     {
-        $wantsBuild = function (ContextInterface $ctx, RuleInterface $ast) {
+        $wantsBuild = function(ContextInterface $ctx, RuleInterface $ast) {
             return $this->buildNode($ctx, $ast);
         };
 
@@ -278,10 +278,10 @@ 
      */
     private function deferred(ContextInterface $context): ContextInterface
     {
-        $this->deferred->attach($this->types->export(function (DefinitionInterface $definition): bool {
+        $this->deferred->attach($this->types->export(function(DefinitionInterface $definition): bool {
             $invocation = new Invocation($definition->getName(), $definition->getContext());
 
-            return ! $definition->isGeneric() && ! $this->types->resolved($invocation);
+            return !$definition->isGeneric() && !$this->types->resolved($invocation);
         }));
 
         [$context] = $this->process->run($context, $this->deferred->getIterator());

src/Frontend/Builder/Definition/ObjectDefinitionBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return mixed|\Generator|void
+     * @return \Generator
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule)
     {

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         /** @var DeferredInterface $definition */
         $definition = yield $rule->first('> #TypeDefinition');
 
-        $definition->then(function (DefinitionInterface $definition, InvocationInterface $from) {
+        $definition->then(function(DefinitionInterface $definition, InvocationInterface $from) {
             /** @var ContextInterface $local */
             $local = yield $definition->getLocalContext(); // Open local context
 

src/Frontend/Builder/Instruction/VariableBuilder.php

Doc Comments +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|\Closure
+     * @return \Generator
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule): \Generator
     {
@@ -66,7 +66,7 @@ 
 
     /**
      * @param RuleInterface $rule
-     * @return iterable|string[]
+     * @return \Generator
      */
     private function getVariableNames(RuleInterface $rule): iterable
     {

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
             $isConstant ? $record->lock() : $record->unlock();
         }
 
-        yield function () use ($rule, $variables) {
+        yield function() use ($rule, $variables) {
             /** @var ValueInterface $value */
             $value = yield $this->getValueNode($rule);
 

src/Frontend/Builder/Value/NullValueBuilder.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
 
     /**
      * @param ContextInterface $ctx
-     * @param RuleInterface|AstValueInterface $rule
-     * @return mixed|Value
+     * @param RuleInterface $rule
+     * @return Value
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule)
     {

src/Frontend/Builder/Value/StringValueBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return mixed|Value
+     * @return ValueInterface
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule): ValueInterface
     {

src/Frontend/Builder/Value/TypeInvocationBuilder.php

Doc Comments +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|InvocationInterface
+     * @return \Generator
      * @throws TypeConflictException
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule): \Generator
@@ -67,7 +67,7 @@ 
 
     /**
      * @param ContextInterface $ctx
-     * @param TypeNameInterface|ValueInterface $name
+     * @param ValueInterface $name
      * @return InvocationInterface
      */
     private function resolveName(ContextInterface $ctx, $name): InvocationInterface
@@ -95,7 +95,7 @@ 
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|TypeNameInterface
+     * @return \Generator
      * @throws TypeConflictException
      */
     private function fetchArgumentName(ContextInterface $ctx, RuleInterface $rule): \Generator
@@ -123,7 +123,7 @@ 
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|ValueInterface
+     * @return \Generator
      */
     private function fetchArgumentValue(ContextInterface $ctx, RuleInterface $rule): \Generator
     {

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
 
         switch (true) {
             case $isConst:
-                return new Invocation(Name::fromString((string)$name->getValue()), $ctx);
+                return new Invocation(Name::fromString((string) $name->getValue()), $ctx);
 
             case $isType:
                 /** @var Invocation $value */
@@ -109,7 +109,7 @@ 
 
         $isConst = $name->getType()->typeOf(Type::const());
 
-        if (! $isConst) {
+        if (!$isConst) {
             $error = 'Generic argument name must be a const but %s given';
             $exception = new TypeConflictException(\sprintf($error, $name));
             $exception->throwsIn($ctx->getFile(), $rule->getOffset());
@@ -117,7 +117,7 @@ 
             throw $exception;
         }
 
-        return Name::fromString((string)$name->getValue());
+        return Name::fromString((string) $name->getValue());
     }
 
     /**
@@ -132,7 +132,7 @@ 
 
         $isType = $value->getType()->typeOf(Type::type());
 
-        if (! $isType) {
+        if (!$isType) {
             $error = 'Generic argument value must be a valid type, but %s given';
             $exception = new TypeConflictException(\sprintf($error, $value));
             $exception->throwsIn($ctx->getFile(), $rule->getOffset());

src/Frontend/Deferred/Deferred.php

Doc Comments +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 
     /**
      * @param mixed ...$args
-     * @return \Generator|mixed
+     * @return \Generator
      */
     public function __invoke(...$args)
     {
@@ -54,7 +54,7 @@ 
 
     /**
      * @param array $args
-     * @return \Generator|mixed
+     * @return \Generator
      */
     public function invoke(...$args): \Generator
     {

Unused Use Statements -2 removed lines

@@ -9,8 +9,6 @@
 
 namespace Railt\SDL\Frontend\Deferred;
 
-use Railt\SDL\Frontend\Context\ContextInterface;
-
 /**
  * Class Deferred
  */

src/Frontend/Deferred/Storage.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     }
 
     /**
-     * @return iterable|\Generator
+     * @return \Generator
      */
     public function getIterator(): iterable
     {

src/Frontend/Definition/Storage.php

Doc Comments +1 added lines, -1 removed lines

@@ -144,7 +144,7 @@
 
     /**
      * @param \Closure $filter
-     * @return \Generator|DeferredInterface[]
+     * @return \Generator
      */
     public function export(\Closure $filter): \Generator
     {

Unused Use Statements -2 removed lines

@@ -9,9 +9,7 @@
 
 namespace Railt\SDL\Frontend\Definition;
 
-use Railt\Reflection\Contracts\Definition\TypeDefinition;
 use Railt\SDL\Exception\NotFoundException;
-use Railt\SDL\Exception\TypeConflictException;
 use Railt\SDL\Frontend\Context\ContextInterface;
 use Railt\SDL\Frontend\Deferred\Deferred;
 use Railt\SDL\Frontend\Deferred\DeferredInterface;

Spacing +3 added lines, -3 removed lines

@@ -53,11 +53,11 @@ 
     {
         $this->naming = $naming;
 
-        $this->definitions = new Map(function (TypeNameInterface $name): string {
+        $this->definitions = new Map(function(TypeNameInterface $name): string {
             return $name->getFullyQualifiedName();
         });
 
-        $this->resolvers = new Map(function (DefinitionInterface $definition): string {
+        $this->resolvers = new Map(function(DefinitionInterface $definition): string {
             return $definition->getName()->getFullyQualifiedName();
         });
     }
@@ -176,7 +176,7 @@ 
      */
     private function toInvocationDeferredCallback(DefinitionInterface $original, DeferredInterface $then): \Closure
     {
-        return function (DefinitionInterface $definition = null, InvocationInterface $invocation = null) use (
+        return function(DefinitionInterface $definition = null, InvocationInterface $invocation = null) use (
             $original,
             $then
         ) {