No need to return true/false from AutoLoader
authorKevin Israel <pleasestand@live.com>
Sat, 4 Jan 2014 19:25:44 +0000 (14:25 -0500)
committerKevin Israel <pleasestand@live.com>
Mon, 6 Jan 2014 02:21:53 +0000 (21:21 -0500)
Partially reverts r36942 (7e627faec6bd).

Change-Id: I5b3dfd148863413025c71de5fadef1cb5bd70dce

includes/AutoLoader.php

index a9b0376..cd062e0 100644 (file)
@@ -1157,9 +1157,6 @@ class AutoLoader {
         * autoload - take a class name and attempt to load it
         *
         * @param string $className name of class we're looking for.
-        * @return bool Returning false is important on failure as
-        * it allows Zend to try and look in other registered autoloaders
-        * as well.
         */
        static function autoload( $className ) {
                global $wgAutoloadClasses, $wgAutoloadLocalClasses,
@@ -1209,7 +1206,7 @@ class AutoLoader {
                        }
 
                        # Give up
-                       return false;
+                       return;
                }
 
                # Make an absolute path, this improves performance by avoiding some stat calls
@@ -1219,8 +1216,6 @@ class AutoLoader {
                }
 
                require $filename;
-
-               return true;
        }
 
        /**