Skip to content

K
k3s
Commit 3aa4f50a authored by Justin Santa Barbara's avatar Justin Santa Barbara
Browse files
Options

Avoid closing over range variables

The consequence is only a misleading error message, but it is easy to avoid.
parent c347f579
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 2 deletions
pkg/api/v1/conversion.go
View file @ 3aa4f50a
......@@ -164,7 +164,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error {
}
// Add field label conversions for kinds having selectable nothing but ObjectMeta fields.
for _, kind := range []string{
for _, k := range []string{
"Endpoints",
"ResourceQuota",
"PersistentVolumeClaim",
......@@ -172,6 +172,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error {
"ServiceAccount",
"ConfigMap",
} {
kind := k // don't close over range variables
err = scheme.AddFieldLabelConversionFunc("v1", kind,
func(label, value string) (string, string, error) {
switch label {
......
pkg/apis/extensions/v1beta1/conversion.go
View file @ 3aa4f50a
......@@ -57,7 +57,8 @@ func addConversionFuncs(scheme *runtime.Scheme) error {
}
// Add field label conversions for kinds having selectable nothing but ObjectMeta fields.
for _, kind := range []string{"DaemonSet", "Deployment", "Ingress"} {
for _, k := range []string{"DaemonSet", "Deployment", "Ingress"} {
kind := k // don't close over range variables
err = api.Scheme.AddFieldLabelConversionFunc("extensions/v1beta1", kind,
func(label, value string) (string, string, error) {
switch label {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment