kadet1090 / KeyLighter

Language/Http.php

Spacing +2 added lines, -2 removed lines

@@ -54,8 +54,8 @@
         $split = explode("\r\n\r\n", $source);
 
         $http = $split[0];
-        if(isset($split[1]) && $payload = $split[1]) {
-            if(preg_match('/Content-Type: ([^;]*)/', $http, $matches)) {
+        if (isset($split[1]) && $payload = $split[1]) {
+            if (preg_match('/Content-Type: ([^;]*)/', $http, $matches)) {
                 $mime = $matches[1];
             } else {
                 $mime = 'text/plain';

KeyLighter.php

Spacing +8 added lines, -10 removed lines

@@ -68,21 +68,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);
             }
         }
@@ -108,7 +106,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]);
     }
@@ -133,7 +131,7 @@ 
 
     public function init()
     {
-        foreach(include __DIR__.'/Config/aliases.php' as $alias) {
+        foreach (include __DIR__.'/Config/aliases.php' as $alias) {
             $class = $alias[0];
             unset($alias[0]);
 
@@ -147,13 +145,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($options as $name => $values) {
+        foreach ($options as $name => $values) {
             $this->_languages[$name] = array_merge($this->_languages[$name], array_fill_keys($values, $class));
         }
     }

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];
         }
 

Parser/Validator/Validator.php

Spacing +15 added lines, -15 removed lines

@@ -44,8 +44,8 @@ 
 
     public function setRules($rules)
     {
-        if(empty($rules)) {
-            $this->_rules = [ 'none' => Validator::CONTEXT_IN_ONE_OF ];
+        if (empty($rules)) {
+            $this->_rules = ['none' => Validator::CONTEXT_IN_ONE_OF];
         } else {
             foreach ($rules as $key => $rule) {
                 list($plain, $type)   = $this->_parse($rule);
@@ -57,8 +57,8 @@ 
     private function _clean($rule, &$required)
     {
         if (strpos($rule, '.') !== false) {
-            foreach (array_filter(array_keys($required), function ($key) use ($rule) {
-                return fnmatch($key . '.*', $rule);
+            foreach (array_filter(array_keys($required), function($key) use ($rule) {
+                return fnmatch($key.'.*', $rule);
             }) as $remove) {
                 unset($required[$remove]);
             }
@@ -66,11 +66,11 @@ 
     }
 
     protected function _validate($context, $rules, $result = false) {
-        if(empty($context)) {
+        if (empty($context)) {
             $context = ['none'];
         }
 
-        while(list($rule, $type) = each($rules)) {
+        while (list($rule, $type) = each($rules)) {
             $matched = $this->_matches($context, $rule, $type);
 
             if ($type & Validator::CONTEXT_NOT_IN) {
@@ -123,7 +123,7 @@ 
 
         $rule = substr($rule, $pos);
 
-        if($type & self::CONTEXT_REGEX) {
+        if ($type & self::CONTEXT_REGEX) {
             $rule = "/^$rule(\\.\\w+)?/i";
         }
 
@@ -131,22 +131,22 @@ 
     }
 
     private function _matches($context, $rule, $type) {
-        if($type & self::CONTEXT_EXACTLY) {
+        if ($type & self::CONTEXT_EXACTLY) {
             return in_array($rule, $context, true);
-        } elseif($type & self::CONTEXT_REGEX) {
-            foreach($context as $item) {
-                if(preg_match($rule, $item)) {
+        } elseif ($type & self::CONTEXT_REGEX) {
+            foreach ($context as $item) {
+                if (preg_match($rule, $item)) {
                     return true;
                 }
             }
             return false;
         } else {
-            if(in_array($rule, $context, true)) {
+            if (in_array($rule, $context, true)) {
                 return true;
             }
 
-            foreach($context as $item) {
-                if(fnmatch("$rule.*", $item)) {
+            foreach ($context as $item) {
+                if (fnmatch("$rule.*", $item)) {
                     return true;
                 }
             }
@@ -158,7 +158,7 @@ 
     {
         static $validator;
         if (!$validator) {
-            $validator = new DelegateValidator(function () {
+            $validator = new DelegateValidator(function() {
                 return true;
             });
         }

bin/VerboseOutput.php

Doc Comments +16 added lines, -1 removed lines

@@ -47,7 +47,7 @@ 
      * @param InputInterface     $input
      * @param Language           $language
      * @param FormatterInterface $formatter
-     * @param                    $source
+     * @param                    string $source
      */
     public function __construct(
         OutputInterface $output, InputInterface $input, Language $language, FormatterInterface $formatter, $source
@@ -97,11 +97,17 @@ 
         return $formatted;
     }
 
+    /**
+     * @param string $feature
+     */
     public function wants($feature)
     {
         return in_array($feature, $this->_input->getOption('debug'));
     }
 
+    /**
+     * @return Tokens
+     */
     protected function tokenize()
     {
         $tokens = $this->benchmark(function () {
@@ -125,12 +131,18 @@ 
         return $return;
     }
 
+    /**
+     * @param string $message
+     */
     private function _tree(Tokens $tree, $message, $indented = true)
     {
         $this->_output->writeln("Token tree <comment>$message</comment>: ");
         $this->_output->writeln($this->_debug->format(clone $tree, $indented));
     }
 
+    /**
+     * @return Tokens
+     */
     protected function parse(Tokens $tokens)
     {
         $tokens = $this->benchmark(function () use ($tokens) {
@@ -152,6 +164,9 @@ 
         }, $this->_times['formatting']);
     }
 
+    /**
+     * @param string $message
+     */
     private function _slashed($message, $data)
     {
         $this->_output->writeln(sprintf(

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/HighlightCommand.php

Spacing +7 added lines, -7 removed lines

@@ -47,23 +47,23 @@ 
 
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        if(!empty($input->getOption('debug')) && $output->getVerbosity() < OutputInterface::VERBOSITY_VERBOSE) {
+        if (!empty($input->getOption('debug')) && $output->getVerbosity() < OutputInterface::VERBOSITY_VERBOSE) {
             $output->setVerbosity(OutputInterface::VERBOSITY_VERBOSE);
         }
 
         $output->writeln($this->getApplication()->getLongVersion()."\n", Output::VERBOSITY_VERBOSE);
         $formatter = KeyLighter::get()->getFormatter($input->getOption('format')) ?: KeyLighter::get()->getDefaultFormatter();
 
-        foreach($input->getArgument('path') as $filename) {
+        foreach ($input->getArgument('path') as $filename) {
             $language = $input->getOption('language')
                 ? Language::byName($input->getOption('language'))
                 : Language::byFilename($filename);
 
-            if(!($source = $this->content($filename))) {
+            if (!($source = $this->content($filename))) {
                 throw new InvalidArgumentException(sprintf('Specified file %s doesn\'t exist, check if given path is correct.', $filename));
             }
 
-            if($output->isVerbose()) {
+            if ($output->isVerbose()) {
                 $output->writeln(sprintf(
                     "Used file: <path>%s</path>, Language: <language>%s</language>, Formatter: <formatter>%s</formatter>",
                     $filename, $language->getFQN(), get_class($formatter)
@@ -75,7 +75,7 @@ 
                 $formatted = KeyLighter::get()->highlight($source, $language, $formatter);
             }
             
-            if(!$input->getOption('no-output')) {
+            if (!$input->getOption('no-output')) {
                 $output->writeln($formatted);
             }
         }
@@ -83,12 +83,12 @@ 
 
     protected function content($path)
     {
-        if(!($file = @fopen($path, 'r'))) {
+        if (!($file = @fopen($path, 'r'))) {
             return false;
         }
 
         $content = '';
-        while(!feof($file)) {
+        while (!feof($file)) {
             $content .= fgets($file);
         }
         fclose($file);