Fix spacing after @param and friends in comments
authorumherirrender <umherirrender_de.wp@web.de>
Sat, 5 Apr 2014 19:54:24 +0000 (21:54 +0200)
committerIAlex <codereview@emsenhuber.ch>
Sat, 5 Apr 2014 20:02:29 +0000 (20:02 +0000)
Searched for:
\@(param|return|throws|since|deprecated|access|todo|var)[ \t]{2,}

Change-Id: Icce22ba9fe0635455691ca58d9872d618151f346

39 files changed:
includes/Block.php
includes/HttpFunctions.php
includes/Linker.php
includes/MagicWord.php
includes/OutputPage.php
includes/Skin.php
includes/UserArrayFromResult.php
includes/Xml.php
includes/actions/FormAction.php
includes/api/ApiBase.php
includes/api/ApiHelp.php
includes/api/ApiMove.php
includes/api/ApiQueryRandom.php
includes/api/ApiQueryRecentChanges.php
includes/db/DatabaseOracle.php
includes/deferred/SqlDataUpdate.php
includes/diff/DifferenceEngine.php
includes/filebackend/filejournal/FileJournal.php
includes/htmlform/HTMLCheckField.php
includes/htmlform/HTMLMultiSelectField.php
includes/htmlform/HTMLSelectAndOtherField.php
includes/htmlform/HTMLSelectOrOtherField.php
includes/logging/LogPage.php
includes/media/Bitmap_ClientOnly.php
includes/objectcache/MemcachedClient.php
includes/parser/CacheTime.php
includes/parser/CoreParserFunctions.php
includes/parser/Parser.php
includes/parser/Preprocessor_Hash.php
includes/parser/StripState.php
includes/revisiondelete/RevisionDeleteUser.php
includes/revisiondelete/RevisionDeleter.php
includes/specialpage/FormSpecialPage.php
includes/specials/SpecialBlock.php
includes/specials/SpecialImport.php
includes/specials/SpecialUserrights.php
languages/Language.php
maintenance/benchmarks/bench_delete_truncate.php
tests/phpunit/MediaWikiPHPUnitTestListener.php

