Kill "* @return void"
authorSam Reed <reedy@users.mediawiki.org>
Thu, 12 Jan 2012 19:12:24 +0000 (19:12 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Thu, 12 Jan 2012 19:12:24 +0000 (19:12 +0000)
54 files changed:
includes/HistoryBlob.php
includes/Hooks.php
includes/ImagePage.php
includes/Pager.php
includes/Preferences.php
includes/SquidPurgeClient.php
includes/Title.php
includes/TitleArray.php
includes/UserArray.php
includes/Wiki.php
includes/WikiPage.php
includes/api/ApiImport.php
includes/api/ApiQueryAllLinks.php
includes/api/ApiQueryAllimages.php
includes/api/ApiQueryAllpages.php
includes/api/ApiQueryBacklinks.php
includes/api/ApiQueryBase.php
includes/api/ApiQueryCategoryMembers.php
includes/api/ApiQueryExtLinksUsage.php
includes/api/ApiQueryIWBacklinks.php
includes/api/ApiQueryInfo.php
includes/api/ApiQueryLangBacklinks.php
includes/api/ApiQueryProtectedTitles.php
includes/api/ApiQueryRandom.php
includes/api/ApiQuerySearch.php
includes/api/ApiQueryWatchlist.php
includes/api/ApiQueryWatchlistRaw.php
includes/cache/FileCacheBase.php
includes/cache/HTMLFileCache.php
includes/cache/SquidUpdate.php
includes/db/CloneDatabase.php
includes/db/Database.php
includes/filerepo/FileRepo.php
includes/filerepo/LocalRepo.php
includes/filerepo/backend/FileBackend.php
includes/filerepo/backend/FileBackendGroup.php
includes/filerepo/backend/FileOp.php
includes/filerepo/backend/lockmanager/DBLockManager.php
includes/filerepo/backend/lockmanager/LockManagerGroup.php
includes/filerepo/file/File.php
includes/filerepo/file/TempFSFile.php
includes/media/GIFMetadataExtractor.php
includes/media/MediaTransformOutput.php
includes/parser/CoreParserFunctions.php
includes/parser/CoreTagHooks.php
includes/parser/ParserOutput.php
includes/resourceloader/ResourceLoader.php
includes/specials/SpecialImport.php
includes/specials/SpecialMergeHistory.php
includes/specials/SpecialShortpages.php
includes/specials/SpecialUserlogin.php
maintenance/benchmarks/bench_delete_truncate.php
maintenance/locking/LockServerDaemon.php
thumb.php

index f707b3f..bf4103e 100644 (file)
@@ -94,7 +94,6 @@ class ConcatenatedGzipHistoryBlob implements HistoryBlob
 
        /**
         * @param $text string
-        * @return void
         */
        public function setText( $text ) {
                $this->uncompress();
index e1c1d50..2a6e5cb 100644 (file)
@@ -39,7 +39,6 @@ class Hooks {
         *
         * @param $name Mixed: name of hook
         * @param $callback Mixed: callback function to attach
-        * @return void
         */
        public static function register( $name, $callback ) {
                if( !isset( self::$handlers[$name] ) ) {
index a44a7d3..ef72414 100644 (file)
@@ -39,8 +39,7 @@ class ImagePage extends Article {
        }
 
        /**
-        * @param $file File:
-        * @return void
+        * @param $file File
         */
        public function setFile( $file ) {
                $this->mPage->setFile( $file );
index a03a1d1..ee7bde3 100644 (file)
@@ -394,8 +394,6 @@ abstract class IndexPager extends ContextSource implements Pager {
         * Called from getBody(), before getStartBody() is called and
         * after doQuery() was called. This will be called only if there
         * are rows in the result set.
-        *
-        * @return void
         */
        protected function doBatchLookups() {}
 
index fde135b..153984b 100644 (file)
@@ -129,7 +129,6 @@ class Preferences {
         * @param $user User
         * @param $context IContextSource
         * @param $defaultPreferences
-        * @return void
         */
        static function profilePreferences( $user, IContextSource $context, &$defaultPreferences ) {
                global $wgAuth, $wgContLang, $wgParser, $wgCookieExpiration, $wgLanguageCode,
@@ -456,7 +455,6 @@ class Preferences {
         * @param $user User
         * @param $context IContextSource
         * @param $defaultPreferences
-        * @return void
         */
        static function skinPreferences( $user, IContextSource $context, &$defaultPreferences ) {
                ## Skin #####################################
@@ -532,7 +530,6 @@ class Preferences {
         * @param $user User
         * @param $context IContextSource
         * @param $defaultPreferences
-        * @return void
         */
        static function datetimePreferences( $user, IContextSource $context, &$defaultPreferences ) {
                ## Date and time #####################################
index 506ada9..6f45f20 100644 (file)
@@ -340,7 +340,6 @@ class SquidPurgeClientPool {
 
        /**
         * @param $client SquidPurgeClient
-        * @return void
         */
        public function addClient( $client ) {
                $this->clients[] = $client;
index 2f285bb..e034867 100644 (file)
@@ -261,7 +261,6 @@ class Title {
         * If false is given, the title will be treated as non-existing.
         *
         * @param $row Object|false database row
-        * @return void
         */
        public function loadFromRow( $row ) {
                if ( $row ) { // page found
index 9696008..72717c2 100644 (file)
@@ -53,7 +53,6 @@ class TitleArrayFromResult extends TitleArray {
 
        /**
         * @param $row ResultWrapper
-        * @return void
         */
        protected function setCurrent( $row ) {
                if ( $row === false ) {
index c5ba0b2..89d9374 100644 (file)
@@ -60,7 +60,6 @@ class UserArrayFromResult extends UserArray {
 
        /**
         * @param  $row
-        * @return void
         */
        protected function setCurrent( $row ) {
                if ( $row === false ) {
index 55ff89e..8153eae 100644 (file)
@@ -145,8 +145,6 @@ class MediaWiki {
         * - redirect loop
         * - special pages
         * - normal pages
-        *
-        * @return void
         */
        private function performRequest() {
                global $wgServer, $wgUsePathInfo, $wgTitle;
index 83f73e0..4e9e3f5 100644 (file)
@@ -367,7 +367,6 @@ class WikiPage extends Page {
         *              A DB query result object or...
         *              "fromdb" to get from a slave DB or...
         *              "fromdbmaster" to get from the master DB
-        * @return void
         */
        public function loadPageData( $data = 'fromdb' ) {
                if ( $data === 'fromdbmaster' ) {
@@ -622,7 +621,6 @@ class WikiPage extends Page {
        /**
         * Set the page timestamp (use only to avoid DB queries)
         * @param $ts string MW timestamp of last article revision
-        * @return void
         */
        public function setTimestamp( $ts ) {
                $this->mTimestamp = wfTimestamp( TS_MW, $ts );
@@ -705,7 +703,6 @@ class WikiPage extends Page {
         * Set the cached timestamp for the last time the page changed.
         * This is only used to help handle slave lag by comparing to page_touched.
         * @param $timestamp string
-        * @return void
         */
        public function setCachedLastEditTime( $timestamp ) {
                global $wgMemc;
index ade9f1f..527a0f0 100644 (file)
@@ -181,7 +181,6 @@ class ApiImportReporter extends ImportReporter {
         * @param $revisionCount int
         * @param $successCount int
         * @param $pageInfo
-        * @return void
         */
        function reportPage( $title, $origTitle, $revisionCount, $successCount, $pageInfo ) {
                // Add a result entry
index 903f144..127d359 100644 (file)
@@ -49,7 +49,6 @@ class ApiQueryAllLinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $db = $this->getDB();
index ca344f7..fec4ed4 100644 (file)
@@ -61,7 +61,6 @@ class ApiQueryAllimages extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        public function executeGenerator( $resultPageSet ) {
                if ( $resultPageSet->isResolvingRedirects() ) {
@@ -73,7 +72,6 @@ class ApiQueryAllimages extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $repo = $this->mRepo;
index e003ee9..e429a37 100644 (file)
@@ -45,7 +45,6 @@ class ApiQueryAllpages extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        public function executeGenerator( $resultPageSet ) {
                if ( $resultPageSet->isResolvingRedirects() ) {
@@ -57,7 +56,6 @@ class ApiQueryAllpages extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $db = $this->getDB();
index 381ef55..3dbc437 100644 (file)
@@ -119,7 +119,6 @@ class ApiQueryBacklinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function prepareFirstQuery( $resultPageSet = null ) {
                /* SELECT page_id, page_title, page_namespace, page_is_redirect
@@ -161,7 +160,6 @@ class ApiQueryBacklinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function prepareSecondQuery( $resultPageSet = null ) {
                /* SELECT page_id, page_title, page_namespace, page_is_redirect, pl_title, pl_namespace
@@ -223,7 +221,6 @@ class ApiQueryBacklinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $this->params = $this->extractRequestParams( false );
index 4fe82de..add7417 100644 (file)
@@ -505,7 +505,6 @@ abstract class ApiQueryBase extends ApiBase {
         * Also adds relevant block information
         *
         * @param bool $showBlockInfo
-        * @return void
         */
        public function showHiddenUsersAddBlockInfo( $showBlockInfo ) {
                $userCanViewHiddenUsers = $this->getUser()->isAllowed( 'hideuser' );
index 4b19b7e..1beadb3 100644 (file)
@@ -49,7 +49,6 @@ class ApiQueryCategoryMembers extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index 93c71e2..fca4a55 100644 (file)
@@ -47,7 +47,6 @@ class ApiQueryExtLinksUsage extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index feda177..ff17650 100644 (file)
@@ -45,7 +45,6 @@ class ApiQueryIWBacklinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        public function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index f0d0faa..a480c1b 100644 (file)
@@ -51,7 +51,6 @@ class ApiQueryInfo extends ApiQueryBase {
 
        /**
         * @param $pageSet ApiPageSet
-        * @return void
         */
        public function requestExtraData( $pageSet ) {
                global $wgDisableCounters;
index 1573494..c98e8da 100644 (file)
@@ -45,7 +45,6 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        public function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index 44cc1d3..eba16ea 100644 (file)
@@ -45,7 +45,6 @@ class ApiQueryProtectedTitles extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index 2e9e2dd..2245f00 100644 (file)
@@ -46,12 +46,11 @@ class ApiQueryRandom extends ApiQueryGeneratorBase {
        }
 
        /**
-        * @param  $randstr
-        * @param  $limit
-        * @param  $namespace
+        * @param $randstr
+        * @param $limit
+        * @param $namespace
         * @param $resultPageSet ApiPageSet
-        * @param  $redirect
-        * @return void
+        * @param $redirect
         */
        protected function prepareQuery( $randstr, $limit, $namespace, &$resultPageSet, $redirect ) {
                $this->resetQueryParams();
@@ -101,7 +100,6 @@ class ApiQueryRandom extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        public function run( $resultPageSet = null ) {
                $params = $this->extractRequestParams();
index 40aac05..8e04e88 100644 (file)
@@ -45,7 +45,6 @@ class ApiQuerySearch extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                global $wgContLang;
index ea56fcd..cbe50de 100644 (file)
@@ -50,7 +50,6 @@ class ApiQueryWatchlist extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $this->selectNamedDB( 'watchlist', DB_SLAVE, 'watchlist' );
index 506944f..2fad137 100644 (file)
@@ -46,7 +46,6 @@ class ApiQueryWatchlistRaw extends ApiQueryGeneratorBase {
 
        /**
         * @param $resultPageSet ApiPageSet
-        * @return void
         */
        private function run( $resultPageSet = null ) {
                $this->selectNamedDB( 'watchlist', DB_SLAVE, 'watchlist' );
index d355818..ab7d8de 100644 (file)
@@ -151,7 +151,6 @@ abstract class FileCacheBase {
 
        /**
         * Clear the cache for this page
-        * @return void
         */
        public function clearCache() {
                wfSuppressWarnings();
@@ -162,7 +161,6 @@ abstract class FileCacheBase {
 
        /**
         * Create parent directors of $this->cachePath()
-        * @return void
         */
        protected function checkCacheDirs() {
                wfMkdirParents( dirname( $this->cachePath() ), null, __METHOD__ );
@@ -198,7 +196,6 @@ abstract class FileCacheBase {
        /**
         * Roughly increments the cache misses in the last hour by unique visitors
         * @param $request WebRequest
-        * @return void
         */
        public function incrMissesRecent( WebRequest $request ) {
                global $wgMemc;
index b846e41..8866ba3 100644 (file)
@@ -92,7 +92,6 @@ class HTMLFileCache extends FileCacheBase {
        /**
         * Read from cache to context output
         * @param $context IContextSource
-        * @return void
         */
        public function loadFromFileCache( IContextSource $context ) {
                global $wgMimeType, $wgLanguageCode;
index d47b5b5..af620e0 100644 (file)
@@ -99,7 +99,6 @@ class SquidUpdate {
         * XXX report broken Squids per mail or log
         *
         * @param $urlArr array
-        * @return void
         */
        static function purge( $urlArr ) {
                global $wgSquidServers, $wgHTCPMulticastAddress, $wgHTCPPort;
index bd0895c..5a8232e 100644 (file)
@@ -86,18 +86,18 @@ class CloneDatabase {
         * Clone the table structure
         */
        public function cloneTableStructure() {
-               
+
                foreach( $this->tablesToClone as $tbl ) {
                        # Clean up from previous aborted run.  So that table escaping
                        # works correctly across DB engines, we need to change the pre-
                        # fix back and forth so tableName() works right.
-                       
+
                        self::changePrefix( $this->oldTablePrefix );
                        $oldTableName = $this->db->tableName( $tbl, 'raw' );
-                       
+
                        self::changePrefix( $this->newTablePrefix );
                        $newTableName = $this->db->tableName( $tbl, 'raw' );
-                       
+
                        if( $this->dropCurrentTables && !in_array( $this->db->getType(), array( 'postgres', 'oracle' ) ) ) {
                                $this->db->dropTable( $tbl, __METHOD__ );
                                wfDebug( __METHOD__." dropping {$newTableName}\n", true);
@@ -107,9 +107,9 @@ class CloneDatabase {
                        # Create new table
                        wfDebug( __METHOD__." duplicating $oldTableName to $newTableName\n", true );
                        $this->db->duplicateTableStructure( $oldTableName, $newTableName, $this->useTemporaryTables );
-                       
+
                }
-               
+
        }
 
        /**
@@ -129,8 +129,7 @@ class CloneDatabase {
        /**
         * Change the table prefix on all open DB connections/
         *
-        * @param  $prefix
-        * @return void
+        * @param $prefix
         */
        public static function changePrefix( $prefix ) {
                global $wgDBprefix;
@@ -139,18 +138,16 @@ class CloneDatabase {
        }
 
        /**
-        * @param  $lb LoadBalancer
-        * @param  $prefix
-        * @return void
+        * @param $lb LoadBalancer
+        * @param $prefix
         */
        public static function changeLBPrefix( $lb, $prefix ) {
                $lb->forEachOpenConnection( array( 'CloneDatabase', 'changeDBPrefix' ), array( $prefix ) );
        }
 
        /**
-        * @param  $db DatabaseBase
-        * @param  $prefix
-        * @return void
+        * @param $db DatabaseBase
+        * @param $prefix
         */
        public static function changeDBPrefix( $db, $prefix ) {
                $db->tablePrefix( $prefix );
index 5c186df..c94f5ce 100644 (file)
@@ -3052,7 +3052,6 @@ abstract class DatabaseBase implements DatabaseType {
         * Override database's default connection timeout
         *
         * @param $timeout Integer in seconds
-        * @return void
         * @deprecated since 1.19; use setSessionOptions()
         */
        public function setTimeout( $timeout ) {
@@ -3068,7 +3067,6 @@ abstract class DatabaseBase implements DatabaseType {
         *                     hours or days.
         *
         * @param $options Array
-        * @return void
         */
        public function setSessionOptions( array $options ) {}
 
index cd9b2bb..1a51020 100644 (file)
@@ -126,7 +126,6 @@ class FileRepo {
         * directories, in case the user has not configured offline storage
         *
         * @param $dir string
-        * @return void
         */
        protected function initDeletedDir( $dir ) {
                $this->backend->secure( // prevent web access & dir listings
@@ -718,8 +717,7 @@ class FileRepo {
         * Each file can be a (zone, rel) pair, virtual url, storage path, or FS path.
         * It will try to delete each file, but ignores any errors that may occur.
         *
-        * @param $pairs array List of files to delete
-        * @return void
+        * @param $files array List of files to delete
         */
        public function cleanupBatch( $files ) {
                $operations = array();
@@ -1217,7 +1215,6 @@ class FileRepo {
         * May use either the database or the filesystem.
         *
         * @param $callback Array|string
-        * @return void
         */
        public function enumFiles( $callback ) {
                $this->enumFilesInStorage( $callback );
@@ -1228,7 +1225,6 @@ class FileRepo {
         * May use either the database or the filesystem.
         *
         * @param $callback Array|string
-        * @return void
         */
        protected function enumFilesInStorage( $callback ) {
                $publicRoot = $this->getZonePath( 'public' );
index 92d8f5d..a5607df 100644 (file)
@@ -261,7 +261,6 @@ class LocalRepo extends FileRepo {
         * Invalidates image redirect cache related to that image
         *
         * @param $title Title of page
-        * @return void
         */
        function invalidateImageRedirect( Title $title ) {
                global $wgMemc;
index d971098..958c56e 100644 (file)
@@ -515,7 +515,6 @@ abstract class FileBackendBase {
         * If $paths is given, then only the cache for those files will be cleared.
         *
         * @param $paths Array Storage paths
-        * @return void
         */
        abstract public function clearCache( array $paths = null );
 
@@ -1164,8 +1163,6 @@ abstract class FileBackend extends FileBackendBase {
 
        /**
         * Prune the cache if it is too big to add an item
-        * 
-        * @return void
         */
        protected function trimCache() {
                if ( count( $this->cache ) >= $this->maxCacheSize ) {
index 24277b1..0b97235 100644 (file)
@@ -32,8 +32,6 @@ class FileBackendGroup {
 
        /**
         * Destroy the singleton instance
-        * 
-        * @return void
         */
        public static function destroySingleton() {
                self::$instance = null;
@@ -41,8 +39,6 @@ class FileBackendGroup {
 
        /**
         * Register file backends from the global variables
-        * 
-        * @return void
         */
        protected function initFromGlobals() {
                global $wgLocalFileRepo, $wgForeignFileRepos, $wgFileBackends;
@@ -93,7 +89,6 @@ class FileBackendGroup {
         * Register an array of file backend configurations
         *
         * @param $configs Array
-        * @return void
         * @throws MWException
         */
        protected function register( array $configs ) {
index 11fe865..858c9ea 100644 (file)
@@ -54,8 +54,6 @@ abstract class FileOp {
 
        /**
         * Disable file backups for this operation
-        *
-        * @return void
         */
        final protected function disableBackups() {
                $this->useBackups = false;
@@ -65,8 +63,6 @@ abstract class FileOp {
         * Allow stale data for file reads and existence checks.
         * If this is called, then disableBackups() should also be called
         * unless the affected files are known to have not changed recently.
-        *
-        * @return void
         */
        final protected function allowStaleReads() {
                $this->useLatest = false;
@@ -504,7 +500,6 @@ abstract class FileOp {
         * Log a file operation failure and preserve any temp files
         * 
         * @param $fileOp FileOp
-        * @return void
         */
        final protected function logFailure( $action ) {
                $params = $this->params;
index ac9c079..331a7ad 100644 (file)
@@ -281,7 +281,6 @@ class DBLockManager extends LockManager {
         *
         * @param $lockDb string
         * @param $db DatabaseBase
-        * @return void
         * @throws DBError
         */
        protected function initConnection( $lockDb, DatabaseBase $db ) {}
index e501a0f..a97bc2d 100644 (file)
@@ -27,8 +27,6 @@ class LockManagerGroup {
 
        /**
         * Register lock managers from the global variables
-        * 
-        * @return void
         */
        protected function initFromGlobals() {
                global $wgLockManagers;
@@ -40,7 +38,6 @@ class LockManagerGroup {
         * Register an array of file lock manager configurations
         *
         * @param $configs Array
-        * @return void
         * @throws MWException
         */
        protected function register( array $configs ) {
index 67345a6..b130d8c 100644 (file)
@@ -1694,8 +1694,7 @@ abstract class File {
        }
 
        /**
-        * @param  $from
-        * @return void
+        * @param $from
         */
        function redirectedFrom( $from ) {
                $this->redirected = $from;
index bee00fa..ab9402b 100644 (file)
@@ -66,7 +66,6 @@ class TempFSFile extends FSFile {
         * Clean up the temporary file only after an object goes out of scope
         *
         * @param $object Object
-        * @return void
         */
        public function bind( $object ) {
                if ( is_object( $object ) ) {
@@ -76,8 +75,6 @@ class TempFSFile extends FSFile {
 
        /**
         * Set flag to not clean up after the temporary file
-        *
-        * @return void
         */
        public function preserve() {
                $this->canDelete = false;
index 5dbeb8f..8e6fd74 100644 (file)
@@ -233,7 +233,6 @@ class GIFMetadataExtractor {
        /**
         * @param $fh
         * @param $bpp
-        * @return void
         */
        static function readGCT( $fh, $bpp ) {
                if ( $bpp > 0 ) {
index de6b1c0..0719b50 100644 (file)
@@ -50,7 +50,6 @@ abstract class MediaTransformOutput {
 
        /**
         * @param $storagePath string The permanent storage path
-        * @return void
         */
        public function setStoragePath( $storagePath ) {
                $this->storagePath = $storagePath;
index cd5e0a5..fb3b2d6 100644 (file)
@@ -12,7 +12,6 @@
 class CoreParserFunctions {
        /**
         * @param $parser Parser
-        * @return void
         */
        static function register( $parser ) {
                global $wgAllowDisplayTitle, $wgAllowSlowParserFunctions;
index 7d488c4..86b66b9 100644 (file)
@@ -12,7 +12,6 @@
 class CoreTagHooks {
        /**
         * @param $parser Parser
-        * @return void
         */
        static function register( $parser ) {
                global $wgRawHtml;
index 2877dcb..a5d2a4b 100644 (file)
@@ -286,7 +286,6 @@ class ParserOutput extends CacheTime {
         * @param $name string Title dbKey
         * @param $timestamp string MW timestamp of file creation (or false if non-existing)
         * @param $sha string base 36 SHA-1 of file (or false if non-existing)
-        * @return void
         */
        function addImage( $name, $timestamp = null, $sha1 = null ) {
                $this->mImages[$name] = 1;
@@ -300,7 +299,6 @@ class ParserOutput extends CacheTime {
         * @param $title Title
         * @param $page_id
         * @param $rev_id
-        * @return void
         */
        function addTemplate( $title, $page_id, $rev_id ) {
                $ns = $title->getNamespace();
index 0d1eef0..aa61063 100644 (file)
@@ -521,7 +521,6 @@ class ResourceLoader {
         * @param $context ResourceLoaderContext
         * @param $mtime string TS_MW timestamp to use for last-modified
         * @param $private bool True iff response contains any private modules
-        * @return void
         */
        protected function sendResponseHeaders( ResourceLoaderContext $context, $mtime, $private ) {
                global $wgResourceLoaderMaxage;
index a2380fb..3614504 100644 (file)
@@ -349,7 +349,6 @@ class ImportReporter extends ContextSource {
         * @param int $revisionCount
         * @param  $successCount
         * @param  $pageInfo
-        * @return void
         */
        function reportPage( $title, $origTitle, $revisionCount, $successCount, $pageInfo ) {
                global $wgContLang;
index 19650da..70e24d2 100644 (file)
@@ -39,9 +39,6 @@ class SpecialMergeHistory extends SpecialPage {
                parent::__construct( 'MergeHistory', 'mergehistory' );
        }
 
-       /**
-        * @return void
-        */
        private function loadRequestParams() {
                $request = $this->getRequest();
                $this->mAction = $request->getVal( 'action' );
index c176f91..c2157df 100644 (file)
@@ -56,7 +56,6 @@ class ShortPagesPage extends QueryPage {
        /**
         * @param $db DatabaseBase
         * @param $res
-        * @return void
         */
        function preprocessResults( $db, $res ) {
                # There's no point doing a batch check if we aren't caching results;
index 4882058..572cca5 100644 (file)
@@ -656,7 +656,6 @@ class LoginForm extends SpecialPage {
        /**
         * Clear the login attempt throttle hit count for the (username,current IP) tuple.
         * @param $username string The user name
-        * @return void
         */
        public static function clearLoginThrottle( $username ) {
                global $wgMemc, $wgRequest;
index 9fe9bea..377948b 100644 (file)
@@ -44,8 +44,7 @@ class BenchmarkDeleteTruncate extends Benchmarker {
        }
 
        /**
-        * @param  $dbw DatabaseBase
-        * @return void
+        * @param $dbw DatabaseBase
         */
        private function insertData( $dbw ) {
                $range = range( 0, 1024 );
@@ -57,16 +56,14 @@ class BenchmarkDeleteTruncate extends Benchmarker {
        }
 
        /**
-        * @param  $dbw DatabaseBase
-        * @return void
+        * @param $dbw DatabaseBase
         */
        private function delete( $dbw ) {
                $dbw->delete( 'text', '*', __METHOD__ );
        }
 
        /**
-        * @param  $dbw DatabaseBase
-        * @return void
+        * @param $dbw DatabaseBase
         */
        private function truncate( $dbw ) {
                $test = $dbw->tableName( 'test' );
index 2cb6eb2..73868ec 100644 (file)
@@ -101,9 +101,6 @@ class LockServerDaemon {
                        : 10;
        }
 
-       /**
-        * @return void
-        */
        protected function setupSocket() {
                if ( !function_exists( 'socket_create' ) ) {
                        throw new Exception( "PHP sockets extension missing from PHP CLI mode." );
@@ -125,9 +122,6 @@ class LockServerDaemon {
                $this->startTime = time();
        }
 
-       /**
-        * @return void
-        */
        public function main() {
                // Setup socket and start listing
                $this->setupSocket();
@@ -354,8 +348,6 @@ class LockServerDaemon {
 
        /**
         * Clear locks for sessions that have been dead for a while
-        *
-        * @return void
         */
        protected function purgeExpiredLocks() {
                $now = time();
@@ -370,7 +362,6 @@ class LockServerDaemon {
        /**
         * @param $key string
         * @param $session string
-        * @return void
         */
        protected function set_sh_lock( $key, $session ) {
                if ( !isset( $this->shLocks[$key][$session] ) ) {
@@ -383,7 +374,6 @@ class LockServerDaemon {
        /**
         * @param $key string
         * @param $session string
-        * @return void
         */
        protected function set_ex_lock( $key, $session ) {
                if ( !isset( $this->exLocks[$key][$session] ) ) {
@@ -396,7 +386,6 @@ class LockServerDaemon {
        /**
         * @param $key string
         * @param $session string
-        * @return void
         */
        protected function unset_sh_lock( $key, $session ) {
                if ( isset( $this->shLocks[$key][$session] ) ) {
@@ -415,7 +404,6 @@ class LockServerDaemon {
        /**
         * @param $key string
         * @param $session string
-        * @return void
         */
        protected function unset_ex_lock( $key, $session ) {
                if ( isset( $this->exLocks[$key] ) && $this->exLocks[$key] === $session ) {
index 51f7ec6..758a05a 100644 (file)
--- a/thumb.php
+++ b/thumb.php
@@ -29,8 +29,6 @@ wfLogProfilingData();
 
 /**
  * Handle a thumbnail request via query parameters
- *
- * @return void
  */
 function wfThumbHandleRequest() {
        $params = get_magic_quotes_gpc()
@@ -42,8 +40,6 @@ function wfThumbHandleRequest() {
 
 /**
  * Handle a thumbnail request via thumbnail file URL
- *
- * @return void
  */
 function wfThumbHandle404() {
        # lighttpd puts the original request in REQUEST_URI, while
@@ -69,7 +65,6 @@ function wfThumbHandle404() {
  * Stream a thumbnail specified by parameters
  *
  * @param $params Array
- * @return void
  */
 function wfStreamThumb( array $params ) {
        wfProfileIn( __METHOD__ );
@@ -266,7 +261,6 @@ function wfExtractThumbParams( $uri ) {
  *
  * @param $status integer
  * @param $msg string
- * @return void
  */
 function wfThumbError( $status, $msg ) {
        global $wgShowHostnames;