Merge pull request #6323 from avoidalone/master

chore: fix some typos in comments

pkg/chainsync/syncer/syncer.go

@@ -673,7 +673,7 @@ func (syncer *Syncer) Head() *types.TipSet {
 // maybe this code should actually live in blocksync??
 func zipTipSetAndMessages(bs blockstore.Blockstore, ts *types.TipSet, allbmsgs []*types.Message, allsmsgs []*types.SignedMessage, bmi, smi [][]uint64) (*types.FullTipSet, error) {
 	if len(ts.Blocks()) != len(smi) || len(ts.Blocks()) != len(bmi) {
-		return nil, fmt.Errorf("msgincl length didnt match tipset size")
+		return nil, fmt.Errorf("msgincl length didn't match tipset size")
 	}
 
 	fts := &types.FullTipSet{}
@@ -704,7 +704,7 @@ func zipTipSetAndMessages(bs blockstore.Blockstore, ts *types.TipSet, allbmsgs [
 		}
 
 		if b.Messages != mrcid {
-			return nil, fmt.Errorf("messages didnt match message root in header for ts %s", ts.Key())
+			return nil, fmt.Errorf("messages didn't match message root in header for ts %s", ts.Key())
 		}
 
 		fb := &types.FullBlock{

pkg/consensus/block_validator.go

@@ -915,7 +915,7 @@ func (bv *BlockValidator) checkBlockMessages(ctx context.Context,
 		return fmt.Errorf("serialize tx meta failed: %v", err)
 	}
 	if blk.Messages != b.Cid() {
-		return fmt.Errorf("messages didnt match message root in header")
+		return fmt.Errorf("messages didn't match message root in header")
 	}
 	return nil
 }

pkg/market/fundmanager.go

@@ -561,7 +561,7 @@ func (a *fundedAddress) processWithdrawals(withdrawals []*fundRequest) (msgCid c
 	return withdrawFundsCid, nil
 }
 
-// asynchonously wait for results of message
+// asynchronously wait for results of message
 func (a *fundedAddress) startWaitForResults(ctx context.Context, msgCid cid.Cid) {
 	go func() {
 		err := a.env.WaitMsg(a.ctx, msgCid)

pkg/net/blocksub/validator.go

@@ -27,7 +27,7 @@ type BlockHeaderValidator interface {
 	ValidateBlockMsg(context.Context, *types.BlockMsg) pubsub.ValidationResult
 }
 
-// NewBlockTopicValidator retruns a BlockTopicValidator using `bv` for message validation
+// NewBlockTopicValidator returns a BlockTopicValidator using `bv` for message validation
 func NewBlockTopicValidator(bv BlockHeaderValidator, opts ...pubsub.ValidatorOpt) *BlockTopicValidator {
 	return &BlockTopicValidator{
 		opts: opts,

pkg/net/exchange/client.go

@@ -237,7 +237,7 @@ func (c *client) processResponse(req *exchange.Request, res *exchange.Response,
 			// If we didn't request the headers they should have been provided
 			// by the caller.
 			if len(tipsets) < len(res.Chain) {
-				return nil, fmt.Errorf("not enought tipsets provided for message response validation, needed %d, have %d", len(res.Chain), len(tipsets))
+				return nil, fmt.Errorf("not enough tipsets provided for message response validation, needed %d, have %d", len(res.Chain), len(tipsets))
 			}
 			chain := make([]*exchange.BSTipSet, 0, resLength)
 			for i, resChain := range res.Chain {

pkg/repo/fskeystore/fskeystore_test.go

@@ -148,15 +148,15 @@ func TestKeystoreBasics(t *testing.T) {
 	}
 
 	if err := ks.Put("..///foo/", k1); err == nil {
-		t.Fatal("shouldnt be able to put a poorly named key")
+		t.Fatal("shouldn't be able to put a poorly named key")
 	}
 
 	if err := ks.Put("", k1); err == nil {
-		t.Fatal("shouldnt be able to put a key with no name")
+		t.Fatal("shouldn't be able to put a key with no name")
 	}
 
 	if err := ks.Put(".foo", k1); err == nil {
-		t.Fatal("shouldnt be able to put a key with a 'hidden' name")
+		t.Fatal("shouldn't be able to put a key with a 'hidden' name")
 	}
 }
 
@@ -205,7 +205,7 @@ func TestInvalidKeyFiles(t *testing.T) {
 	}
 
 	if _, err = ks.Has(".invalid"); err == nil {
-		t.Fatal("shouldnt be able to put a key with a 'hidden' name")
+		t.Fatal("shouldn't be able to put a key with a 'hidden' name")
 	}
 }
 
@@ -233,7 +233,7 @@ func TestMakeKeystoreNoDir(t *testing.T) {
 
 	_, err := NewFSKeystore("/this/is/not/a/real/dir")
 	if err == nil {
-		t.Fatal("shouldnt be able to make a keystore in a nonexistant directory")
+		t.Fatal("shouldn't be able to make a keystore in a nonexistant directory")
 	}
 }
 
@@ -264,7 +264,7 @@ func assertGetKey(ks Keystore, name string, exp []byte) error {
 	}
 
 	if !bytes.Equal(outK, exp) {
-		return fmt.Errorf("key we got out didnt match expectation")
+		return fmt.Errorf("key we got out didn't match expectation")
 	}
 
 	return nil

pkg/repo/fskeystore/memkeystore_test.go

@@ -90,14 +90,14 @@ func TestMemKeyStoreBasics(t *testing.T) {
 	}
 
 	if err := ks.Put("..///foo/", k1); err == nil {
-		t.Fatal("shouldnt be able to put a poorly named key")
+		t.Fatal("shouldn't be able to put a poorly named key")
 	}
 
 	if err := ks.Put("", k1); err == nil {
-		t.Fatal("shouldnt be able to put a key with no name")
+		t.Fatal("shouldn't be able to put a key with no name")
 	}
 
 	if err := ks.Put(".foo", k1); err == nil {
-		t.Fatal("shouldnt be able to put a key with a 'hidden' name")
+		t.Fatal("shouldn't be able to put a key with a 'hidden' name")
 	}
 }

venus-component/libp2p/exchange/client/client.go

@@ -242,7 +242,7 @@ func (c *client) processResponse(req *exchange.Request, res *exchange.Response,
 			// If we didn't request the headers they should have been provided
 			// by the caller.
 			if len(tipsets) < len(res.Chain) {
-				return nil, fmt.Errorf("not enought tipsets provided for message response validation, needed %d, have %d", len(res.Chain), len(tipsets))
+				return nil, fmt.Errorf("not enough tipsets provided for message response validation, needed %d, have %d", len(res.Chain), len(tipsets))
 			}
 			chain := make([]*exchange.BSTipSet, 0, resLength)
 			for i, resChain := range res.Chain {

venus-shared/types/tipset_marshal.go

@@ -16,7 +16,7 @@ type ExpTipSet struct {
 }
 
 func (ts *TipSet) MarshalJSON() ([]byte, error) {
-	// why didnt i just export the fields? Because the struct has methods with the
+	// why didn't i just export the fields? Because the struct has methods with the
 	// same names already
 	return json.Marshal(ExpTipSet{
 		Cids:   ts.cids,