From a4819996a860066dfba5383e4fb6db2a243ad03e Mon Sep 17 00:00:00 2001
From: Alexander Zielenski <351783+alexzielenski@users.noreply.github.com>
Date: Wed, 20 Jul 2022 10:18:42 -0700
Subject: [PATCH] optimize nil and empty also for slices

brings to parity with maps
---
 .../forked/golang/reflect/deep_equal.go          | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/staging/src/k8s.io/apimachinery/third_party/forked/golang/reflect/deep_equal.go b/staging/src/k8s.io/apimachinery/third_party/forked/golang/reflect/deep_equal.go
index cc5d236bf84..511e625b63d 100644
--- a/staging/src/k8s.io/apimachinery/third_party/forked/golang/reflect/deep_equal.go
+++ b/staging/src/k8s.io/apimachinery/third_party/forked/golang/reflect/deep_equal.go
@@ -169,6 +169,18 @@ func (e Equalities) deepValueEqual(v1, v2 reflect.Value, visited map[visit]bool,
 			if v1.IsNil() != v2.IsNil() {
 				return false
 			}
+
+			// Optimize nil and empty cases
+			// Two lists that are BOTH nil are equal
+			// No need to check v2 is nil since v1.IsNil == v2.IsNil from above
+			if v1.IsNil() {
+				return true
+			}
+
+			// Two lists that are both empty and both non nil are equal
+			if v1.Len() == 0 || v2.Len() == 0 {
+				return true
+			}
 		}
 		if v1.Len() != v2.Len() {
 			return false
@@ -210,13 +222,13 @@ func (e Equalities) deepValueEqual(v1, v2 reflect.Value, visited map[visit]bool,
 			}
 
 			// Optimize nil and empty cases
-			// Two lists that are BOTH nil are equal
+			// Two maps that are BOTH nil are equal
 			// No need to check v2 is nil since v1.IsNil == v2.IsNil from above
 			if v1.IsNil() {
 				return true
 			}
 
-			// Two lists that are both empty and both non nil are equal
+			// Two maps that are both empty and both non nil are equal
 			if v1.Len() == 0 || v2.Len() == 0 {
 				return true
 			}