added many rpc wallet tests

deleted the empty no throw test in rpc_wallet_tests line 65

fixed some comments

starting verify tests

finished verify message tests

changed some comments
[Qt] ensure payment request network matches client network

- replaces checks in SendCoinsDialog::handlePaymentRequest() that belong
  to PaymentServer (normal URIs are special cased, as only an isValid
  check is done on BTC addresses)
- prevents the client to handle payment requests that do not match the
  clients network and shows an error instead (mainly a problem with
  drag&drop payment requests onto the client window)
- includes some small comment changes also
Remove unused imports in macdeploy script
doc: Clarify wording about testing in README.md

Weaken and clarify the wording a bit, it currently implies that we get
more pull requests than we can ever handle which discourages
contribution.
[Qt] Fix Transaction details shows wrong To:
rename fNoListen to fListen and move to net

- better code readability and it belongs to net
- this is a prerequisite for a pull to add -listen to the GUI
gitian: Add cfields to gitian download scripts
No references to centralized databases in help text.
Merge pull request #4251 from theuni/cfields-pgp

gitian: Add cfields to gitian download scripts
Fix name of Cory's pgp key
Merge pull request #4249

b5ef85c No references to centralized databases in help text. (Pieter Wuille)
osx: Fix missing dock menu with qt5

Qt5 Removed the qt_mac_set_dock_menu function and left no replacement. It was
later re-added and deprecated for backwards-compatibility.

Qt5.2 adds the non-deprecated QMenu::setAsDockMenu(). Use that when possible.
Merge pull request #4187

d16f6f8 Remove unused imports in macdeploy script (Federico Bond)
Merge pull request #4252

c21c74b osx: Fix missing dock menu with qt5 (Cory Fields)
Limit number of known addresses per peer
Merge pull request #4218

4665b1a doc: Clarify wording about testing in README.md (Wladimir J. van der Laan)
Use pnode->nLastRecv as sync score directly

NodeSyncScore() should find the node which we recv data most recently, so put a negative sign to pnode->nLastRecv is indeed wrong.

Also change the return value type to int64_t.

Signed-off-by: Huang Le <4tarhl@gmail.com>
Remove template matching params from GetOpName()

Since they are not real opcodes, being reported as OP_UNKNOWN is less confusing for human-readable decoding.

Signed-off-by: Huang Le <4tarhl@gmail.com>
remove dup of extern int nConnectTimeout; in netbase
Merge pull request #4264

cb7a3ed remove dup of extern int nConnectTimeout; in netbase (Philip Kaufmann)