fix: range over int instead of 3-clause condition (#1013)

Signed-off-by: Chris Gianelloni <[email protected]>

Author: wolf31o2

cbor/decode.go

@@ -139,7 +139,7 @@ func DecodeGeneric(cborData []byte, dest any) error {
 			return errors.New("destination must be a pointer to a struct")
 		}
 		destTypeFields := []reflect.StructField{}
-		for i := 0; i < typeDest.NumField(); i++ {
+		for i := range typeDest.NumField() {
 			tmpField := typeDest.Field(i)
 			if tmpField.IsExported() && tmpField.Name != "DecodeStoreCbor" {
 				destTypeFields = append(destTypeFields, tmpField)

cbor/encode.go

@@ -63,7 +63,7 @@ func EncodeGeneric(src any) ([]byte, error) {
 			return nil, errors.New("source must be a pointer to a struct")
 		}
 		srcTypeFields := []reflect.StructField{}
-		for i := 0; i < typeSrc.NumField(); i++ {
+		for i := range typeSrc.NumField() {
 			tmpField := typeSrc.Field(i)
 			if tmpField.IsExported() && tmpField.Name != "DecodeStoreCbor" {
 				srcTypeFields = append(srcTypeFields, tmpField)

cmd/gouroboros/mem_usage.go

@@ -79,7 +79,7 @@ func testMemUsage(f *globalFlags) {
 		)
 	}()
 
-	for i := 0; i < 10; i++ {
+	for range 10 {
 		showMemoryStats("open")
 
 		conn := createClientConnection(f)

ledger/babbage/babbage_test.go

@@ -38,7 +38,7 @@ func TestBabbageBlockTransactions(t *testing.T) {
 	b.TransactionBodies = make([]BabbageTransactionBody, txsCount)
 	b.TransactionWitnessSets = make([]BabbageTransactionWitnessSet, txsCount)
 
-	for i := 0; i < txsCount; i++ {
+	for i := range txsCount {
 		b.TransactionBodies[i] = BabbageTransactionBody{
 			TxTotalCollateral: 1 << i,
 		}

protocol/chainsync/client.go

@@ -453,7 +453,7 @@ func (c *Client) syncLoop() {
 		}
 		// Request the next block(s)
 		msgCount := max(c.config.PipelineLimit, 1)
-		for i := 0; i < msgCount; i++ {
+		for range msgCount {
 			msg := NewMsgRequestNext()
 			if err := c.SendMessage(msg); err != nil {
 				c.SendError(err)