connection: Move makeconn() up, hinting at its purpose via DBU
connection: Fix return type of makeconn so this actually compiles
connection: Add comment, deal with it later
connection: Move the setting of worldname into makeconn
connection: Formatting/order change, no significance
connection: Initialize buffers in makeconn()
connection: Initialize curmsg in makeconn
connection: Use makeconn for passive connections.

It's the moment you've all been waiting for, folks! :) This unifies two pieces
of code, and notably, fixes a bug in sockaccept() wherein it would emit a null
color object, support for which has since been dropped. It also allows further
improvements in the future... some of which will be the extremely near future.
connection: Downgrade the TODO as part of that is now dealt with