kadet1090 / KeyLighter

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)

Language/Perl.php

Indentation +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
             'string.nowdoc'  => new Rule(
                 new RegexMatcher('/<<\s*\'(\w+)\';(?P<string>.*?)\R\1/sm', [
                     'string' => Token::NAME,
-                          0  => 'keyword.nowdoc'
+                            0  => 'keyword.nowdoc'
                 ]), ['context' => ['!comment']]
             ),
 

Matcher/CommentMatcher.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
     public function __construct(array $singleLine = null, array $multiLine = null)
     {
         $this->singleLine = $singleLine ?: [];
-        $this->multiLine  = $multiLine  ?: [];
+        $this->multiLine  = $multiLine ?: [];
     }
 
 
@@ -49,7 +49,7 @@ 
         $all = [];
 
         foreach ($this->multiLine as $name => $comment) {
-            $comment = array_map(function ($e) { return preg_quote($e, '/'); }, $comment);
+            $comment = array_map(function($e) { return preg_quote($e, '/'); }, $comment);
 
             $all[] = [$name, "/({$comment[0]}(.*?){$comment[1]})/ms"];
         }

Language/Shell.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
                 new Rule(new RegexMatcher('/(\$\w+)/i'), ['context' => ['*none', '*string.double']]),
                 'special'  => new Rule(new RegexMatcher('/(\$[#@_])/i'), ['context' => ['*none', '*string.double']]),
                 'argument' => new Rule(new RegexMatcher('/(\$\d+)/i'), ['context' => ['*none', '*string.double']]),
-                new Rule(new RegexMatcher('/\$\{(\w+)(.*?)\}/i', [ 1 => Token::NAME, 2 => 'string' ]), ['context' => ['*none', '*string.double']])
+                new Rule(new RegexMatcher('/\$\{(\w+)(.*?)\}/i', [1 => Token::NAME, 2 => 'string']), ['context' => ['*none', '*string.double']])
             ],
 
             'number'    => new Rule(new RegexMatcher('/(-?(?:0[0-7]+|0[xX][0-9a-fA-F]+|0b[01]+|\d+))/')),

Language/Latex.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
                 new Rule(new RegexMatcher('/(\\\\\[.*?\\\\\])/s')),
                 new Rule(
                     new RegexMatcher(
-                        '/\\\begin{((?:' . implode('|', self::$mathEnvironments) . ')\*?)}(.*?)\\\end{\1}/s',
+                        '/\\\begin{((?:'.implode('|', self::$mathEnvironments).')\*?)}(.*?)\\\end{\1}/s',
                         [2 => Token::NAME]
                     )
                 )

