added list/get received by address/ account tests

fixed some annotations and cleaned a bit

received by tests

d
[Qt] Change Coin control labels and tooltips because of non-rounding fees
Fix formatting
remove unused CNode::Cleanup()
add missing vhListenSocket.clear(); to CNetCleanup()
remove unneded class CNodeCombinedStats; from rpcconsole.cpp

- also 2 small style fixes
Merge pull request #4391

b612bde remove unneded class CNodeCombinedStats; from rpcconsole.cpp (Philip Kaufmann)
Merge pull request #4385

040c2d3 Fix formatting (Drak)
Merge pull request #4388

3dc1464 add missing vhListenSocket.clear(); to CNetCleanup() (Philip Kaufmann)
2831a03 remove unused CNode::Cleanup() (Philip Kaufmann)
Avoid undefined behavior using CFlatData in CScript serialization

`&vch[vch.size()]` and even `&vch[0]` on vectors can cause assertion
errors with VC in debug mode. This is the problem mentioned in #4239.
The deeper problem with this is that we rely on undefined behavior.

- Add `begin_ptr` and `end_ptr` functions that get the beginning and end
  pointer of vector in a reliable way that copes with empty vectors and
  doesn't reference outside the vector
(see https://stackoverflow.com/questions/1339470/how-to-get-the-address-of-the-stdvector-buffer-start-most-elegantly/1339767#1339767).
- Add a convenience constructor to CFlatData that wraps a vector.

I added `begin_ptr` and `end_ptr` as separate functions as I imagine
they will be useful in more places.
Merge pull request #4293

fa126ef Avoid undefined behavior using CFlatData in CScript serialization (Wladimir J. van der Laan)
doc: Remove unused section from release-process.md

It is outdated information. If we ever resurrect gitian-downloader it
can be brought back from history and updated.
Merge pull request #4314

b3c912d [Qt] Change Coin control labels and tooltips because of non-rounding fees (Cozz Lovan)
Merge pull request #3960

3f9a019 added list/get received by address/ account tests (Alon Muroch)
qa/rpc_tests: Wait for handshake to complete in connect_nodes

This avoids a race condition in which the connection was
made but the version handshake is not completed yet. In that
case transactions won't be broadcasted to a peer yet, and
the nodes will wait forever for their mempools to sync.
Merge pull request #4394 from laanwj/2014_06_rpc_tests_connectnode

qa/rpc_tests: Wait for handshake to complete in connect_nodes
build: fix build weirdness after 54372482.

bitcoin-config.h moved, but the old file is likely to still exist when
reconfiguring or switching branches. This would've caused files to not rebuild
correctly, and other strange problems.

Make the path explicit so that the old one cannot be found.

Core libs use config/bitcoin-config.h.

Libs (like crypto) which don't want access to bitcoin's headers continue
to use -Iconfig and #include bitcoin-config.h.
Merge pull request #4395

f3967bc build: fix build weirdness after 54372482. (Cory Fields)
replace 3 separate calls to WSAGetLastError() with 1