Merge pull request #672 from eroderust/master

chore: fix some minor issues in the comments

Author: semyon-dev

escrow/escrow.go

@@ -10,7 +10,7 @@ import (
 
 // lockingPaymentChannelService implements PaymentChannelService interface
 // using locks around proxied service call to guarantee that only one payment
-// at time is applied to channel
+// at a time is applied to channel
 type lockingPaymentChannelService struct {
 	storage          *PaymentChannelStorage
 	paymentStorage   *PaymentStorage
@@ -52,8 +52,8 @@ func (h *lockingPaymentChannelService) PaymentChannel(key *PaymentChannelKey) (c
 	blockchainChannel, blockchainOk, err := h.blockchainReader.GetChannelStateFromBlockchain(key)
 
 	if !storageOk {
-		// Group ID check is only done for the first time, when the channel is added to storage from the blockchain,
-		// if the channel is already present in the storage, the group ID check is skipped.
+		// Group ID check is only done for the first time, when the channel is added to storage from the blockchain.
+		// If the channel is already present in the storage, the group ID check is skipped.
 		if blockchainChannel != nil {
 			blockChainGroupID := h.replicaGroupID()
 			if err = h.verifyGroupId(blockChainGroupID, blockchainChannel.GroupID); err != nil {

escrow/payment_channel_storage.go

@@ -107,7 +107,7 @@ func NewBlockchainChannelReader(processor blockchain.Processor, cfg *viper.Viper
 }
 
 // GetChannelStateFromBlockchain returns channel state from Ethereum
-// blockchain. ok is false if the channel was not found.
+// blockchain. ok is false if the channel is not found.
 func (reader *BlockchainChannelReader) GetChannelStateFromBlockchain(key *PaymentChannelKey) (channel *PaymentChannelData, ok bool, err error) {
 	ch, ok, err := reader.readChannelFromBlockchain(key.ID)
 	if err != nil || !ok {

escrow/state_service.go

@@ -46,7 +46,7 @@ func (service *BlockChainDisabledStateService) GetChannelState(context context.C
 	return &ChannelStateReply{}, nil
 }
 
-// verifies whether storage channel nonce is equal to blockchain nonce or not
+// StorageNonceMatchesWithBlockchainNonce verifies whether storage channel nonce is equal to blockchain nonce or not
 func (service *PaymentChannelStateService) StorageNonceMatchesWithBlockchainNonce(storageChannel *PaymentChannelData) (equal bool, err error) {
 	h := service.channelService
 
@@ -128,7 +128,7 @@ func (service *PaymentChannelStateService) GetChannelState(context context.Conte
 		return nil, err
 
 	} else if !nonceEqual {
-		// check for payments in the payment storage with current nonce - 1, this will happen  cli has issues in claiming process
+		// check for payments in the payment storage with current nonce - 1, this will happen if cli has issues in claiming process
 
 		paymentID := PaymentID(channel.ChannelID, (&big.Int{}).Sub(channel.Nonce, big.NewInt(1)))
 		payment, ok, err := service.paymentStorage.Get(paymentID)

handler/stream_interceptor.go

@@ -45,9 +45,9 @@ const (
 	// PaymentChannelSignatureHeader is a signature of the client to confirm
 	// amount withdrawing authorization. Value is an array of bytes.
 	PaymentChannelSignatureHeader = "snet-payment-channel-signature-bin"
-	// This is useful information in the header sent in by the client
+	// This is useful information in the header sent by the client
 	// All clients will have this information and they need this to Sign anyways
-	// When Daemon is running in the block chain disabled mode , it would use this
+	// When Daemon is running in the block chain disabled mode, it would use this
 	// header to get the MPE address. The goal here is to keep the client oblivious to the
 	// Daemon block chain enabled or disabled mode and also standardize the signatures.
 	// id. Value is a string containing a decimal number.