fix minor formatting
Merge pull request #366 from vikramrajkumar/master

Sort account and wallet name lists
format balances
fix duplicate names, break good code architecture
Configure the user appender to re-enable status messages
fix bad assumption on get_head_block_num()+1 for pending_chain_state, added warning to remind us to remove code for next dry run
Merge branch 'master' of github.com:BitShares/bitshares_toolkit
Fix variables that were uninitialized when accepting a network connection (and therefore caused nonsensical data to be displayedd for firewalled status for incoming connections)
Register regression tests as individual tests
Update submodule
Fix test names
Change upnp messages to use ulog()
Return names in blockchain_list_current_round_active_delegates
Suppress duplicate "now connected to n peers" messages
Update submodule
Refactor wallet import/export to JSON (#66)
Fix error in new splitting of wallet tests, assertion
When we log messages to the error database, also log them to our error log streams
Disable upnp in existing regression tests
Ticket #237 clean up gracefully without crashing

I believe this commit fixes everything except in the async
accept_connection_task() in node_impl::accept_loop()