Use `range` over int after 1.22 upgrade #1317

New issue

Closed

opened 2024-08-19 07:51:05 +00:00 by fyrchik · 1 comment

fyrchik commented

2024-08-19 07:51:05 +00:00

Owner

There are many range b.N cases, maybe something else.
https://go.dev/ref/spec#For_range

I would like to do this with gopatch, attach patch here.

There are many `range b.N` cases, maybe something else. https://go.dev/ref/spec#For_range I would like to do this with gopatch, attach patch here.

fyrchik added the

label 2024-08-19 07:51:05 +00:00

fyrchik added the

internal

good first issue

labels 2024-08-19 07:52:34 +00:00

elebedeva was assigned by fyrchik

2024-08-19 07:52:40 +00:00

fyrchik added this to the v0.43.0 milestone 2024-08-19 08:17:03 +00:00

elebedeva referenced this issue from a pull request that will close it,

2024-08-30 16:39:59 +00:00

Use range over int after 1.22 upgrade #1345

elebedeva commented

2024-08-30 16:45:27 +00:00

Member

gopatch script:

@@
var i, e expression
@@
-for i := 0; i <= e - 1; i++ {
+for i := range e {
    ...
}

@@
var i, e expression
@@
-for i := 0; i <= e; i++ {
+for i := range e + 1 {
    ...
}

@@
var i, e expression
@@
-for i := 0; i < e; i++ {
+for i := range e {
    ...
}

`gopatch` script: ``` @@ var i, e expression @@ -for i := 0; i <= e - 1; i++ { +for i := range e { ... } @@ var i, e expression @@ -for i := 0; i <= e; i++ { +for i := range e + 1 { ... } @@ var i, e expression @@ -for i := 0; i < e; i++ { +for i := range e { ... } ```

fyrchik referenced this issue from a commit

2024-09-03 11:26:44 +00:00

[#1317] go.mod: Use range over int

fyrchik closed this issue

2024-09-03 11:26:47 +00:00

elebedeva referenced this issue from TrueCloudLab/frostfs-api-go

2024-09-04 13:55:05 +00:00

Use range over int after 1.22 upgrade #113

elebedeva referenced this issue from TrueCloudLab/frostfs-sdk-go

2024-09-04 13:59:51 +00:00

Use range over int after 1.22 upgrade #265

fyrchik referenced this issue