bin/Commands/HighlightCommand.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
     {
         $this->setName('highlight')
             ->addArgument('path', InputArgument::REQUIRED | InputArgument::IS_ARRAY, 'File(s) to highlight')
-            ->addOption('normalize',  null, InputOption::VALUE_OPTIONAL, 'Normalize input to LF')
+            ->addOption('normalize', null, InputOption::VALUE_OPTIONAL, 'Normalize input to LF')
             ->addOption('language', 'l', InputOption::VALUE_OPTIONAL, 'Source Language to highlight, see <comment>list-languages</comment> command')
             ->addOption('format', 'f', InputOption::VALUE_OPTIONAL, 'Output format, see <comment>list-languages</comment> command', 'cli')
             ->addOption(
@@ -48,26 +48,26 @@ 
 
     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) {
             $this->process($input, $output, $filename, $formatter);
         }
     }
 
     protected function content($path, $normalize = false)
     {
-        if(!($file = @fopen($path, 'r'))) {
+        if (!($file = @fopen($path, 'r'))) {
             return false;
         }
 
         $content = '';
-        while(!feof($file)) {
+        while (!feof($file)) {
             $content .= fgets($file);
         }
         fclose($file);

bin/Commands/Test/RegenerateCommand.php

Doc Comments +3 added lines

@@ -99,6 +99,9 @@
             || $input->hasParameterOption('-r');
     }
 
+    /**
+     * @param string $filename
+     */
     private function regenerate(InputInterface $input, $filename) {
         $filename = str_replace(DIRECTORY_SEPARATOR, '/', $filename);
         $patterns = $input->getArgument('files');

Spacing +5 added lines, -5 removed lines

@@ -59,7 +59,7 @@ 
         /** @var \SplFileInfo $file */
         foreach ($iterator as $file) {
             $pathname = substr($file->getPathname(), strlen($this->_input) + 1);
-            if(!$this->regenerate($input, $pathname)) {
+            if (!$this->regenerate($input, $pathname)) {
                 continue;
             }
 
@@ -71,7 +71,7 @@ 
             if ($this->review($input, $output, $tokens, $reviewer)) {
                 $result = StringHelper::normalize($this->_formatter->format($tokens));
 
-                if(!file_exists($this->_output.'/'.dirname($pathname))) {
+                if (!file_exists($this->_output.'/'.dirname($pathname))) {
                     mkdir($this->_output.'/'.dirname($pathname), 755, true);
                 }
 
@@ -103,12 +103,12 @@ 
         $filename = str_replace(DIRECTORY_SEPARATOR, '/', $filename);
         $patterns = $input->getArgument('files');
 
-        foreach($patterns as $pattern) {
-            if($input->getOption('new') && file_exists("{$this->_output}/$filename.tkn")) {
+        foreach ($patterns as $pattern) {
+            if ($input->getOption('new') && file_exists("{$this->_output}/$filename.tkn")) {
                 continue;
             }
 
-            if(fnmatch($pattern, $filename)) {
+            if (fnmatch($pattern, $filename)) {
                 return true;
             }
         }

bin/Commands/Benchmark/AnalyzeCommand.php

Unused Use Statements -2 removed lines

@@ -16,9 +16,7 @@
 namespace Kadet\Highlighter\bin\Commands\Benchmark;
 
 
-use Kadet\Highlighter\Formatter\FormatterInterface;
 use Kadet\Highlighter\KeyLighter;
-use Kadet\Highlighter\Language\Language;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Helper\TableCell;

Spacing +7 added lines, -7 removed lines

@@ -49,7 +49,7 @@ 
             $this->separator($file, $table);
 
             foreach ($data['times'] as $set => $times) {
-                $result = array_map(function ($time) use ($data, $input) {
+                $result = array_map(function($time) use ($data, $input) {
                     return $input->getOption('relative') ? $data['size'] / $time : $time * 1000;
                 }, $times);
 
@@ -63,8 +63,8 @@ 
             }
 
             foreach ($data['memory'] as $set => $memory) {
-                $result = array_map(function ($memory) use ($data, $input) {
-                    $bytes = $input->getOption('relative') ? $memory/$data['size'] : $memory;
+                $result = array_map(function($memory) use ($data, $input) {
+                    $bytes = $input->getOption('relative') ? $memory / $data['size'] : $memory;
                     return $this->formatBytes($bytes, (bool)$input->getOption('relative'));
                 }, $memory);
 
@@ -72,7 +72,7 @@ 
             }
         }
 
-        if(!$input->hasParameterOption('--summary')) {
+        if (!$input->hasParameterOption('--summary')) {
             $table->render();
         }
 
@@ -81,7 +81,7 @@ 
         }, ARRAY_FILTER_USE_KEY);
 
         $max = max(array_map('strlen', array_keys($summary)));
-        foreach($summary as $name => $set) {
+        foreach ($summary as $name => $set) {
             $output->writeln(sprintf(
                 "<comment>%s</comment> %s %s",
                 str_pad($name, $max, ' ', STR_PAD_LEFT),
@@ -130,7 +130,7 @@ 
     {
         $mean = array_sum($result) / count($result);
 
-        return sqrt(array_sum(array_map(function ($result) use ($mean) {
+        return sqrt(array_sum(array_map(function($result) use ($mean) {
             return pow($result - $mean, 2);
         }, $result)) / count($result));
     }
@@ -144,7 +144,7 @@ 
 
         $bytes /= (1 << (10 * $pow));
 
-        return $this->format($bytes);//.$units[$pow].($relative ? '/byte' : '');
+        return $this->format($bytes); //.$units[$pow].($relative ? '/byte' : '');
     }
 
     protected function configure()

bin/Commands/Benchmark/RunCommand.php

Doc Comments +6 added lines

@@ -110,6 +110,9 @@ 
         ;
     }
 
+    /**
+     * @param string $source
+     */
     protected function benchmark($source, Language $language, FormatterInterface $formatter, $geshi = null)
     {
         gc_collect_cycles(); // force garbage collector
@@ -161,6 +164,9 @@ 
         ];
     }
 
+    /**
+     * @param string $source
+     */
     protected function geshi($source, $language)
     {
         $geshi = microtime(true);

Spacing +4 added lines, -4 removed lines

@@ -70,13 +70,13 @@ 
             $times  = [];
             $memory = [];
 
-            for($i = $input->getOption('times'), $progress = new ProgressBar($output, $i); $i > 0; $i--) {
+            for ($i = $input->getOption('times'), $progress = new ProgressBar($output, $i); $i > 0; $i--) {
                 $progress->display();
                 $result = $this->benchmark($source, $language, $formatter);
-                $times  = array_merge_recursive($times,  $result['times']);
+                $times  = array_merge_recursive($times, $result['times']);
                 $memory = array_merge_recursive($memory, $result['memory']);
 
-                if($input->getOption('geshi') && class_exists('GeSHi')) {
+                if ($input->getOption('geshi') && class_exists('GeSHi')) {
                     $times['geshi'][] = $this->geshi($source, $file->getExtension());
                 }
 
@@ -178,7 +178,7 @@ 
     {
         $result = json_encode($results, $input->getOption('pretty') ? JSON_PRETTY_PRINT : 0);
 
-        if($input->hasArgument('output')) {
+        if ($input->hasArgument('output')) {
             file_put_contents($input->getArgument('output'), $result);
         } else {
             $output->write($result);