railt / sdl

src/Compiler.php

Unused Use Statements -2 removed lines

@@ -14,14 +14,12 @@
 use Psr\Log\LoggerInterface;
 use Railt\Io\File;
 use Railt\Io\Readable;
-use Railt\Reflection\Contracts\Document;
 use Railt\Reflection\Contracts\Document as DocumentInterface;
 use Railt\Reflection\Reflection;
 use Railt\SDL\Compiler\Dictionary;
 use Railt\SDL\Compiler\Store;
 use Railt\SDL\Exception\CompilerException;
 use Railt\SDL\Exception\InternalException;
-use Railt\SDL\Frontend\Record\RecordInterface;
 
 /**
  * Class Compiler

src/Frontend.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     /**
      * @param Readable $readable
-     * @return iterable
+     * @return \Generator
      * @throws SyntaxException
      * @throws \Railt\Io\Exception\ExternalFileException
      */

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Psr\Log\LoggerInterface;
 use Railt\Io\Readable;
 use Railt\Parser\Ast\RuleInterface;
-use Railt\Parser\Exception\UnexpectedTokenException;
 use Railt\Parser\Exception\UnrecognizedTokenException;
 use Railt\SDL\Exception\SyntaxException;
 use Railt\SDL\Frontend\Builder;

src/Frontend/Builder.php

Doc Comments +2 added lines, -2 removed lines

@@ -101,8 +101,8 @@
 
     /**
      * @param Readable $file
-     * @param iterable $ast
-     * @return iterable
+     * @param RuleInterface $ast
+     * @return \Generator
      * @throws \Railt\Io\Exception\ExternalFileException
      */
     public function build(Readable $file, iterable $ast): iterable

Unused Use Statements -2 removed lines

@@ -16,11 +16,9 @@
 use Railt\SDL\Frontend;
 use Railt\SDL\Frontend\Context\ContextInterface;
 use Railt\SDL\Frontend\Context\GlobalContext;
-use Railt\SDL\Frontend\Context\GlobalContextInterface;
 use Railt\SDL\IR\SymbolTable;
 use Railt\SDL\IR\SymbolTable\ValueInterface;
 use Railt\SDL\IR\SymbolTableInterface;
-use Railt\SDL\IR\Type\TypeNameInterface;
 
 /**
  * Class Builder

Spacing +5 added lines, -5 removed lines

@@ -116,13 +116,13 @@
         }
 
 
-        echo \str_repeat('=', 60) . "\n";
-        echo \sprintf('| %5s | %-48s |', 'ID', 'VARIABLE') . "\n";
-        echo \str_repeat('-', 60) . "\n";
+        echo \str_repeat('=', 60)."\n";
+        echo \sprintf('| %5s | %-48s |', 'ID', 'VARIABLE')."\n";
+        echo \str_repeat('-', 60)."\n";
         foreach ($this->table as $id => $var) {
-            echo \sprintf('| %5d | %-48s |', $id, $var) . "\n";
+            echo \sprintf('| %5d | %-48s |', $id, $var)."\n";
         }
-        echo \str_repeat('=', 60) . "\n";
+        echo \str_repeat('=', 60)."\n";
     }
 
     /**

src/Frontend/Builder/Instruction/ImportBuilder.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 +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
         $value = yield $rule->getChild(0);
 
         if ($value->getType()->typeOf(Type::string())) {
-            yield from $this->load($this->include($ctx, (string)$value->getValue()));
+            yield from $this->load($this->include($ctx, (string) $value->getValue()));
         }
 
         $error = 'Argument of include should be a string, but %s given';
@@ -96,7 +96,7 @@ 
             return $inclusion;
         }
 
-        $pathname = \dirname($file->getPathname()) . \DIRECTORY_SEPARATOR . $inclusion;
+        $pathname = \dirname($file->getPathname()).\DIRECTORY_SEPARATOR.$inclusion;
         $pathname = \str_replace('/./', '/', $pathname);
 
         return $pathname;

src/Frontend/Builder/Instruction/VariableBuilder.php

Doc Comments +1 added lines, -1 removed lines

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

src/Frontend/Builder/Instruction/VariableReassigmentBuilder.php

Doc Comments +1 added lines, -1 removed lines

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

src/Frontend/Builder/TypeDefinitionBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|Definition
+     * @return \Generator
      * @throws InvalidArgumentException
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule): \Generator
@@ -60,7 +60,7 @@ 
 
     /**
      * @param RuleInterface $argument
-     * @return \Generator|TypeNameInterface
+     * @return \Generator
      */
     private function getArgumentName(RuleInterface $argument): \Generator
     {
@@ -70,7 +70,7 @@ 
 
     /**
      * @param RuleInterface $argument
-     * @return \Generator|TypeNameInterface
+     * @return \Generator
      */
     private function getArgumentValue(RuleInterface $argument): \Generator
     {

Unused Use Statements -5 removed lines

@@ -11,14 +11,9 @@
 
 use Railt\Parser\Ast\RuleInterface;
 use Railt\SDL\Exception\InvalidArgumentException;
-use Railt\SDL\Frontend\AST\TypeNameNode;
 use Railt\SDL\Frontend\Builder\Definition\Definition;
 use Railt\SDL\Frontend\Context\ContextInterface;
-use Railt\SDL\Frontend\Matcher;
-use Railt\SDL\Frontend\Parser;
 use Railt\SDL\IR\SymbolTable\ValueInterface;
-use Railt\SDL\IR\Type;
-use Railt\SDL\IR\Type\Name;
 use Railt\SDL\IR\Type\TypeNameInterface;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
             yield from $name = $this->getArgumentName($argument);
             yield from $value = $this->getArgumentValue($argument);
 
-            $definition->addArgument((string)$name->getReturn(), $value->getReturn());
+            $definition->addArgument((string) $name->getReturn(), $value->getReturn());
         }
 
         return $definition;

src/Frontend/Builder/Value/ScalarValueBuilder.php

Doc Comments +2 added lines, -2 removed lines

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

src/Frontend/Builder/Value/TypeInvocationBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * @param ContextInterface $ctx
      * @param RuleInterface $rule
-     * @return \Generator|ValueInterface
+     * @return \Generator
      */
     public function reduce(ContextInterface $ctx, RuleInterface $rule): \Generator
     {

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         /** @var ValueInterface $value */
         $value = yield $rule->first('> #GenericInvocationName')->getChild(0);
 
-        $name = Name::new((string)$value->getValue());
+        $name = Name::new((string) $value->getValue());
 
         // TODO Add arguments