@@ -16,7 +16,6 @@ |
||
16 | 16 | * Generate a new Job script to be executed under a separate PHP process |
17 | 17 | * |
18 | 18 | * @param null|string $mutantFile |
19 | - * @param array $args |
|
20 | 19 | * @param string $bootstrap |
21 | 20 | * @param null|string $replacingFile |
22 | 21 | * @return string |
@@ -10,8 +10,6 @@ |
||
10 | 10 | |
11 | 11 | namespace Humbug\Command; |
12 | 12 | |
13 | -use Humbug\Exception\FilesystemException; |
|
14 | -use Humbug\Console\Application; |
|
15 | 13 | use Symfony\Component\Console\Command\Command; |
16 | 14 | use Symfony\Component\Console\Input\InputInterface; |
17 | 15 | use Symfony\Component\Console\Input\InputOption; |
@@ -167,7 +167,7 @@ |
||
167 | 167 | |
168 | 168 | protected function update(Updater $updater) |
169 | 169 | { |
170 | - $this->output->writeln('Updating...'.PHP_EOL); |
|
170 | + $this->output->writeln('Updating...' . PHP_EOL); |
|
171 | 171 | try { |
172 | 172 | $result = $updater->update(); |
173 | 173 |
@@ -136,6 +136,9 @@ discard block |
||
136 | 136 | } |
137 | 137 | } |
138 | 138 | |
139 | + /** |
|
140 | + * @param string $sectionName |
|
141 | + */ |
|
139 | 142 | private function printSectionHeaderColor($sectionName) |
140 | 143 | { |
141 | 144 | switch ($sectionName) { |
@@ -163,7 +166,7 @@ discard block |
||
163 | 166 | /** |
164 | 167 | * @param string $class |
165 | 168 | * @param array $data |
166 | - * @param $verbosityLevel |
|
169 | + * @param integer $verbosityLevel |
|
167 | 170 | */ |
168 | 171 | protected function printSectionData($class, $data, $verbosityLevel) |
169 | 172 | { |
@@ -280,6 +283,10 @@ discard block |
||
280 | 283 | return $this->generateDiffString($old, $new); |
281 | 284 | } |
282 | 285 | |
286 | + /** |
|
287 | + * @param string $old |
|
288 | + * @param string $new |
|
289 | + */ |
|
283 | 290 | private function generateDiffString($old, $new) |
284 | 291 | { |
285 | 292 | $fromStart = strspn($old ^ $new, "\0"); |
@@ -306,8 +313,8 @@ discard block |
||
306 | 313 | |
307 | 314 | /** |
308 | 315 | * @param $string |
309 | - * @param $index |
|
310 | - * @return array |
|
316 | + * @param integer $index |
|
317 | + * @return string[] |
|
311 | 318 | */ |
312 | 319 | private function stringSplitByIndex($string, $index) |
313 | 320 | { |
@@ -230,9 +230,9 @@ discard block |
||
230 | 230 | } |
231 | 231 | |
232 | 232 | $diff[$diffPos] = $this->generateDiffString($old, $new); |
233 | - unset($diff[$diffPos + 1]); |
|
233 | + unset($diff[$diffPos+1]); |
|
234 | 234 | |
235 | - $startLine = $diffLine - $diffPos + 3; |
|
235 | + $startLine = $diffLine-$diffPos+3; |
|
236 | 236 | $tag = ($startLine != $diffLine) ? 'comment' : 'fg=red'; |
237 | 237 | |
238 | 238 | foreach ($diff as $num => $item) { |
@@ -285,13 +285,13 @@ discard block |
||
285 | 285 | $fromStart = strspn($old ^ $new, "\0"); |
286 | 286 | $fromEnd = strspn(strrev($old) ^ strrev($new), "\0"); |
287 | 287 | |
288 | - $oldEnd = strlen($old) - $fromEnd; |
|
289 | - $newEnd = strlen($new) - $fromEnd; |
|
288 | + $oldEnd = strlen($old)-$fromEnd; |
|
289 | + $newEnd = strlen($new)-$fromEnd; |
|
290 | 290 | |
291 | 291 | $start = substr($new, 0, $fromStart); |
292 | 292 | $end = substr($new, $newEnd); |
293 | - $newDiff = substr($new, $fromStart, $newEnd - $fromStart); |
|
294 | - $oldDiff = substr($old, $fromStart, $oldEnd - $fromStart); |
|
293 | + $newDiff = substr($new, $fromStart, $newEnd-$fromStart); |
|
294 | + $oldDiff = substr($old, $fromStart, $oldEnd-$fromStart); |
|
295 | 295 | |
296 | 296 | $result = sprintf( |
297 | 297 | '%s<fg=red>%s</fg=red><fg=green>%s</fg=green>%s', |
@@ -14,7 +14,6 @@ |
||
14 | 14 | use Humbug\File\Collector as FileCollector; |
15 | 15 | use Humbug\File\Collection as FileCollection; |
16 | 16 | use Humbug\TestSuite\Mutant\Collector as ResultCollector; |
17 | -use Humbug\Adapter\AdapterAbstract; |
|
18 | 17 | use Humbug\Utility\CoverageData; |
19 | 18 | use Humbug\Exception\RuntimeException; |
20 | 19 |
@@ -30,7 +30,7 @@ |
||
30 | 30 | } elseif ($num == 1) { |
31 | 31 | $replace = 0; |
32 | 32 | } else { |
33 | - $replace = $num + 1; |
|
33 | + $replace = $num+1; |
|
34 | 34 | } |
35 | 35 | $tokens[$index] = [ |
36 | 36 | T_LNUMBER, |
@@ -31,7 +31,7 @@ |
||
31 | 31 | } elseif ($num == 1) { |
32 | 32 | $replace = 0.0; |
33 | 33 | } elseif ($num < 2) { |
34 | - $replace = $num + 1; |
|
34 | + $replace = $num+1; |
|
35 | 35 | } else { |
36 | 36 | $replace = 1.0; |
37 | 37 | } |
@@ -35,13 +35,13 @@ |
||
35 | 35 | /** |
36 | 36 | * Exclude likely uses of ampersand for references |
37 | 37 | */ |
38 | - if (self::getNextToken($tokens, $index, [ T_WHITESPACE ]) === T_VARIABLE) { |
|
38 | + if (self::getNextToken($tokens, $index, [T_WHITESPACE]) === T_VARIABLE) { |
|
39 | 39 | return false; |
40 | 40 | } |
41 | - if (self::getNextToken($tokens, $index, [ T_WHITESPACE ]) === T_FUNCTION) { |
|
41 | + if (self::getNextToken($tokens, $index, [T_WHITESPACE]) === T_FUNCTION) { |
|
42 | 42 | return false; |
43 | 43 | } |
44 | - if (self::getPreviousToken($tokens, $index, [ T_WHITESPACE ]) === '=') { |
|
44 | + if (self::getPreviousToken($tokens, $index, [T_WHITESPACE]) === '=') { |
|
45 | 45 | return false; |
46 | 46 | } |
47 | 47 | return true; |
@@ -30,7 +30,7 @@ discard block |
||
30 | 30 | |
31 | 31 | private static function shouldSkip($token, array $excludeTokens) |
32 | 32 | { |
33 | - if (! is_array($token) && in_array($token, $excludeTokens, true)) { |
|
33 | + if (!is_array($token) && in_array($token, $excludeTokens, true)) { |
|
34 | 34 | return true; |
35 | 35 | } |
36 | 36 | |
@@ -49,11 +49,11 @@ discard block |
||
49 | 49 | { |
50 | 50 | $tokenCount = count($tokens); |
51 | 51 | while ($index < $tokenCount && isset($tokens[$index+1]) |
52 | - && self::shouldSkip($tokens[$index + 1], $excludeTokens)) { |
|
52 | + && self::shouldSkip($tokens[$index+1], $excludeTokens)) { |
|
53 | 53 | $index++; |
54 | 54 | } |
55 | 55 | |
56 | - if (! isset($tokens[$index+1])) { |
|
56 | + if (!isset($tokens[$index+1])) { |
|
57 | 57 | return false; |
58 | 58 | } |
59 | 59 | |
@@ -72,11 +72,11 @@ discard block |
||
72 | 72 | protected static function getPreviousToken(array &$tokens, $index, array $excludeTokens = []) |
73 | 73 | { |
74 | 74 | while ($index > 0 && isset($tokens[$index-1]) |
75 | - && self::shouldSkip($tokens[$index - 1], $excludeTokens)) { |
|
75 | + && self::shouldSkip($tokens[$index-1], $excludeTokens)) { |
|
76 | 76 | $index--; |
77 | 77 | } |
78 | 78 | |
79 | - if (! isset($tokens[$index-1])) { |
|
79 | + if (!isset($tokens[$index-1])) { |
|
80 | 80 | return false; |
81 | 81 | } |
82 | 82 |
@@ -47,10 +47,10 @@ |
||
47 | 47 | |
48 | 48 | protected static function isANonNegatedFunctionInIfStatement(array &$tokens, $index) |
49 | 49 | { |
50 | - $max = count($tokens) - $index + 1; |
|
50 | + $max = count($tokens)-$index+1; |
|
51 | 51 | $bracketCount = 0; |
52 | 52 | $inArg = false; |
53 | - for ($i=$index+1; $i < $max; $i++) { |
|
53 | + for ($i = $index+1; $i < $max; $i++) { |
|
54 | 54 | if (is_array($tokens[$i]) && $tokens[$i][0] == T_WHITESPACE) { |
55 | 55 | continue; |
56 | 56 | } elseif (!is_array($tokens[$i]) && $tokens[$i] == '(') { |