1 | <?php |
||
5 | class PullRequests |
||
6 | { |
||
7 | protected $prs = []; |
||
8 | |||
9 | /** |
||
10 | * PullRequests constructor |
||
11 | */ |
||
12 | public function __construct() |
||
15 | |||
16 | public function addSearchResults($searchResults, $pattern = '') |
||
27 | |||
28 | public function add($pr) |
||
32 | |||
33 | public function titles() |
||
43 | |||
44 | public function prNumbers() |
||
48 | |||
49 | public function isEmpty() |
||
53 | } |
||
54 |
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVar
assignment in line 1 and the$higher
assignment in line 2 are dead. The first because$myVar
is never used and the second because$higher
is always overwritten for every possible time line.