Skip to content

Commit f35ac19

Browse files
ivanmatmatioktalz
authored andcommitted
MINOR: use copy instead of loops in CRD converters
1 parent 7cebb7a commit f35ac19

File tree

2 files changed

+6
-18
lines changed

2 files changed

+6
-18
lines changed

crs/converters/defaults-spec-a1-a2.go

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -77,16 +77,12 @@ func DeepConvertDefaultsSpecA1toA2(o corev1alpha1.DefaultsSpec) corev1alpha2.Def
7777
cp.Config.Compression = new(models.Compression)
7878
if o.Config.Compression.Algorithms != nil {
7979
cp.Config.Compression.Algorithms = make([]string, len(o.Config.Compression.Algorithms))
80-
for i6 := range o.Config.Compression.Algorithms { //nolint:gosimple
81-
cp.Config.Compression.Algorithms[i6] = o.Config.Compression.Algorithms[i6]
82-
}
80+
copy(cp.Config.Compression.Algorithms, o.Config.Compression.Algorithms)
8381
}
8482
cp.Config.Compression.Offload = o.Config.Compression.Offload
8583
if o.Config.Compression.Types != nil {
8684
cp.Config.Compression.Types = make([]string, len(o.Config.Compression.Types))
87-
for i6 := range o.Config.Compression.Types { //nolint:gosimple
88-
cp.Config.Compression.Types[i6] = o.Config.Compression.Types[i6]
89-
}
85+
copy(cp.Config.Compression.Types, o.Config.Compression.Types)
9086
}
9187
}
9288
if o.Config.ConnectTimeout != nil {
@@ -228,9 +224,7 @@ func DeepConvertDefaultsSpecA1toA2(o corev1alpha1.DefaultsSpec) corev1alpha2.Def
228224
cp.Config.DefaultServer.Proto = o.Config.DefaultServer.Proto
229225
if o.Config.DefaultServer.ProxyV2Options != nil {
230226
cp.Config.DefaultServer.ProxyV2Options = make([]string, len(o.Config.DefaultServer.ProxyV2Options))
231-
for i6 := range o.Config.DefaultServer.ProxyV2Options { //nolint:gosimple
232-
cp.Config.DefaultServer.ProxyV2Options[i6] = o.Config.DefaultServer.ProxyV2Options[i6]
233-
}
227+
copy(cp.Config.DefaultServer.ProxyV2Options, o.Config.DefaultServer.ProxyV2Options)
234228
}
235229
cp.Config.DefaultServer.Redir = o.Config.DefaultServer.Redir
236230
cp.Config.DefaultServer.ResolveNet = o.Config.DefaultServer.ResolveNet

crs/converters/defaults-spec.go

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -78,16 +78,12 @@ func DeepConvertDefaultsSpecA2toV1(o corev1alpha2.DefaultsSpec) v1.DefaultsSpec
7878
cp.Config.Compression = new(models.Compression)
7979
if o.Config.Compression.Algorithms != nil {
8080
cp.Config.Compression.Algorithms = make([]string, len(o.Config.Compression.Algorithms))
81-
for i6 := range o.Config.Compression.Algorithms { //nolint:gosimple
82-
cp.Config.Compression.Algorithms[i6] = o.Config.Compression.Algorithms[i6]
83-
}
81+
copy(cp.Config.Compression.Algorithms, o.Config.Compression.Algorithms)
8482
}
8583
cp.Config.Compression.Offload = o.Config.Compression.Offload
8684
if o.Config.Compression.Types != nil {
8785
cp.Config.Compression.Types = make([]string, len(o.Config.Compression.Types))
88-
for i6 := range o.Config.Compression.Types { //nolint:gosimple
89-
cp.Config.Compression.Types[i6] = o.Config.Compression.Types[i6]
90-
}
86+
copy(cp.Config.Compression.Types, o.Config.Compression.Types)
9187
}
9288
}
9389
if o.Config.ConnectTimeout != nil {
@@ -234,9 +230,7 @@ func DeepConvertDefaultsSpecA2toV1(o corev1alpha2.DefaultsSpec) v1.DefaultsSpec
234230
cp.Config.DefaultServer.Proto = o.Config.DefaultServer.Proto
235231
if o.Config.DefaultServer.ProxyV2Options != nil {
236232
cp.Config.DefaultServer.ProxyV2Options = make([]string, len(o.Config.DefaultServer.ProxyV2Options))
237-
for i6 := range o.Config.DefaultServer.ProxyV2Options { //nolint:gosimple
238-
cp.Config.DefaultServer.ProxyV2Options[i6] = o.Config.DefaultServer.ProxyV2Options[i6]
239-
}
233+
copy(cp.Config.DefaultServer.ProxyV2Options, o.Config.DefaultServer.ProxyV2Options)
240234
}
241235
cp.Config.DefaultServer.Redir = o.Config.DefaultServer.Redir
242236
cp.Config.DefaultServer.ResolveNet = o.Config.DefaultServer.ResolveNet

0 commit comments

Comments
 (0)