kadet1090 / KeyLighter

Matcher/WordMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         ], $options);
 
         if ($options['escape']) {
-            $words = array_map(function ($word) {
+            $words = array_map(function($word) {
                 return preg_quote($word, '/');
             }, $words);
         }

Language/Sql.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         'INTERSECT', 'INTERVAL', 'INTO', 'IS', 'ITERATE', 'LANGUAGE', 'LARGE', 'LATERAL', 'LEADING',
         'LEAVE', 'LEFT', 'LIKE', 'LOCAL', 'LOOP', 'MATCH', 'MEMBER', 'MERGE', 'METHOD', 'MODIFIES',
         'MODULE', 'MONTH', 'MULTISET', 'NATIONAL', 'NATURAL', 'NCHAR', 'NCLOB', 'NEW', 'NO', 'NONE', 'NOT',
-        'OF', 'OLD', 'ON', 'ONLY', 'OPEN', 'OR', 'ORDER', 'OUT','OUTER', 'OUTPUT', 'OVER',
+        'OF', 'OLD', 'ON', 'ONLY', 'OPEN', 'OR', 'ORDER', 'OUT', 'OUTER', 'OUTPUT', 'OVER',
         'OVERLAPS', 'PARAMETER', 'PARTITION', 'PRECISION', 'PREPARE', 'PRIMARY', 'PROCEDURE', 'RANGE', 'READS',
         'RECURSIVE', 'REF', 'REFERENCES', 'REFERENCING', 'RELEASE',
         'REPEAT', 'RESIGNAL', 'RESULT', 'RETURN', 'RETURNS', 'REVOKE', 'RIGHT', 'ROLLBACK', 'ROLLUP', 'ROW',

Parser/Token/Token.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
     }
 
     /**
-     * @return Token|null|false
+     * @return Token
      */
     public function getEnd()
     {

Spacing +1 added lines, -2 removed lines

@@ -185,8 +185,7 @@
         }
 
         return $this->isStart() ?
-            $this->processStart($context, $language, $result, $tokens) :
-            $this->processEnd($context, $language, $result, $tokens);
+            $this->processStart($context, $language, $result, $tokens) : $this->processEnd($context, $language, $result, $tokens);
     }
 
     protected function processStart(Context $context, Language $language, Result $result, TokenIterator $tokens)

Parser/Token/ContextualToken.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
                 $this->_end->_valid = false;
             }
         } else {
-            if (!$this->rule->validator->validate($context, [ $this->name => Validator::CONTEXT_IN ])) {
+            if (!$this->rule->validator->validate($context, [$this->name => Validator::CONTEXT_IN])) {
                 $this->setValid(false);
             } else {
                 $this->_valid       = false;

Parser/Result.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         $this->_source = $source;
         $this->_start  = $start;
 
-        parent::__construct($start !== null ? [ $start->id => $start ] : [], 0, \ArrayIterator::class);
+        parent::__construct($start !== null ? [$start->id => $start] : [], 0, \ArrayIterator::class);
     }
 
     public function getSource()

Matcher/DelegateRegexMatcher.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     /**
      * RegexMatcher constructor.
      *
-     * @param          $regex
+     * @param          string $regex
      * @param callable $callable
      */
     public function __construct($regex, callable $callable)

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']]),

Utils/ArrayHelper.php

Spacing +2 added lines, -2 removed lines

@@ -19,14 +19,14 @@
 {
     public static function rearrange(array $array, array $keys)
     {
-        return array_combine($keys, array_map(function ($key) use ($array) {
+        return array_combine($keys, array_map(function($key) use ($array) {
             return $array[$key];
         }, $keys));
     }
 
     public static function column(array $array, $index)
     {
-        return array_map(function ($e) use ($index) { return $e[$index]; }, $array);
+        return array_map(function($e) use ($index) { return $e[$index]; }, $array);
     }
 
     public static function find(array $array, callable $tester)