minor #10717 unified return null usages (fabpot)

fabpot · fabpot · commit 0d2ab42d4953 · 2014-04-18T22:35:25.000+02:00
This PR was merged into the 2.3 branch.

Discussion
----------

unified return null usages

| Q             | A
| ------------- | ---
| License       | MIT

This PR unifies the way we return `null` from a function or method:

 * always use `return;` instead of `return null;` (the current code base uses both);
 * never use `return;` at the end of a function/method.

Commits
-------

d1d569b unified return null usages
diff --git a/Parser/TokenStream.php b/Parser/TokenStream.php
@@ -162,7 +162,7 @@ public function getNextIdentifierOrStar()
         }
 
         if ($next->isDelimiter(array('*'))) {
-            return null;
+            return;
         }
 
         throw SyntaxErrorException::unexpectedToken('identifier or "*"', $next);

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ public function getNextIdentifierOrStar()`
`162`	`162`	`}`
`163`	`163`
`164`	`164`	`if ($next->isDelimiter(array('*'))) {`
`165`		`- return null;`
	`165`	`+ return;`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`throw SyntaxErrorException::unexpectedToken('identifier or "*"', $next);`