Merge pull request #2045 from GrapeBaBa/comment_blockchain

Fix typo when reading code

core/ledger/blockchain.go

@@ -165,7 +165,7 @@ func (blockchain *blockchain) getTransactionsByBlockHash(blockHash []byte) ([]*p
 	return block.GetTransactions(), nil
 }
 
-// getTransaction get a transaction identified by blocknumber and index within the block
+// getTransaction get a transaction identified by block number and index within the block
 func (blockchain *blockchain) getTransaction(blockNumber uint64, txIndex uint64) (*protos.Transaction, error) {
 	block, err := blockchain.getBlock(blockNumber)
 	if err != nil {
@@ -174,7 +174,7 @@ func (blockchain *blockchain) getTransaction(blockNumber uint64, txIndex uint64)
 	return block.GetTransactions()[txIndex], nil
 }
 
-// getTransactionByBlockHash get a transaction identified by blockhash and index within the block
+// getTransactionByBlockHash get a transaction identified by block hash and index within the block
 func (blockchain *blockchain) getTransactionByBlockHash(blockHash []byte, txIndex uint64) (*protos.Transaction, error) {
 	block, err := blockchain.getBlockByHash(blockHash)
 	if err != nil {
@@ -265,8 +265,8 @@ func (blockchain *blockchain) persistRawBlock(block *protos.Block, blockNumber u
 		return err
 	}
 
-	// Need to check as we suport out of order blocks in cases such as block/state synchronization. This is
-	// really blockchain height, not size.
+	// Need to check as we support out of order blocks in cases such as block/state synchronization. This is
+	// real blockchain height, not size.
 	if blockchain.getSize() < blockNumber+1 {
 		sizeBytes := encodeUint64(blockNumber + 1)
 		writeBatch.PutCF(db.GetDBHandle().BlockchainCF, blockCountKey, sizeBytes)

core/ledger/blockchain_indexes_async.go

@@ -210,7 +210,7 @@ func (indexer *blockchainIndexerAsync) stop() {
 // Since, we index blocks asynchronously, there may be a case when
 // a client query arrives before a block has been indexed.
 //
-// Do we really need strict symantics such that an index query results
+// Do we really need strict semantics such that an index query results
 // should include up to block number (or higher) that may have been committed
 // when user query arrives?
 // If a delay of a couple of blocks are allowed, we can get rid of this synchronization stuff

core/ledger/blockchain_indexes_async_test.go

@@ -81,7 +81,7 @@ func TestIndexesAsync_IndexingErrorScenario(t *testing.T) {
 
 	defer func() {
 		// first stop and then set the error to nil.
-		// Otherwise stop may hang (waiting for cathing up the index with the committing block)
+		// Otherwise stop may hang (waiting for catching up the index with the committing block)
 		testBlockchainWrapper.blockchain.indexer.stop()
 		asyncIndexer.indexerState.setError(nil)
 	}()

core/ledger/ledger_test.go

@@ -677,10 +677,10 @@ func TestPreviewTXBatchBlock(t *testing.T) {
 	testutil.AssertNoError(t, err, "Error fetching preview block info.")
 
 	ledger.CommitTxBatch(0, []*protos.Transaction{transaction}, nil, []byte("proof"))
-	commitedBlockInfo, err := ledger.GetBlockchainInfo()
+	committedBlockInfo, err := ledger.GetBlockchainInfo()
 	testutil.AssertNoError(t, err, "Error fetching committed block hash.")
 
-	testutil.AssertEquals(t, previewBlockInfo, commitedBlockInfo)
+	testutil.AssertEquals(t, previewBlockInfo, committedBlockInfo)
 }
 
 func TestGetTransactionByUUID(t *testing.T) {

core/ledger/perf_test.go

@@ -99,7 +99,7 @@ func BenchmarkLedgerSingleKeyTransaction(b *testing.B) {
 	}
 	b.StopTimer()
 
-	//varify value persisted
+	//verify value persisted
 	value, _ = ledger.GetState(chaincode, *key, true)
 	size := ledger.GetBlockchainSize()
 	b.Logf("Value size=%d, Blockchain height=%d", len(value), size)

core/ledger/pkg_test.go

@@ -127,7 +127,7 @@ func (testWrapper *blockchainTestWrapper) populateBlockChainWithSampleData() (bl
 	// -----------------------------</Block 2>------------------------------------
 
 	// -----------------------------<Block 3>-------------------------------------
-	// Create a transaction'
+	// Create a transaction
 	transaction3a, err := protos.NewTransaction(protos.ChaincodeID{Path: "MyContract"}, testutil.GenerateUUID(testWrapper.t), "setX", []string{"{x: \"hello\"}"})
 	if err != nil {
 		return nil, nil, err