From b15537a32f7b3bc6bab4ddf99991d9bf2955a1d8 Mon Sep 17 00:00:00 2001
From: embeddedt <42941056+embeddedt@users.noreply.github.com>
Date: Wed, 31 May 2023 09:37:34 -0400
Subject: [PATCH] Make CanonizingStringMap not crash on null keys/values

---
 .../org/embeddedt/modernfix/util/CanonizingStringMap.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/common/src/main/java/org/embeddedt/modernfix/util/CanonizingStringMap.java b/common/src/main/java/org/embeddedt/modernfix/util/CanonizingStringMap.java
index 05bd219b..1202500b 100644
--- a/common/src/main/java/org/embeddedt/modernfix/util/CanonizingStringMap.java
+++ b/common/src/main/java/org/embeddedt/modernfix/util/CanonizingStringMap.java
@@ -141,6 +141,7 @@ public class CanonizingStringMap<T> implements Map<String, T> {
     }
 
     public static <T> CanonizingStringMap<T> deepCopy(CanonizingStringMap<T> inputMap, Function<T, T> deepCopier) {
+        Objects.requireNonNull(deepCopier);
         Object2ObjectMap<String, T> copiedBackingMap;
         int size = inputMap.backingMap.size();
         if(size > GROWTH_THRESHOLD) {
@@ -148,7 +149,8 @@ public class CanonizingStringMap<T> implements Map<String, T> {
         } else
             copiedBackingMap = new Object2ObjectArrayMap<>(size);
         inputMap.backingMap.object2ObjectEntrySet().forEach(entry -> {
-            copiedBackingMap.put(entry.getKey(), deepCopier.apply(entry.getValue()));
+            if(entry.getKey() != null && entry.getValue() != null)
+                copiedBackingMap.put(entry.getKey(), deepCopier.apply(entry.getValue()));
         });
         return new CanonizingStringMap<>(copiedBackingMap);
     }