clean up the logic and fix typos

When you get the pixels it will also have the synchronized tag

Author: broken1arrow

Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/MapRendererData.java

@@ -1,5 +1,6 @@
 package org.broken.arrow.library.itemcreator.meta.map;
 
+import org.broken.arrow.library.itemcreator.meta.map.builder.RenderConfigurator;
 import org.broken.arrow.library.itemcreator.meta.map.color.parser.AmpersandHexColorParser;
 import org.broken.arrow.library.itemcreator.meta.map.color.parser.ColorParser;
 import org.broken.arrow.library.itemcreator.meta.map.cursor.MapCursorAdapter;
@@ -23,6 +24,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.function.Consumer;
 import java.util.stream.Collectors;
 
 /**
@@ -202,7 +204,7 @@ public void addText(final int layer, @Nonnull final TextOverlay textOverlay) {
      * and inserts it into the pixel list. Only basic scaling is performed at this
      * stage. Advanced preprocessing—such as color balancing, palette matching,
      * or pixel extraction—is applied only when using {@link MapRendererDataCache}
-     * together with {@link BuildMapView#loadCachedLayer(int, MapRendererDataCache)}.
+     * together with {@link RenderConfigurator#withCachedLayer(int, int, MapRendererDataCache)} .
      * </p>
      *
      * <p>When using the cache, all images are preprocessed asynchronously and then

Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RenderConfigurator.java

@@ -41,9 +41,9 @@ public RenderConfigurator(@Nonnull final MapRendererData renderer, @Nonnull fina
      * @param config a consumer that modifies the {@link MapRendererData} instance
      * @return a step object allowing cached layer loading or builder configuration
      */
-    public RendererStepAfter withRenderer(@Nonnull final Consumer<MapRendererData> config) {
+    public RenderStepAfter withRenderer(@Nonnull final Consumer<MapRendererData> config) {
         config.accept(this.renderer);
-        return new RendererStepAfter(this.renderer, this.mapRendererBuilder);
+        return new RenderStepAfter(this.renderer, this.mapRendererBuilder);
     }
 
     /**
@@ -57,8 +57,8 @@ public RendererStepAfter withRenderer(@Nonnull final Consumer<MapRendererData> c
      * @param cache the cache providing pixel data
      * @return a step object allowing renderer or builder configuration
      */
-    public RendererStepAfterCache withCachedLayer(final int layer, final int cacheId, @Nonnull final MapRendererDataCache cache) {
+    public RenderStepAfterCache withCachedLayer(final int layer, final int cacheId, @Nonnull final MapRendererDataCache cache) {
         cache.setLayerToRender(layer,cacheId ,this.renderer);
-        return new RendererStepAfterCache(renderer, mapRendererBuilder);
+        return new RenderStepAfterCache(renderer, mapRendererBuilder);
     }
 }

…/meta/map/builder/RendererStepAfter.java …or/meta/map/builder/RenderStepAfter.javaItem Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RendererStepAfter.java renamed to Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RenderStepAfter.java Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RendererStepAfter.java renamed to Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RenderStepAfter.java

@@ -12,7 +12,7 @@
  *
  * <p>Allows optional loading of cached layers or builder configuration.</p>
  */
-public class RendererStepAfter extends FinalRenderStep {
+public class RenderStepAfter extends FinalRenderStep {
 
     private final MapRendererData renderer;
 
@@ -22,7 +22,7 @@ public class RendererStepAfter extends FinalRenderStep {
      * @param renderer the renderer instance configured in the previous step
      * @param mapRendererBuilder the builder instance used for automatic layer assignment
      */
-    public RendererStepAfter(@Nonnull final MapRendererData renderer, @Nonnull final MapRendererBuilder mapRendererBuilder) {
+    public RenderStepAfter(@Nonnull final MapRendererData renderer, @Nonnull final MapRendererBuilder mapRendererBuilder) {
         super(mapRendererBuilder);
         this.renderer = renderer;
     }

…/map/builder/RendererStepAfterCache.java …ta/map/builder/RenderStepAfterCache.javaItem Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RendererStepAfterCache.java renamed to Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RenderStepAfterCache.java Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RendererStepAfterCache.java renamed to Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/builder/RenderStepAfterCache.java

@@ -11,7 +11,7 @@
  *
  * <p>Allows optional renderer configuration or builder configuration.</p>
  */
-public class RendererStepAfterCache extends FinalRenderStep {
+public class RenderStepAfterCache extends FinalRenderStep {
     private final MapRendererData renderer;
 
     /**
@@ -20,7 +20,7 @@ public class RendererStepAfterCache extends FinalRenderStep {
      * @param renderer the renderer instance receiving cached layers
      * @param mapRendererBuilder the builder instance used for automatic layer assignment
      */
-    public RendererStepAfterCache(@Nonnull final MapRendererData renderer, @Nonnull final MapRendererBuilder mapRendererBuilder) {
+    public RenderStepAfterCache(@Nonnull final MapRendererData renderer, @Nonnull final MapRendererBuilder mapRendererBuilder) {
         super(mapRendererBuilder);
         this.renderer = renderer;
     }

Item Creator/src/main/java/org/broken/arrow/library/itemcreator/meta/map/cache/MapRendererDataCache.java

@@ -409,7 +409,7 @@ public synchronized void appendPixels(@Nonnull final List<MapPixel> extraPixels)
          *
          * @return the list of pixels
          */
-        public List<MapPixel> getPixels() {
+        public synchronized List<MapPixel> getPixels() {
             return pixels;
         }
     }