Small code cleanups to MWCallableUpdate

Change-Id: I269199f65ffc6ad50d16cb984c28b51a815e9a8f

diff --git a/includes/deferred/CallableUpdate.php b/includes/deferred/CallableUpdate.php
index 808626d..4b19c20 100644 (file)
--- a/includes/deferred/CallableUpdate.php
+++ b/includes/deferred/CallableUpdate.php
@@ -4,25 +4,20 @@
  * Deferrable Update for closure/callback
  */
 class MWCallableUpdate implements DeferrableUpdate {
-       /**
-        * @var Closure|callable
-        */
+       /** @var Closure|callable */
        private $callback;
 
        /**
         * @param callable $callback
-        * @throws MWException
+        * @throws InvalidArgumentException
         */
        public function __construct( $callback ) {
                if ( !is_callable( $callback ) ) {
-                       throw new MWException( 'Not a valid callback/closure!' );
+                       throw new InvalidArgumentException( 'Not a valid callback/closure!' );
                }
                $this->callback = $callback;
        }
 
-       /**
-        * Run the update
-        */
        public function doUpdate() {
                call_user_func( $this->callback );
        }