Skip to content

Commit ecb058d

Browse files
committed
simplify to use slices.Reverse
Signed-off-by: Tonis Tiigi <[email protected]>
1 parent 70be02f commit ecb058d

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

cache/remotecache/inline/inline.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package inline
33
import (
44
"context"
55
"encoding/json"
6+
"slices"
67

78
"github.com/containerd/containerd/labels"
89
"github.com/moby/buildkit/cache/remotecache"
@@ -167,8 +168,6 @@ func layerToBlobs(idx int, layers []v1.CacheLayer) []digest.Digest {
167168
idx = layer.ParentIndex
168169
}
169170
// reverse so they go lowest to highest
170-
for i, j := 0, len(ds)-1; i < j; i, j = i+1, j-1 {
171-
ds[i], ds[j] = ds[j], ds[i]
172-
}
171+
slices.Reverse(ds)
173172
return ds
174173
}

frontend/dockerfile/shell/lex.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"bytes"
55
"fmt"
66
"regexp"
7+
"slices"
78
"strings"
89
"text/scanner"
910
"unicode"
@@ -646,11 +647,7 @@ func reversePattern(pattern string) string {
646647

647648
func reverseString(str string) string {
648649
out := []rune(str)
649-
outIdx := len(out) - 1
650-
for i := 0; i < outIdx; i++ {
651-
out[i], out[outIdx] = out[outIdx], out[i]
652-
outIdx--
653-
}
650+
slices.Reverse(out)
654651
return string(out)
655652
}
656653

0 commit comments

Comments
 (0)