kadet1090 / KeyLighter

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();

Language/Xml.php

Spacing +3 added lines, -3 removed lines

@@ -36,14 +36,14 @@
                 new OpenRule(new RegexMatcher('/(<[\w\.-]+)[:\/>:\s]/')),
                 new CloseRule(new SubStringMatcher('>'), ['context' => ['!string', '!comment']])
             ],
-            'tag.close' => new Rule(new RegexMatcher('/(<\/' . self::IDENTIFIER . '>)/')),
+            'tag.close' => new Rule(new RegexMatcher('/(<\/'.self::IDENTIFIER.'>)/')),
 
-            'symbol.tag' => new Rule(new RegexMatcher('/<\\/?' . self::IDENTIFIER . '/', [
+            'symbol.tag' => new Rule(new RegexMatcher('/<\\/?'.self::IDENTIFIER.'/', [
                 'name'      => Token::NAME,
                 'namespace' => '$.namespace'
             ]), ['context' => ['tag', '!string']]),
 
-            'symbol.attribute' => new Rule(new RegexMatcher('/' . self::IDENTIFIER . '=/', [
+            'symbol.attribute' => new Rule(new RegexMatcher('/'.self::IDENTIFIER.'=/', [
                 'name'      => Token::NAME,
                 'namespace' => '$.namespace'
             ]), ['context' => ['tag', '!string']]),

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'];
         }
 
-        foreach($rules as $rule => &$type) {
+        foreach ($rules as $rule => &$type) {
             $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;
             });
         }