index 95a8386..74ece4c 100644 (file)
@@ -312,7 +312,7 @@ class Block {
        /**
         * Get the component of an IP address which is certain to be the same between an IP
         * address and a rangeblock containing that IP address.
-        * @param  $hex String Hexadecimal IP representation
+        * @param $hex String Hexadecimal IP representation
         * @return String
         */
        protected static function getIpFragment( $hex ) {
@@ -362,7 +362,7 @@ class Block {
 
        /**
         * Create a new Block object from a database row
-        * @param  $row ResultWrapper row from the ipblocks table
+        * @param $row ResultWrapper row from the ipblocks table
         * @return Block
         */
        public static function newFromRow( $row ) {
@@ -862,7 +862,7 @@ class Block {
        /**
         * Get/set whether the Block is a hardblock (affects logged-in users on a given IP/range
         * @param $x Bool
-        * @return  Bool
+        * @return Bool
         */
        public function isHardblock( $x = null ) {
                wfSetVar( $this->isHardblock, $x );
index 274d4a7..1fd437e 100644 (file)
@@ -205,7 +205,7 @@ class MWHttpRequest {
        protected $followRedirects = false;
 
        /**
-        * @var  CookieJar
+        * @var CookieJar
         */
        protected $cookieJar;
 
index 033926a..094a304 100644 (file)
@@ -742,7 +742,7 @@ class Linker {
 
        /**
         * @param $title Title
-        * @param  $file File
+        * @param $file File
         * @param array $frameParams
         * @param array $handlerParams
         * @param bool $time
@@ -1467,8 +1467,8 @@ class Linker {
 
        /**
         * @param $contextTitle Title
-        * @param  $target
-        * @param  $text
+        * @param $target
+        * @param $text
         * @return string
         */
        public static function normalizeSubpageLink( $contextTitle, $target, &$text ) {
index 377d406..e301cea 100644 (file)
@@ -512,7 +512,7 @@ class MagicWord {
        }
 
        /**
-        * @param  $text
+        * @param $text
         * @return bool
         */
        function matchStartAndRemove( &$text ) {
index 3a6787f..f469732 100644 (file)
@@ -1315,8 +1315,8 @@ class OutputPage extends ContextSource {
 
        /**
         * Set the highest level of CSS/JS untrustworthiness allowed
-        * @param  $type String ResourceLoaderModule TYPE_ constant
-        * @param  $level Int ResourceLoaderModule class constant
+        * @param $type String ResourceLoaderModule TYPE_ constant
+        * @param $level Int ResourceLoaderModule class constant
         */
        public function setAllowedModules( $type, $level ) {
                $this->mAllowedModules[$type] = $level;
@@ -1324,8 +1324,8 @@ class OutputPage extends ContextSource {
 
        /**
         * As for setAllowedModules(), but don't inadvertently make the page more accessible
-        * @param  $type String
-        * @param  $level Int ResourceLoaderModule class constant
+        * @param $type String
+        * @param $level Int ResourceLoaderModule class constant
         */
        public function reduceAllowedModules( $type, $level ) {
                $this->mAllowedModules[$type] = min( $this->getAllowedModules( $type ), $level );
index 38c3623..9ec7c31 100644 (file)
@@ -1573,14 +1573,14 @@ abstract class Skin extends ContextSource {
         * Create a section edit link.  This supersedes editSectionLink() and
         * editSectionLinkForOther().
         *
-        * @param $nt      Title  The title being linked to (may not be the same as
+        * @param $nt Title  The title being linked to (may not be the same as
         *   the current page, if the section is included from a template)
         * @param string $section The designation of the section being pointed to,
         *   to be included in the link, like "&section=$section"
         * @param string $tooltip The tooltip to use for the link: will be escaped
         *   and wrapped in the 'editsectionhint' message
-        * @param $lang    string Language code
-        * @return         string HTML to use for edit link
+        * @param $lang string Language code
+        * @return string HTML to use for edit link
         */
        public function doEditSectionLink( Title $nt, $section, $tooltip = null, $lang = false ) {
                // HTML generated here should probably have userlangattributes
index 5d5859d..00366e4 100644 (file)
@@ -38,7 +38,7 @@ class UserArrayFromResult extends UserArray implements Countable {
        }
 
        /**
-        * @param  $row
+        * @param $row
         * @return void
         */
        protected function setCurrent( $row ) {
index 45441fd..3b82c64 100644 (file)
@@ -901,7 +901,7 @@ class XmlSelect {
         * label => value
         * label => ( label => value, label => value )
         *
-        * @param  $options
+        * @param $options
         */
        public function addOptions( $options ) {
                $this->options[] = $options;
@@ -912,7 +912,7 @@ class XmlSelect {
         * label => value
         * label => ( label => value, label => value )
         *
-        * @param  $options
+        * @param $options
         * @param bool $default
         * @return string
         */
index 974ee94..73bf187 100644 (file)
@@ -91,7 +91,7 @@ abstract class FormAction extends Action {
         * Process the form on POST submission.  If you return false from getFormFields(),
         * this will obviously never be reached.  If you don't want to do anything with the
         * form, just return false here
-        * @param  $data Array
+        * @param $data Array
         * @return Bool|Array true for success, false for didn't-try, array of errors on failure
         */
        abstract public function onSubmit( $data );
index 72b1f88..fc1bfd1 100644 (file)
@@ -896,7 +896,7 @@ abstract class ApiBase extends ContextSource {
        /**
         * Callback function used in requireOnlyOneParameter to check whether required parameters are set
         *
-        * @param  $x object Parameter to check is not null/false
+        * @param $x object Parameter to check is not null/false
         * @return bool
         */
        private function parameterNotEmpty( $x ) {
@@ -1322,8 +1322,8 @@ abstract class ApiBase extends ContextSource {
        /**
         * Adds a warning to the output, else dies
         *
-        * @param  $msg String Message to show as a warning, or error message if dying
-        * @param  $enforceLimits Boolean Whether this is an enforce (die)
+        * @param $msg String Message to show as a warning, or error message if dying
+        * @param $enforceLimits Boolean Whether this is an enforce (die)
         */
        private function warnOrDie( $msg, $enforceLimits = false ) {
                if ( $enforceLimits ) {
index e584d0d..ed64cd5 100644 (file)
@@ -107,8 +107,8 @@ class ApiHelp extends ApiBase {
        }
 
        /**
-        * @param  $module ApiBase
-        * @param  $type String What type of request is this? e.g. action, query, list, prop, meta, format
+        * @param $module ApiBase
+        * @param $type String What type of request is this? e.g. action, query, list, prop, meta, format
         * @return string
         */
        private function buildModuleHelp( $module, $type ) {
index 10b655f..1190781 100644 (file)
@@ -148,8 +148,8 @@ class ApiMove extends ApiBase {
        /**
         * @param Title $fromTitle
         * @param Title $toTitle
-        * @param  $reason
-        * @param  $noredirect
+        * @param $reason
+        * @param $noredirect
         * @return array
         */
        public function moveSubpages( $fromTitle, $toTitle, $reason, $noredirect ) {
index 5f85e0e..ad26cce 100644 (file)
@@ -46,11 +46,11 @@ class ApiQueryRandom extends ApiQueryGeneratorBase {
        }
 
        /**
-        * @param  $randstr
-        * @param  $limit
-        * @param  $namespace
+        * @param $randstr
+        * @param $limit
+        * @param $namespace
         * @param $resultPageSet ApiPageSet
-        * @param  $redirect
+        * @param $redirect
         * @return void
         */
        protected function prepareQuery( $randstr, $limit, $namespace, &$resultPageSet, $redirect ) {
index 0284916..79a8dc9 100644 (file)
@@ -69,8 +69,8 @@ class ApiQueryRecentChanges extends ApiQueryGeneratorBase {
        }
 
        /**
-        * @param  $pageid
-        * @param  $title
+        * @param $pageid
+        * @param $title
         * @param $rc RecentChange (optional)
         * @return bool|string
         */
index 6e0490d..0d13433 100644 (file)
@@ -1133,8 +1133,8 @@ class DatabaseOracle extends DatabaseBase {
 
        /**
         * @throws DBUnexpectedError
-        * @param  string $table
-        * @param  string $field
+        * @param string $table
+        * @param string $field
         * @return ORAField
         */
        function fieldInfo( $table, $field ) {
index 09d18c4..121af04 100644 (file)
@@ -40,7 +40,7 @@ abstract class SqlDataUpdate extends DataUpdate {
        /** @var bool Whether a transaction is open on this object (internal use only!) */
        private $mHasTransaction;
 
-       /** @var  bool Whether this update should be wrapped in a transaction */
+       /** @var bool Whether this update should be wrapped in a transaction */
        protected $mUseTransaction;
 
        /**
index 414b9f8..f77a4eb 100644 (file)
@@ -53,16 +53,16 @@ class DifferenceEngine extends ContextSource {
        /** @var Language */
        protected $mDiffLang;
 
-       /** @var  Title */
+       /** @var Title */
        public $mOldPage;
 
-       /** @var  Title */
+       /** @var Title */
        public $mNewPage;
 
        /** @var Revision */
        public $mOldRev;
 
-       /** @var  Revision */
+       /** @var Revision */
        public $mNewRev;
 
        /** @var bool Have the revisions IDs been loaded */
index 3ab9f5d..0464e03 100644 (file)
@@ -36,7 +36,7 @@
  * @since 1.20
  */
 abstract class FileJournal {
-       /** @var  string */
+       /** @var string */
        protected $backend;
 
        /** @var int */
index 237fa32..7baab66 100644 (file)
@@ -50,7 +50,7 @@ class HTMLCheckField extends HTMLFormField {
        }
 
        /**
-        * @param  $request WebRequest
+        * @param $request WebRequest
         *
         * @return String
         */
index 3cf3188..f802771 100644 (file)
@@ -70,7 +70,7 @@ class HTMLMultiSelectField extends HTMLFormField implements HTMLNestedFilterable
        }
 
        /**
-        * @param  $request WebRequest
+        * @param $request WebRequest
         *
         * @return String
         */
index 564927f..4f501f7 100644 (file)
@@ -60,7 +60,7 @@ class HTMLSelectAndOtherField extends HTMLSelectField {
        }
 
        /**
-        * @param  $request WebRequest
+        * @param $request WebRequest
         *
         * @return Array("<overall message>","<select value>","<text field value>")
         */
index e8bcb5b..21f3b77 100644 (file)
@@ -62,7 +62,7 @@ class HTMLSelectOrOtherField extends HTMLTextField {
        }
 
        /**
-        * @param  $request WebRequest
+        * @param $request WebRequest
         *
         * @return String
         */
index 0e8d1de..2ec4f69 100644 (file)
@@ -358,10 +358,10 @@ class LogPage {
 
        /**
         * @todo Document
-        * @param  string $type
-        * @param  Language|null $lang
-        * @param  Title $title
-        * @param  array $params
+        * @param string $type
+        * @param Language|null $lang
+        * @param Title $title
+        * @param array $params
         * @return string
         */
        protected static function getTitleLink( $type, $lang, $title, &$params ) {
index 15ec794..d296269 100644 (file)
@@ -32,7 +32,7 @@
 class BitmapHandler_ClientOnly extends BitmapHandler {
        /**
         * @param $image File
-        * @param  $params
+        * @param $params
         * @return bool
         */
        function normaliseParams( $image, &$params ) {
index d9ea54c..d39b288 100644 (file)
@@ -102,8 +102,8 @@ class MWMemcached {
        /**
         * Command statistics
         *
-        * @var     array
-        * @access  public
+        * @var array
+        * @access public
         */
        var $stats;
 
@@ -113,112 +113,112 @@ class MWMemcached {
        /**
         * Cached Sockets that are connected
         *
-        * @var     array
-        * @access  private
+        * @var array
+        * @access private
         */
        var $_cache_sock;
 
        /**
         * Current debug status; 0 - none to 9 - profiling
         *
-        * @var     boolean
-        * @access  private
+        * @var boolean
+        * @access private
         */
        var $_debug;
 
        /**
         * Dead hosts, assoc array, 'host'=>'unixtime when ok to check again'
         *
-        * @var     array
-        * @access  private
+        * @var array
+        * @access private
         */
        var $_host_dead;
 
        /**
         * Is compression available?
         *
-        * @var     boolean
-        * @access  private
+        * @var boolean
+        * @access private
         */
        var $_have_zlib;
 
        /**
         * Do we want to use compression?
         *
-        * @var     boolean
-        * @access  private
+        * @var boolean
+        * @access private
         */
        var $_compress_enable;
 
        /**
         * At how many bytes should we compress?
         *
-        * @var     integer
-        * @access  private
+        * @var integer
+        * @access private
         */
        var $_compress_threshold;
 
        /**
         * Are we using persistent links?
         *
-        * @var     boolean
-        * @access  private
+        * @var boolean
+        * @access private
         */
        var $_persistent;
 
        /**
         * If only using one server; contains ip:port to connect to
         *
-        * @var     string
-        * @access  private
+        * @var string
+        * @access private
         */
        var $_single_sock;
 
        /**
         * Array containing ip:port or array(ip:port, weight)
         *
-        * @var     array
-        * @access  private
+        * @var array
+        * @access private
         */
        var $_servers;
 
        /**
         * Our bit buckets
         *
-        * @var     array
-        * @access  private
+        * @var array
+        * @access private
         */
        var $_buckets;
 
        /**
         * Total # of bit buckets we have
         *
-        * @var     integer
-        * @access  private
+        * @var integer
+        * @access private
         */
        var $_bucketcount;
 
        /**
         * # of total servers we have
         *
-        * @var     integer
-        * @access  private
+        * @var integer
+        * @access private
         */
        var $_active;
 
        /**
         * Stream timeout in seconds. Applies for example to fread()
         *
-        * @var     integer
-        * @access  private
+        * @var integer
+        * @access private
         */
        var $_timeout_seconds;
 
        /**
         * Stream timeout in microseconds
         *
-        * @var     integer
-        * @access  private
+        * @var integer
+        * @access private
         */
        var $_timeout_microseconds;
 
@@ -243,7 +243,7 @@ class MWMemcached {
         *
         * @param array $args Associative array of settings
         *
-        * @return  mixed
+        * @return mixed
         */
        public function __construct( $args ) {
                $this->set_servers( isset( $args['servers'] ) ? $args['servers'] : array() );
@@ -708,7 +708,7 @@ class MWMemcached {
         *
         * @param string $sock socket to close
         *
-        * @access  private
+        * @access private
         */
        function _close_sock( $sock ) {
                $host = array_search( $sock, $this->_cache_sock );
@@ -725,8 +725,8 @@ class MWMemcached {
         * @param $sock Integer: socket to connect
         * @param string $host Host:IP to connect to
         *
-        * @return  boolean
-        * @access  private
+        * @return boolean
+        * @access private
         */
        function _connect_sock( &$sock, $host ) {
                list( $ip, $port ) = preg_split( '/:(?=\d)/', $host );
@@ -767,7 +767,7 @@ class MWMemcached {
         *
         * @param string $sock socket to mark as dead
         *
-        * @access  private
+        * @access private
         */
        function _dead_sock( $sock ) {
                $host = array_search( $sock, $this->_cache_sock );
index 6dda64c..dfe3c80 100644 (file)
@@ -27,7 +27,7 @@
  * @ingroup Parser
  */
 class CacheTime {
-       /** @var  array|bool ParserOptions which have been taken into account to
+       /** @var array|bool ParserOptions which have been taken into account to
         * produce output or false if not available.
         */
        public $mUsedOptions;
index ad6bc0f..8fa39fa 100644 (file)
@@ -92,7 +92,7 @@ class CoreParserFunctions {
 
        /**
         * @param $parser Parser
-        * @param  $date
+        * @param $date
         * @param null $defaultPref
         * @return mixed|string
         */
@@ -787,7 +787,7 @@ class CoreParserFunctions {
 
        /**
         * @param $parser Parser
-        * @param  $text
+        * @param $text
         * @return string
         */
        static function anchorencode( $parser, $text ) {
index 0fefe69..d7584ca 100644 (file)
@@ -1731,10 +1731,10 @@ class Parser {
         * @param $url String
         * @return String
         *
-        * @todo  This can merge genuinely required bits in the path or query string,
-        *        breaking legit URLs. A proper fix would treat the various parts of
-        *        the URL differently; as a workaround, just use the output for
-        *        statistical records, not for actual linking/output.
+        * @todo This can merge genuinely required bits in the path or query string,
+        *       breaking legit URLs. A proper fix would treat the various parts of
+        *       the URL differently; as a workaround, just use the output for
+        *       statistical records, not for actual linking/output.
         */
        static function replaceUnusualEscapes( $url ) {
                return preg_replace_callback( '/%[0-9A-Fa-f]{2}/',
index deaf288..d6edb21 100644 (file)
@@ -1619,7 +1619,7 @@ class PPNode_Hash_Tree implements PPNode {
        }
 
        /**
-        * @param  $i
+        * @param $i
         * @return bool
         */
        function item( $i ) {
index 5f3f18e..5b94453 100644 (file)
@@ -97,7 +97,7 @@ class StripState {
        }
 
        /**
-        * @param  $text
+        * @param $text
         * @return mixed
         */
        function unstripBoth( $text ) {
index 4505ee1..b0b2253 100644 (file)
@@ -33,10 +33,10 @@ class RevisionDeleteUser {
 
        /**
         * Update *_deleted bitfields in various tables to hide or unhide usernames
-        * @param  $name String username
-        * @param  $userId Int user id
-        * @param  $op String operator '|' or '&'
-        * @param  $dbw null|DatabaseBase, if you happen to have one lying around
+        * @param $name String username
+        * @param $userId Int user id
+        * @param $op String operator '|' or '&'
+        * @param $dbw null|DatabaseBase, if you happen to have one lying around
         * @return bool
         */
        private static function setUsernameBitfields( $name, $userId, $op, $dbw ) {
index 02e1dda..bea8460 100644 (file)
@@ -207,7 +207,7 @@ class RevisionDeleter {
         * so that this key can be used instead.
         *
         * @param $title Title
-        * @param  $revid
+        * @param $revid
         * @return bool|mixed
         */
        public static function checkRevisionExistence( $title, $revid ) {
index 90f82e0..7496262 100644 (file)
@@ -113,7 +113,7 @@ abstract class FormSpecialPage extends SpecialPage {
 
        /**
         * Process the form on POST submission.
-        * @param  $data Array
+        * @param $data Array
         * @return Bool|Array true for success, false for didn't-try, array of errors on failure
         */
        abstract public function onSubmit( array $data );
index 57f23d3..9b1b2b3 100644 (file)
@@ -35,7 +35,7 @@ class SpecialBlock extends FormSpecialPage {
        /// @var Block::TYPE_ constant
        protected $type;
 
-       /// @var  User|String the previous block target
+       /// @var User|String the previous block target
        protected $previousTarget;
 
        /// @var Bool whether the previous submission of the form asked for HideUser
@@ -593,8 +593,8 @@ class SpecialBlock extends FormSpecialPage {
 
        /**
         * Given the form data, actually implement a block
-        * @param  $data Array
-        * @param  $context IContextSource
+        * @param $data Array
+        * @param $context IContextSource
         * @return Bool|String
         */
        public static function processForm( array $data, IContextSource $context ) {
@@ -947,7 +947,7 @@ class SpecialBlock extends FormSpecialPage {
 
        /**
         * Process the form on POST submission.
-        * @param  $data Array
+        * @param $data Array
         * @return Bool|Array true for success, false for didn't-try, array of errors on failure
         */
        public function onSubmit( array $data ) {
index 29722c8..1bd4228 100644 (file)
@@ -448,8 +448,8 @@ class ImportReporter extends ContextSource {
         * @param Title $title
         * @param Title $origTitle
         * @param int $revisionCount
-        * @param  $successCount
-        * @param  $pageInfo
+        * @param $successCount
+        * @param $pageInfo
         * @return void
         */
        function reportPage( $title, $origTitle, $revisionCount, $successCount, $pageInfo ) {
index db9a992..f8e5c84 100644 (file)
@@ -682,7 +682,7 @@ class UserrightsPage extends SpecialPage {
        }
 
        /**
-        * @param  $group String: the name of the group to check
+        * @param $group String: the name of the group to check
         * @return bool Can we remove the group?
         */
        private function canRemove( $group ) {
index 2b98a18..1ad2b7a 100644 (file)
@@ -814,7 +814,7 @@ class Language {
        }
 
        /**
-        * @param  $image
+        * @param $image
         * @return array|null
         */
        function getImageFile( $image ) {
@@ -829,7 +829,7 @@ class Language {
        }
 
        /**
-        * @param  $tog
+        * @param $tog
         * @return string
         */
        function getUserToggle( $tog ) {
index 3eff534..9f11c63 100644 (file)
@@ -70,7 +70,7 @@ class BenchmarkDeleteTruncate extends Benchmarker {
        }
 
        /**
-        * @param  $dbw DatabaseBase
+        * @param $dbw DatabaseBase
         * @return void
         */
        private function insertData( $dbw ) {
@@ -83,7 +83,7 @@ class BenchmarkDeleteTruncate extends Benchmarker {
        }
 
        /**
-        * @param  $dbw DatabaseBase
+        * @param $dbw DatabaseBase
         * @return void
         */
        private function delete( $dbw ) {
@@ -91,7 +91,7 @@ class BenchmarkDeleteTruncate extends Benchmarker {
        }
 
        /**
-        * @param  $dbw DatabaseBase
+        * @param $dbw DatabaseBase
         * @return void
         */
        private function truncate( $dbw ) {
index 7237ef3..c8ec411 100644 (file)
@@ -30,9 +30,9 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * An error occurred.
         *
-        * @param  PHPUnit_Framework_Test $test
-        * @param  Exception              $e
-        * @param  float                  $time
+        * @param PHPUnit_Framework_Test $test
+        * @param Exception $e
+        * @param float $time
         */
        public function addError( PHPUnit_Framework_Test $test, Exception $e, $time ) {
                wfDebugLog( $this->logChannel, 'ERROR in ' . $this->getTestName( $test ) . ': ' . $this->getErrorName( $e ) );
@@ -41,9 +41,9 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * A failure occurred.
         *
-        * @param  PHPUnit_Framework_Test                 $test
-        * @param  PHPUnit_Framework_AssertionFailedError $e
-        * @param  float                                  $time
+        * @param PHPUnit_Framework_Test $test
+        * @param PHPUnit_Framework_AssertionFailedError $e
+        * @param float $time
         */
        public function addFailure( PHPUnit_Framework_Test $test, PHPUnit_Framework_AssertionFailedError $e, $time ) {
                wfDebugLog( $this->logChannel, 'FAILURE in ' . $this->getTestName( $test ) . ': ' . $this->getErrorName( $e ) );
@@ -52,9 +52,9 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * Incomplete test.
         *
-        * @param  PHPUnit_Framework_Test $test
-        * @param  Exception              $e
-        * @param  float                  $time
+        * @param PHPUnit_Framework_Test $test
+        * @param Exception $e
+        * @param float $time
         */
        public function addIncompleteTest( PHPUnit_Framework_Test $test, Exception $e, $time ) {
                wfDebugLog( $this->logChannel, 'Incomplete test ' . $this->getTestName( $test ) . ': ' . $this->getErrorName( $e ) );
@@ -63,11 +63,11 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * Skipped test.
         *
-        * @param  PHPUnit_Framework_Test $test
-        * @param  Exception              $e
-        * @param  float                  $time
+        * @param PHPUnit_Framework_Test $test
+        * @param Exception $e
+        * @param float $time
         *
-        * @since  Method available since Release 3.0.0
+        * @since Method available since Release 3.0.0
         */
        public function addSkippedTest( PHPUnit_Framework_Test $test, Exception $e, $time ) {
                wfDebugLog( $this->logChannel, 'Skipped test ' . $this->getTestName( $test ) . ': ' . $this->getErrorName( $e ) );
@@ -76,8 +76,8 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * A test suite started.
         *
-        * @param  PHPUnit_Framework_TestSuite $suite
-        * @since  Method available since Release 2.2.0
+        * @param PHPUnit_Framework_TestSuite $suite
+        * @since Method available since Release 2.2.0
         */
        public function startTestSuite( PHPUnit_Framework_TestSuite $suite ) {
                wfDebugLog( $this->logChannel, 'START suite ' . $suite->getName() );
@@ -86,8 +86,8 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * A test suite ended.
         *
-        * @param  PHPUnit_Framework_TestSuite $suite
-        * @since  Method available since Release 2.2.0
+        * @param PHPUnit_Framework_TestSuite $suite
+        * @since Method available since Release 2.2.0
         */
        public function endTestSuite( PHPUnit_Framework_TestSuite $suite ) {
                wfDebugLog( $this->logChannel, 'END suite ' . $suite->getName() );
@@ -96,7 +96,7 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * A test started.
         *
-        * @param  PHPUnit_Framework_Test $test
+        * @param PHPUnit_Framework_Test $test
         */
        public function startTest( PHPUnit_Framework_Test $test ) {
                wfDebugLog( $this->logChannel, 'Start test ' . $this->getTestName( $test ) );
@@ -105,8 +105,8 @@ class MediaWikiPHPUnitTestListener implements PHPUnit_Framework_TestListener {
        /**
         * A test ended.
         *
-        * @param  PHPUnit_Framework_Test $test
-        * @param  float                  $time
+        * @param PHPUnit_Framework_Test $test
+        * @param float $time
         */
        public function endTest( PHPUnit_Framework_Test $test, $time ) {
                wfDebugLog( $this->logChannel, 'End test ' . $this->getTestName( $test ) );