Agree. On #2124 RPORT can be initialized via DefaultOptions too, since it's already registered. According to the last discussions on #metasploit around  DefaultOptions vs ReRegister, using DefaultOptions is okey when the option is already registered. Actually I use to be more comfortable with DefaultOptions over RE-registering when the options are already registered, with these exceptions:

* When the "semantics" of the Option must be changed, for example, make the description a little more accurate for an specific module.
* When there are Opt builtin methods: RHOST, RPORT, LHOST, LPORT, CPORT, CHOST, Proxies, and these builtin methods are used. Because the semantics/properties of the options aren't changed at all when re-registering. And also because using them looks like more compliant with the "msf style". It is, most modules reregister options with builtin methods.

Just my actual feelings, and yeah, they are just personal, and no strict... even for me which is weird :S