diff options
author | Thiemo Kreuz <thiemo.kreuz@wikimedia.de> | 2021-06-17 16:32:05 +0200 |
---|---|---|
committer | Krinkle <krinklemail@gmail.com> | 2021-06-18 21:23:56 +0000 |
commit | 2ba01c7ee725bef3322e0bf76f429607554f7ff8 (patch) | |
tree | 51df557ac5a83a53052011e82f4f33df02004278 /tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php | |
parent | f2877a80ba4717649b540bca22f0c792371da906 (diff) | |
download | mediawikicore-2ba01c7ee725bef3322e0bf76f429607554f7ff8.tar.gz mediawikicore-2ba01c7ee725bef3322e0bf76f429607554f7ff8.zip |
Remove some more comments that literally repeat the code
… including PHPDoc tags like `@return <type> $variableName`.
A return value doesn't have a variable name. I can see that
some people do this intentionally, repeating the variable
name that was used in the final `return $var;` at the end
of a method. This can indeed be helpful. I leave a lot of
these untouched and removed them only when it's obviously
wrong, or does not provide any additional information in
addition to what the code already says.
Change-Id: Ia18cd9f25ef658b08ad25b97a744897e2a8deffc
Diffstat (limited to 'tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php')
-rw-r--r-- | tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php b/tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php index 3719502a8d8a..d0819841681b 100644 --- a/tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php +++ b/tests/phpunit/unit/includes/libs/objectcache/WANObjectCacheTest.php @@ -22,8 +22,8 @@ class WANObjectCacheTest extends PHPUnit\Framework\TestCase { use MediaWikiCoversValidator; /** - * @param array $params [optional] - * @return WANObjectCache[]|HashBagOStuff[] (WANObjectCache, BagOStuff) + * @param array $params + * @return array [ WANObjectCache, HashBagOStuff ] */ private function newWanCache( array $params = [] ) { if ( isset( $params['broadcastRoutingPrefix'] ) ) { |