return inside a block returns from outer method

So no need to check its return value.

Author: egypt

lib/msf/core/module_set.rb

@@ -169,12 +169,14 @@ def on_module_reload(mod)
   end
 
   # @!attribute [rw] postpone_recalc
-  #   Whether or not recalculations should be postponed.  This is used from the context of the each_module_list
-  #   handler in order to prevent the demand # loader from calling recalc for each module if it's possible that more
-  #   than one module may be loaded.  This field is not initialized until used.
+  #   Whether or not recalculations should be postponed.  This is used
+  #   from the context of the {#each_module_list} handler in order to
+  #   prevent the demand loader from calling recalc for each module if
+  #   it's possible that more than one module may be loaded.  This field
+  #   is not initialized until used.
   #
-  #   @return [true] if recalculate should not be called immediately
-  #   @return [false] if recalculate should be called immediately
+  #   @return [true] if {#recalculate} should not be called immediately
+  #   @return [false] if {#recalculate} should be called immediately
   attr_accessor :postpone_recalculate
 
   # Dummy placeholder to recalculate aliases and other fun things.

lib/msf/core/modules/loader/base.rb

@@ -125,7 +125,7 @@ def load_module(parent_path, type, module_reference_name, options={})
 	    return false
     end
 
-    loaded = namespace_module_transaction(type + "/" + module_reference_name, :reload => reload) { |namespace_module|
+    namespace_module_transaction(type + "/" + module_reference_name, :reload => reload) { |namespace_module|
       # set the parent_path so that the module can be reloaded with #load_module
       namespace_module.parent_path = parent_path
 
@@ -180,10 +180,6 @@ def load_module(parent_path, type, module_reference_name, options={})
       true
     }
 
-    unless loaded
-      return false
-    end
-
     if reload
       # Delete the original copy of the module so that module_manager.on_load_module called from inside load_module does
       # not trigger an ambiguous name warning, which would cause the reloaded module to not be stored in the