Needs rebase, but ut ACK.

Agree on removing the VerifySignature function that is only used by the tests.

NACK on partially removing CScriptCheck::nHashType. Either it should be removed completely and this removal propagated through VerifyScript, or it would be better to keep it like this. Mind that CScriptCheck is meant to be a closure around VerifyScript (so should expose the same arguments). Now you just push the hardcoded 0 to CScriptCheck::operator()().

Oh, that's what you do in #4555, well then it's fine with me.

Rebased

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4775_5ad29370edc22558539bd89135c02d3de4c7bc52/ for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

ut ACK.

Included in #4555, after #4754 has been merged there's no point in keeping this one separated.