change the for loop to a new for range

Signed-off-by: Fedor Partanskiy <[email protected]>

Author: pfi79

cmd/cryptogen/main.go

@@ -344,9 +344,9 @@ func extendPeerOrg(orgSpec OrgSpec) {
 	publicKeyAlg := getPublicKeyAlg(orgSpec.Users.PublicKeyAlgorithm)
 	// TODO: add ability to specify usernames
 	users := []NodeSpec{}
-	for j := 1; j <= orgSpec.Users.Count; j++ {
+	for j := range orgSpec.Users.Count {
 		user := NodeSpec{
-			CommonName:         fmt.Sprintf("%s%d@%s", userBaseName, j, orgName),
+			CommonName:         fmt.Sprintf("%s%d@%s", userBaseName, j+1, orgName),
 			PublicKeyAlgorithm: publicKeyAlg,
 		}
 
@@ -485,7 +485,7 @@ func renderNodeSpec(domain string, spec *NodeSpec) error {
 func renderOrgSpec(orgSpec *OrgSpec, prefix string) error {
 	publickKeyAlg := getPublicKeyAlg(orgSpec.Template.PublicKeyAlgorithm)
 	// First process all of our templated nodes
-	for i := 0; i < orgSpec.Template.Count; i++ {
+	for i := range orgSpec.Template.Count {
 		data := HostnameData{
 			Prefix: prefix,
 			Index:  i + orgSpec.Template.Start,
@@ -563,9 +563,9 @@ func generatePeerOrg(baseDir string, orgSpec OrgSpec) {
 	publicKeyAlg := getPublicKeyAlg(orgSpec.Users.PublicKeyAlgorithm)
 	// TODO: add ability to specify usernames
 	users := []NodeSpec{}
-	for j := 1; j <= orgSpec.Users.Count; j++ {
+	for j := range orgSpec.Users.Count {
 		user := NodeSpec{
-			CommonName:         fmt.Sprintf("%s%d@%s", userBaseName, j, orgName),
+			CommonName:         fmt.Sprintf("%s%d@%s", userBaseName, j+1, orgName),
 			PublicKeyAlgorithm: publicKeyAlg,
 		}
 
@@ -638,7 +638,7 @@ func generateNodes(baseDir string, nodes []NodeSpec, signCA *ca.CA, tlsCA *ca.CA
 			if node.isAdmin && nodeOUs {
 				currentNodeType = msp.ADMIN
 			}
-			err := msp.GenerateLocalMSP(nodeDir, node.CommonName, node.SANS, signCA, tlsCA, currentNodeType, nodeOUs, node.PublicKeyAlgorithm)
+			err = msp.GenerateLocalMSP(nodeDir, node.CommonName, node.SANS, signCA, tlsCA, currentNodeType, nodeOUs, node.PublicKeyAlgorithm)
 			if err != nil {
 				fmt.Printf("Error generating local MSP for %v:\n%v\n", node, err)
 				os.Exit(1)

common/channelconfig/standardvalues.go

@@ -80,7 +80,7 @@ func (sv *StandardValues) initializeProtosStruct(objValue reflect.Value) error {
 	}
 
 	numFields := objValue.Elem().NumField()
-	for i := 0; i < numFields; i++ {
+	for i := range numFields {
 		structField := objType.Elem().Field(i)
 		logger.Debugf("Processing field: %s\n", structField.Name)
 		switch structField.Type.Kind() {

common/channelconfig/util_test.go

@@ -351,7 +351,7 @@ func TestMarshalEtcdRaftMetadata(t *testing.T) {
 		inputCerts[i], _ = os.ReadFile(fmt.Sprintf("testdata/tls-client-%d.pem", i+1))
 	}
 
-	for i := 0; i < len(inputCerts)-1; i++ {
+	for i := range len(inputCerts) - 1 {
 		require.NotEqual(t, outputCerts[i+1], outputCerts[i], "expected extracted certs to differ from each other")
 	}
 }

common/deliver/acl_test.go

@@ -70,7 +70,7 @@ var _ = Describe("SessionAccessControl", func() {
 		sac, err := deliver.NewSessionAC(fakeChain, envelope, fakePolicyChecker, "chain-id", expiresAt)
 		Expect(err).NotTo(HaveOccurred())
 
-		for i := 0; i < 5; i++ {
+		for range 5 {
 			err = sac.Evaluate()
 			Expect(err).NotTo(HaveOccurred())
 		}

common/deliver/deliver_test.go

@@ -369,7 +369,7 @@ var _ = Describe("Deliver", func() {
 				Expect(err).NotTo(HaveOccurred())
 
 				Expect(fakeResponseSender.SendBlockResponseCallCount()).To(Equal(5))
-				for i := 0; i < 5; i++ {
+				for i := range 5 {
 					b, _, _, _ := fakeResponseSender.SendBlockResponseArgsForCall(i)
 					Expect(b).To(Equal(&cb.Block{
 						Header: &cb.BlockHeader{Number: 995 + uint64(i)},
@@ -393,7 +393,7 @@ var _ = Describe("Deliver", func() {
 
 				Expect(fakeBlocksSent.AddCallCount()).To(Equal(5))
 				Expect(fakeBlocksSent.WithCallCount()).To(Equal(5))
-				for i := 0; i < 5; i++ {
+				for i := range 5 {
 					Expect(fakeBlocksSent.AddArgsForCall(i)).To(BeNumerically("~", 1.0))
 					labelValues := fakeBlocksSent.WithArgsForCall(i)
 					Expect(labelValues).To(Equal([]string{
@@ -461,7 +461,7 @@ var _ = Describe("Deliver", func() {
 
 				Expect(fakeBlockIterator.NextCallCount()).To(Equal(2))
 				Expect(fakeResponseSender.SendBlockResponseCallCount()).To(Equal(2))
-				for i := 0; i < fakeResponseSender.SendBlockResponseCallCount(); i++ {
+				for i := range fakeResponseSender.SendBlockResponseCallCount() {
 					b, _, _, _ := fakeResponseSender.SendBlockResponseArgsForCall(i)
 					Expect(b).To(ProtoEqual(&cb.Block{
 						Header: &cb.BlockHeader{Number: uint64(i + 1)},
@@ -492,7 +492,7 @@ var _ = Describe("Deliver", func() {
 				Expect(fakeBlockReader.IteratorCallCount()).To(Equal(1))
 				Expect(fakeBlockIterator.NextCallCount()).To(Equal(1))
 				Expect(fakeResponseSender.SendBlockResponseCallCount()).To(Equal(1))
-				for i := 0; i < fakeResponseSender.SendBlockResponseCallCount(); i++ {
+				for i := range fakeResponseSender.SendBlockResponseCallCount() {
 					b, _, _, _ := fakeResponseSender.SendBlockResponseArgsForCall(i)
 					Expect(b).To(ProtoEqual(&cb.Block{
 						Header: &cb.BlockHeader{Number: uint64(i)},
@@ -532,7 +532,7 @@ var _ = Describe("Deliver", func() {
 
 				Expect(fakeBlockIterator.NextCallCount()).To(Equal(2))
 				Expect(fakeResponseSender.SendBlockResponseCallCount()).To(Equal(2))
-				for i := 0; i < fakeResponseSender.SendBlockResponseCallCount(); i++ {
+				for i := range fakeResponseSender.SendBlockResponseCallCount() {
 					b, _, _, _ := fakeResponseSender.SendBlockResponseArgsForCall(i)
 					Expect(b).To(ProtoEqual(&cb.Block{
 						Header:   &cb.BlockHeader{Number: uint64(i + 1)},
@@ -587,7 +587,7 @@ var _ = Describe("Deliver", func() {
 				Expect(start).To(ProtoEqual(&ab.SeekPosition{}))
 				Expect(fakeBlockIterator.NextCallCount()).To(Equal(3))
 				Expect(fakeResponseSender.SendBlockResponseCallCount()).To(Equal(3))
-				for i := 0; i < fakeResponseSender.SendBlockResponseCallCount(); i++ {
+				for i := range fakeResponseSender.SendBlockResponseCallCount() {
 					b, _, _, _ := fakeResponseSender.SendBlockResponseArgsForCall(i)
 					if i+1 == 1 || i+1 == 3 {
 						Expect(b).To(ProtoEqual(&cb.Block{

common/graph/perm_test.go

@@ -66,7 +66,7 @@ func TestPermute(t *testing.T) {
 func TestPermuteTooManyCombinations(t *testing.T) {
 	root := NewTreeVertex("r", nil)
 	root.Threshold = 500
-	for i := 0; i < 1000; i++ {
+	for i := range 1000 {
 		root.AddDescendant(NewTreeVertex(fmt.Sprintf("%d", i), nil))
 	}
 	permutations := root.ToTree().Permute(501)

common/grpcmetrics/interceptor_test.go

@@ -250,7 +250,7 @@ var _ = Describe("Interceptor", func() {
 			}))
 
 			Expect(fakeMessagesSent.AddCallCount()).To(Equal(2))
-			for i := 0; i < fakeMessagesSent.AddCallCount(); i++ {
+			for range fakeMessagesSent.AddCallCount() {
 				Expect(fakeMessagesSent.AddArgsForCall(0)).To(BeNumerically("~", 1.0))
 			}
 		})
@@ -268,7 +268,7 @@ var _ = Describe("Interceptor", func() {
 			}))
 
 			Expect(fakeMessagesReceived.AddCallCount()).To(Equal(2))
-			for i := 0; i < fakeMessagesReceived.AddCallCount(); i++ {
+			for range fakeMessagesReceived.AddCallCount() {
 				Expect(fakeMessagesReceived.AddArgsForCall(0)).To(BeNumerically("~", 1.0))
 			}
 		})

common/ledger/blkstorage/block_serialization.go

@@ -139,7 +139,7 @@ func extractData(buf *buffer) (*common.BlockData, []*txindexInfo, error) {
 	if numItems, err = buf.DecodeVarint(); err != nil {
 		return nil, nil, errors.Wrap(err, "error decoding the length of block data")
 	}
-	for i := uint64(0); i < numItems; i++ {
+	for range numItems {
 		var txEnvBytes []byte
 		var txid string
 		txOffset := buf.GetBytesConsumed()
@@ -165,7 +165,7 @@ func extractMetadata(buf *buffer) (*common.BlockMetadata, error) {
 	if numItems, err = buf.DecodeVarint(); err != nil {
 		return nil, errors.Wrap(err, "error decoding the length of block metadata")
 	}
-	for i := uint64(0); i < numItems; i++ {
+	for range numItems {
 		if metadataEntry, err = buf.DecodeRawBytes(false); err != nil {
 			return nil, errors.Wrap(err, "error decoding the block metadata")
 		}

common/ledger/blkstorage/block_stream_test.go

@@ -75,7 +75,7 @@ func testBlockFileStreamUnexpectedEOF(t *testing.T, numBlocks int, partialBlockB
 	defer s.close()
 	require.NoError(t, err, "Error in constructing blockfile stream")
 
-	for i := 0; i < numBlocks; i++ {
+	for range numBlocks {
 		blockBytes, err := s.nextBlockBytes()
 		require.NotNil(t, blockBytes)
 		require.NoError(t, err, "Error in getting next block")
@@ -102,7 +102,7 @@ func testBlockStream(t *testing.T, numFiles int) {
 	numBlocksInEachFile := 10
 	bg, gb := testutil.NewBlockGenerator(t, ledgerID, false)
 	w.addBlocks([]*common.Block{gb})
-	for i := 0; i < numFiles; i++ {
+	for i := range numFiles {
 		numBlocks := numBlocksInEachFile
 		if i == 0 {
 			// genesis block already added so adding one less block

common/ledger/blkstorage/blockfile_helper_test.go

@@ -104,10 +104,10 @@ func TestBinarySearchBlockFileNum(t *testing.T) {
 	require.NoError(t, err)
 	require.Len(t, files, 11)
 
-	for i := uint64(0); i < 100; i++ {
-		fileNum, err := binarySearchFileNumForBlock(ledgerDir, i)
+	for i := range 100 {
+		fileNum, err := binarySearchFileNumForBlock(ledgerDir, uint64(i))
 		require.NoError(t, err)
-		locFromIndex, err := blkfileMgr.index.getBlockLocByBlockNum(i)
+		locFromIndex, err := blkfileMgr.index.getBlockLocByBlockNum(uint64(i))
 		require.NoError(t, err)
 		expectedFileNum := locFromIndex.fileSuffixNum
 		require.Equal(t, expectedFileNum, fileNum)
@@ -145,7 +145,7 @@ func TestGetLedgersBootstrappedFromSnapshot(t *testing.T) {
 		testDir := t.TempDir()
 
 		// create chains directories for ledgers without bootstrappingSnapshotInfoFile
-		for i := 0; i < 5; i++ {
+		for i := range 5 {
 			require.NoError(t, os.MkdirAll(filepath.Join(testDir, ChainsDir, fmt.Sprintf("ledger_%d", i)), 0o755))
 		}
 
@@ -159,7 +159,7 @@ func TestGetLedgersBootstrappedFromSnapshot(t *testing.T) {
 
 		// create chains directories for ledgers
 		// also create bootstrappingSnapshotInfoFile for ledger_0 and ledger_1
-		for i := 0; i < 5; i++ {
+		for i := range 5 {
 			ledgerChainDir := filepath.Join(testDir, ChainsDir, fmt.Sprintf("ledger_%d", i))
 			require.NoError(t, os.MkdirAll(ledgerChainDir, 0o755))
 			if i < 2 {