Remove unneeded @return documentation

Change-Id: If79d2126cc1b6a6e9876a972c9560d045de42ff6

5 files changed, 0 insertions, 8 deletions

diff --git a/includes/OutputPage.php b/includes/OutputPage.php
index d3c685ae31e8..90bfa44a6d79 100644
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -864,7 +864,6 @@ class OutputPage extends ContextSource {
 	 * @param string $policy The literal string to output as the contents of
 	 *   the meta tag.  Will be parsed according to the spec and output in
 	 *   standardized form.
-	 * @return null
 	 */
 	public function setRobotPolicy( $policy ) {
 		$policy = Article::formatRobotPolicy( $policy );
@@ -892,7 +891,6 @@ class OutputPage extends ContextSource {
 	 * touched.
 	 *
 	 * @param string $policy Either 'index' or 'noindex'.
-	 * @return null
 	 */
 	public function setIndexPolicy( $policy ) {
 		$policy = trim( $policy );
@@ -915,7 +913,6 @@ class OutputPage extends ContextSource {
 	 * touched.
 	 *
 	 * @param string $policy Either 'follow' or 'nofollow'.
-	 * @return null
 	 */
 	public function setFollowPolicy( $policy ) {
 		$policy = trim( $policy );
diff --git a/includes/libs/Timing.php b/includes/libs/Timing.php
index 65a924795acd..5ed7aebc6a7a 100644
--- a/includes/libs/Timing.php
+++ b/includes/libs/Timing.php
@@ -59,7 +59,6 @@ class Timing implements LoggerAwareInterface {
 	 * Sets a logger instance on the object.
 	 *
 	 * @param LoggerInterface $logger
-	 * @return null
 	 */
 	public function setLogger( LoggerInterface $logger ) {
 		$this->logger = $logger;
diff --git a/tests/parser/ParserTestPrinter.php b/tests/parser/ParserTestPrinter.php
index 07a63c48c26a..64067996fe19 100644
--- a/tests/parser/ParserTestPrinter.php
+++ b/tests/parser/ParserTestPrinter.php
@@ -104,7 +104,6 @@ class ParserTestPrinter extends TestRecorder {
 	 * Print a happy success message.
 	 *
 	 * @param ParserTestResult $testResult
-	 * @return bool
 	 */
 	private function showSuccess( ParserTestResult $testResult ) {
 		if ( $this->showProgress ) {
diff --git a/tests/phpunit/includes/TestUserRegistry.php b/tests/phpunit/includes/TestUserRegistry.php
index f28fd614528c..8d88e6d13b4c 100644
--- a/tests/phpunit/includes/TestUserRegistry.php
+++ b/tests/phpunit/includes/TestUserRegistry.php
@@ -96,8 +96,6 @@ class TestUserRegistry {
 	 * are not reused. We don't reset the counter or random string by design.
 	 *
 	 * @since 1.28
-	 *
-	 * @return TestUser
 	 */
 	public static function clear() {
 		self::$testUsers = [];
diff --git a/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php b/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
index 11f4dacce550..a7aafd6e58a3 100644
--- a/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
+++ b/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
@@ -33,7 +33,6 @@ abstract class ApiQueryContinueTestBase extends ApiQueryTestBase {
 	 * @param int $expectedCount Max number of iterations
 	 * @param string $id Unit test id
 	 * @param bool $continue True to use smart continue
-	 * @return array Merged results data array
 	 */
 	protected function checkC( $expected, $params, $expectedCount, $id, $continue = true ) {
 		$result = $this->query( $params, $expectedCount, $id, $continue );