kadet1090 / KeyLighter

Utils/ConsoleHelper.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 
     private function _set($style)
     {
-        $escape = "\e[".implode(';', array_map(function ($style, $name) {
+        $escape = "\e[".implode(';', array_map(function($style, $name) {
             return $this->_style($style, $name);
         }, array_keys($style), $style)).'m';
 

Language/GreedyLanguage.php

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
         $result   = new Result($tokens->getSource(), $tokens->current());
 
         for ($tokens->next(); $tokens->valid(); $tokens->next()) {
-            if(!$tokens->current()->process($context, $this, $result, $tokens)) {
+            if (!$tokens->current()->process($context, $this, $result, $tokens)) {
                 break;
             }
         }
@@ -147,7 +147,7 @@ 
     private function _rules($embedded = false)
     {
         $rules = clone $this->rules;
-        if(is_bool($embedded)) {
+        if (is_bool($embedded)) {
             $rules->addMany(['language.'.$this->getIdentifier() => $this->getEnds($embedded)]);
         }
 

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
     /**
      * @param bool $embedded
      *
-     * @return Rule[]
+     * @return \Generator
      */
     private function _rules($embedded = false)
     {

Language/Language.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
 
     public final function getFQN($class = false)
     {
-        $embedded =$this->getEmbedded();
+        $embedded = $this->getEmbedded();
         return  ($class ? get_class($this) : $this->getIdentifier()).(
             !empty($embedded)
                 ? ' + '.implode(', ', array_map($class ? 'get_class' : function(Language $e) { return $e->getIdentifier(); }, $embedded))

bin/Application.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
     protected function getCommandName(InputInterface $input)
     {
         $command = $input->getFirstArgument();
-        if(!$command && !$input->hasParameterOption('--help')) {
+        if (!$command && !$input->hasParameterOption('--help')) {
             return 'list';
         } elseif ($this->has($command)) {
             return $command;
@@ -56,7 +56,7 @@ 
     protected function getDefaultInputDefinition()
     {
         $input = parent::getDefaultInputDefinition();
-        $input->setOptions(array_filter($input->getOptions(), function (InputOption $option) {
+        $input->setOptions(array_filter($input->getOptions(), function(InputOption $option) {
             return $option->getShortcut() != 'q';
         }));
         $input->addOption(new InputOption('no-output', 's', InputOption::VALUE_NONE, 'Disables output, useful for debug'));

bin/Commands/FormattersCommand.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $formatters = KeyLighter::get()->registeredFormatters();
         $table = new Table($output);
 
-        if(!$input->getOption('headerless')) {
+        if (!$input->getOption('headerless')) {
             $table->setHeaders(['Name', 'Formatter']);
         }
 

bin/Commands/LanguagesCommand.php

Spacing +5 added lines, -5 removed lines

@@ -46,13 +46,13 @@ 
 
         $table = new Table($output);
 
-        if(!$input->getOption('headerless')) {
+        if (!$input->getOption('headerless')) {
             $table->setHeaders([ucfirst($input->getArgument('by')), $input->getOption('classes') ? 'Class name' : 'Language']);
         }
 
         $table->setRows(array_map(function($language) {
             return [
-                implode(', ', array_map(function ($f) { return "<comment>{$f}</comment>"; }, $language['aliases'])),
+                implode(', ', array_map(function($f) { return "<comment>{$f}</comment>"; }, $language['aliases'])),
                 $language['class']
             ];
         }, $input->getOption('no-group') ? $this->processNonGrouped($languages) : $this->processGrouped($languages)));
@@ -63,8 +63,8 @@ 
 
     protected function processGrouped($languages) {
         $result = [];
-        foreach($languages as $alias => $class) {
-            if(!isset($result[$class])) {
+        foreach ($languages as $alias => $class) {
+            if (!isset($result[$class])) {
                 $result[$class] = ['aliases' => [], 'class' => $class];
             }
 
@@ -76,7 +76,7 @@ 
 
     protected function processNonGrouped($languages) {
         $result = [];
-        foreach($languages as $alias => $class) {
+        foreach ($languages as $alias => $class) {
             $result[] = ['aliases' => [$alias], 'class' => $class];
         }
 

bin/VerboseOutput.php

Spacing +9 added lines, -9 removed lines

@@ -68,14 +68,14 @@ 
         $parsed    = $this->parse($tokenized);
         $formatted = $this->format($parsed);
 
-        if($this->wants('time')) {
+        if ($this->wants('time')) {
             $this->_output->writeln(sprintf(
                 '<info>Overall:</info> %.4fs, %s chars/s',
                 array_sum($this->_times), number_format(strlen($this->_source) / array_sum($this->_times))
             ));
         }
 
-        if($this->wants('detailed-time')) {
+        if ($this->wants('detailed-time')) {
             $this->_slashed('Times             [s]', array_map(function($t) {
                 return number_format($t, 5);
             }, $this->_times));
@@ -84,11 +84,11 @@ 
             }, $this->_times));
         }
 
-        if($this->wants('count')) {
+        if ($this->wants('count')) {
             $this->_slashed('Token count', array_map('number_format', $this->_counts));
         }
 
-        if($this->wants('density')) {
+        if ($this->wants('density')) {
             $this->_slashed('Token density [tokens/kB]', array_map(function($c) {
                 return number_format($c / strlen($this->_source) * 1024, 1);
             }, $this->_counts));
@@ -104,7 +104,7 @@ 
 
     protected function tokenize()
     {
-        $tokens = $this->benchmark(function () {
+        $tokens = $this->benchmark(function() {
             return $this->_language->tokenize($this->_source);
         }, $this->_times['tokenization']);
 
@@ -133,7 +133,7 @@ 
 
     protected function parse(Tokens $tokens)
     {
-        $tokens = $this->benchmark(function () use ($tokens) {
+        $tokens = $this->benchmark(function() use ($tokens) {
             return $this->_language->parse($tokens);
         }, $this->_times['parsing']);
 
@@ -147,7 +147,7 @@ 
 
     protected function format(Tokens $tokens)
     {
-        return $this->benchmark(function () use ($tokens) {
+        return $this->benchmark(function() use ($tokens) {
             return $this->_formatter->format($tokens);
         }, $this->_times['formatting']);
     }
@@ -156,10 +156,10 @@ 
     {
         $this->_output->writeln(sprintf(
             "$message %s: %s",
-            implode(' / ', array_map(function ($f) {
+            implode(' / ', array_map(function($f) {
                 return "<comment>{$f}</comment>";
             }, array_keys($data))),
-            implode(' / ', array_map(function ($f) {
+            implode(' / ', array_map(function($f) {
                 return "<info>{$f}</info>";
             }, array_values($data)))
         ));

bin/Commands/Dev/GenerateMetadataCommand.php

Spacing +4 added lines, -4 removed lines

@@ -26,11 +26,11 @@ 
 {
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        if($input->getOption('dry')) {
+        if ($input->getOption('dry')) {
             $output->writeln($this->generate($output));
         } else {
             file_put_contents(
-                __DIR__ . '/../../../Config/metadata.php',
+                __DIR__.'/../../../Config/metadata.php',
                 '<?php return '.$this->generate($output).';'
             );
         }
@@ -47,7 +47,7 @@ 
 
     protected function generate(OutputInterface $output)
     {
-        $dir = __DIR__ . '/../../../Language'.DIRECTORY_SEPARATOR;
+        $dir = __DIR__.'/../../../Language'.DIRECTORY_SEPARATOR;
         $iterator = new \RecursiveIteratorIterator(
             new \RecursiveDirectoryIterator($dir, \RecursiveDirectoryIterator::SKIP_DOTS),
             \RecursiveIteratorIterator::LEAVES_ONLY
@@ -65,7 +65,7 @@ 
             ), OutputInterface::VERBOSITY_VERBOSE);
 
 
-            if($metadata = $this->process($output, $class)) {
+            if ($metadata = $this->process($output, $class)) {
                 $result[] = $metadata;
             }
         }

KeyLighter.php

Spacing +8 added lines, -10 removed lines

@@ -70,21 +70,19 @@ 
     public function languageByName($name, $params = [])
     {
         return isset($this->_languages['name'][$name]) ?
-            $this->_languages['name'][$name]($params) :
-            new PlainText($params);
+            $this->_languages['name'][$name]($params) : new PlainText($params);
     }
 
     public function languageByMime($mime, $params = [])
     {
         return isset($this->_languages['mime'][$mime]) ?
-            $this->_languages['mime'][$mime]($params) :
-            new PlainText($params);
+            $this->_languages['mime'][$mime]($params) : new PlainText($params);
     }
 
     public function languageByExt($filename, $params = [])
     {
-        foreach($this->_languages['extension'] as $mask => $class) {
-            if(fnmatch($mask, $filename)) {
+        foreach ($this->_languages['extension'] as $mask => $class) {
+            if (fnmatch($mask, $filename)) {
                 return $class($params);
             }
         }
@@ -110,7 +108,7 @@ 
 
     public function registeredLanguages($by = 'name', $class = false)
     {
-        return array_map(function ($e) use($class) {
+        return array_map(function($e) use($class) {
             return $e([])->getFQN($class);
         }, $this->_languages[$by]);
     }
@@ -150,7 +148,7 @@ 
 
     public function init()
     {
-        foreach(include __DIR__.'/Config/metadata.php' as $alias) {
+        foreach (include __DIR__.'/Config/metadata.php' as $alias) {
             $class = $alias[0];
             unset($alias[0]);
 
@@ -166,13 +164,13 @@ 
      */
     public function register($class, array $options)
     {
-        if(!is_callable($class) && is_subclass_of($class, Language::class)) {
+        if (!is_callable($class) && is_subclass_of($class, Language::class)) {
             $class = function($arguments = []) use ($class) {
                 return new $class($arguments);
             };
         }
 
-        foreach(array_intersect_key($options, array_flip(['name', 'mime', 'extension'])) as $name => $values) {
+        foreach (array_intersect_key($options, array_flip(['name', 'mime', 'extension'])) as $name => $values) {
             $this->_languages[$name] = array_merge($this->_languages[$name], array_fill_keys($values, $class));
         }
     }

Doc Comments +3 added lines

@@ -135,6 +135,9 @@
         return $this->_formatters;
     }
 
+    /**
+     * @return string
+     */
     public function highlight($source, Language $language, FormatterInterface $formatter = null)
     {
         $formatter = $formatter ?: $this->getDefaultFormatter();