spv: removed reject on invalid block

[edtubbs]

This change removes the transmission of a reject message to a node when invalid or out-of-order blocks are received. A reject should not be sent in response to blocks that are valid but out of order.

[edtubbs]

w/o reject

Downloaded new block with size 190 at height 500 from Sat Dec  7 22:11:26 2013

DEBUG: Start parsing 1 transactions...
DEBUG: done (took 0 secs)
Sync completed, at height 500
Waiting for new blocks or relevant transactions...
DEBUG: received command from node 23: inv
DEBUG: Get inv request with 1 items
DEBUG: Requesting 1 blocks
DEBUG: sending message to node 23: getdata
DEBUG: received command from node 23: block
DEBUG: Got invalid block (not in sequence) from node 23
DEBUG: received command from node 16: addr
DEBUG: received command from node 16: feefilter
DEBUG: received command from node 9: inv

w/reject

Downloaded new block with size 190 at height 500 from Sat Dec  7 22:11:26 2013

DEBUG: Start parsing 1 transactions...
DEBUG: done (took 0 secs)
Sync completed, at height 500
Waiting for new blocks or relevant transactions...
DEBUG: received command from node 7: inv
DEBUG: Get inv request with 1 items
DEBUG: Requesting 1 blocks
DEBUG: sending message to node 7: getdata
DEBUG: received command from node 23: inv
DEBUG: received command from node 7: block
DEBUG: Got invalid block (not in sequence) from node 7
DEBUG: sending message to node 7: reject
DEBUG: received command from node 14: addr
DEBUG: received command from node 14: feefilter
DEBUG: received command from node 21: version

[xanimo]

ACK. Did a manual scrape test of this rebased off upstream/0.1.3-dev and arrived at the correct wallet balance/detected no errors or memory leaks. 🚀