docs: update /block_results RPC docs (#3708)

Fixes #3616

CHANGELOG_PENDING.md

@@ -5,6 +5,8 @@
 ### BREAKING CHANGES:
 
 * CLI/RPC/Config
+- [rpc] \#3616 Improve `/block_results` response format (`results.DeliverTx` ->
+  `results.deliver_tx`). See docs for details.
 
 * Apps
 

rpc/core/blocks.go

@@ -339,7 +339,8 @@ func Commit(ctx *rpctypes.Context, heightPtr *int64) (*ctypes.ResultCommit, erro
 // If no height is provided, it will fetch results for the latest block.
 //
 // Results are for the height of the block containing the txs.
-// Thus response.results[5] is the results of executing getBlock(h).Txs[5]
+// Thus response.results.deliver_tx[5] is the results of executing
+// getBlock(h).Txs[5]
 //
 // ```shell
 // curl 'localhost:26657/block_results?height=10'
@@ -360,18 +361,28 @@ func Commit(ctx *rpctypes.Context, heightPtr *int64) (*ctypes.ResultCommit, erro
 //
 // ```json
 // {
-//  "height": "10",
-//  "results": [
+//   "jsonrpc": "2.0",
+//   "id": "",
+//   "result": {
+//     "height": "39",
+//     "results": {
+//       "deliver_tx": [
 //         {
-//    "code": "0",
-//    "data": "CAFE00F00D"
-//   },
+//           "tags": [
 //             {
-//    "code": "102",
-//    "data": ""
+//               "key": "YXBwLmNyZWF0b3I=",
+//               "value": "Q29zbW9zaGkgTmV0b3dva28="
 //             }
 //           ]
 //         }
+//       ],
+//       "end_block": {
+//         "validator_updates": null
+//       },
+//       "begin_block": {}
+//     }
+//   }
+// }
 // ```
 func BlockResults(ctx *rpctypes.Context, heightPtr *int64) (*ctypes.ResultBlockResults, error) {
 	storeHeight := blockStore.Height()
@@ -380,7 +391,6 @@ func BlockResults(ctx *rpctypes.Context, heightPtr *int64) (*ctypes.ResultBlockR
 		return nil, err
 	}
 
-	// load the results
 	results, err := sm.LoadABCIResponses(stateDB, height)
 	if err != nil {
 		return nil, err

state/store.go

@@ -115,9 +115,9 @@ func saveState(db dbm.DB, state State, key []byte) {
 // of the various ABCI calls during block processing.
 // It is persisted to disk for each height before calling Commit.
 type ABCIResponses struct {
-	DeliverTx  []*abci.ResponseDeliverTx
-	EndBlock   *abci.ResponseEndBlock
-	BeginBlock *abci.ResponseBeginBlock
+	DeliverTx  []*abci.ResponseDeliverTx `json:"deliver_tx"`
+	EndBlock   *abci.ResponseEndBlock    `json:"end_block"`
+	BeginBlock *abci.ResponseBeginBlock  `json:"begin_block"`
 }
 
 // NewABCIResponses returns a new ABCIResponses