Make CanonizingStringMap not crash on null keys/values

Author: embeddedt

common/src/main/java/org/embeddedt/modernfix/util/CanonizingStringMap.java

@@ -141,14 +141,16 @@ public int hashCode() {
     }
 
     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) {
             copiedBackingMap = new Object2ObjectOpenHashMap<>(size);
         } 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);
     }