AddToWallet implies BindWallet

Now that AddToWallet is called when loading transactions from the
wallet database, BindWallet can be integrated into that and does not
need to be an extra step.

Leaves behaviour unchanged, but makes the
fFromLoadWallet/!fFromLoadWallet paths in AddToWallet a bit more
symmetric.
Prevent socket leak in ThreadSocketHandler.

When we are over our outbound limit ThreadSocketHandler would try to
 keep the connection if the peer was addnoded.

This didn't actually work for two reasons: It didn't actually run
 the accept code due to mistaken code flow, and because we have a
 limited number of outbound semaphores it couldn't actually use the
 connection.

Instead it leaked the socket, which might have caused issue #4034.

This patch just takes out the non-functioning white-listing for now.
Correct some proxy related socket leaks.
Improve and expand base58 comments

update comments so doxygen will pick them up
/contrib index reorg

Cleaner and easier to read.

Reordered by functionality
Merge pull request #4037 from gmaxwell/fdleaks

Prevent socket leak in ThreadSocketHandler.
Merge pull request #3701

09ec3af AddToWallet implies BindWallet (Wladimir J. van der Laan)
Merge pull request #4014

4e9667b Improve and expand base58 comments (rxl)
Merge pull request #4047

494cbf8 /contrib index reorg (super3)
Use a typedef for monetary values
Encapsulate coin balances within a new CMoney type

Provides increased type safety of monetary calculations in bitcoin. Also brings code parity between bitcoin and side chains which use a different type for representing and performing arithmetic on coin balances, e.g. Freicoin's use of the GMP library's arbitrary-precision rational number type. This greatly reduces the friction in sharing code and submitting code upsream.