Address review

Author: Zorbatron

src/main/java/gregtech/api/recipes/RecipeBuilder.java

@@ -55,7 +55,7 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.HashMap;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 
@@ -132,6 +132,8 @@ protected RecipeBuilder(RecipeBuilder<R> recipeBuilder) {
         this.hidden = recipeBuilder.hidden;
         this.category = recipeBuilder.category;
         this.recipePropertyStorage = recipeBuilder.recipePropertyStorage.copy();
+        this.ignoreAllBuildActions = recipeBuilder.ignoreAllBuildActions;
+        this.ignoredBuildActions = new Object2ObjectOpenHashMap<>(recipeBuilder.ignoredBuildActions);
     }
 
     public R cleanroom(@Nullable CleanroomType cleanroom) {
@@ -909,9 +911,6 @@ public R ignoreBuildAction(ResourceLocation buildActionName) {
             GTLog.logger.error("Recipe map {} does not contain build action {}!", recipeMap, buildActionName,
                     new Throwable());
             return (R) this;
-        } else if (ignoredBuildActions.containsKey(buildActionName)) {
-            GTLog.logger.error("Builder already ignores {}!", buildActionName, new Throwable());
-            return (R) this;
         }
 
         ignoredBuildActions.put(buildActionName, recipeMap.getBuildActions().get(buildActionName));
@@ -1091,15 +1090,16 @@ public boolean ignoresAllBuildActions() {
 
     /**
      * Get all ignored build actions for the recipe map.
-     * Will return an empty map if none are ignored.
+     * 
+     * @return A map of ignored build actions.
      */
     public @NotNull Map<ResourceLocation, RecipeBuildAction<R>> getIgnoredBuildActions() {
         if (ignoreAllBuildActions) {
             return recipeMap.getBuildActions();
         }
 
         if (ignoredBuildActions == null) {
-            return new HashMap<>();
+            return Collections.emptyMap();
         }
 
         return ignoredBuildActions;