rapid7/metasploit-framework — pull request #3332 by kernelsmith (2014-05-02 23:57:46+00:00)
Title: cleans up numerous superfluous returns in msf/core/module

while making changes to support running a local exploit as an AutoRunScript, I noticed lib/msf/core/module.rb had numerous superfluous return statements and some unnecessarily unwieldy if statements. There's only a few instances where the logic could be affected. I tested those by hand to make sure I wasn't crazy.
Since the framework is so dependent on module.rb, verification should be as simple as

[ ] Travis passes
[ ] msfconsole launches and you can use it normally, load modules, run them etc.