Compare fmt.Sprint output directly

Using `reflect.deepEqual()` is unnecessary Signed-off-by: Dale Haiducek <19750917+dhaiducek@users.noreply.github.com>
open-cluster-management-io · Oct 11, 2023 · 59d32f2 · 59d32f2
1 parent c4993c8
commit 59d32f2
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/controllers/configurationpolicy_utils.go b/controllers/configurationpolicy_utils.go
@@ -141,12 +141,8 @@ func equalObjWithSort(mergedObj interface{}, oldObj interface{}) (areEqual bool)
 			return fmt.Sprint(zero) == fmt.Sprint(oldObj)
 		}
 
-		if !reflect.DeepEqual(fmt.Sprint(mergedObj), fmt.Sprint(oldObj)) {
-			return false
-		}
+		return fmt.Sprint(mergedObj) == fmt.Sprint(oldObj)
 	}
-
-	return true
 }
 
 // checFieldsWithSort is a check for maps that uses an arbitrary sort to ensure it is