Improve memory usage of the GridRenderer

Instead of caching the entire grid image, cache and render the Grid per tile. Some of the grid images consumed around 150MB of memory... Issue #428, Issue #404
gurkenlabs · Dec 28, 2021 · a1571c1 · a1571c1
1 parent b3c47f9
commit a1571c1
Showing 1 changed file with 18 additions and 15 deletions.
diff --git a/utiliti/src/main/java/de/gurkenlabs/utiliti/renderers/GridRenderer.java b/utiliti/src/main/java/de/gurkenlabs/utiliti/renderers/GridRenderer.java
@@ -18,6 +18,7 @@
 import java.util.concurrent.ConcurrentHashMap;
 
 public class GridRenderer implements IEditorRenderer {
+
   private final Map<String, GridImages> gridCache = new ConcurrentHashMap<>();
 
   @Override
@@ -57,12 +58,18 @@ public void render(Graphics2D g) {
               : camera.getRenderScale() > GridImages.RENDERSCALE_MID
                   ? images.getMidImage()
                   : images.getSmallImage();
-      ImageRenderer.renderScaled(
-          g,
-          image,
-          viewPortLocation.getX() * camera.getRenderScale(),
-          viewPortLocation.getY() * Game.world().camera().getRenderScale(),
-          camera.getRenderScale() / scale);
+
+      for (int x = 0; x < map.getWidth(); x++) {
+        for (int y = 0; y < map.getHeight(); y++) {
+          ImageRenderer.renderScaled(
+              g,
+              image,
+              (viewPortLocation.getX() + (x * map.getTileWidth())) * camera.getRenderScale(),
+              (viewPortLocation.getY() + (y * map.getTileHeight())) * Game.world().camera().getRenderScale(),
+              camera.getRenderScale() / scale);
+        }
+      }
+
     }
   }
 
@@ -71,6 +78,7 @@ public void clearCache() {
   }
 
   private static class GridImages {
+
     private static final float RENDERSCALE_LARGE = 6;
     private static final float RENDERSCALE_MID = 1.5f;
 
@@ -102,20 +110,15 @@ public BufferedImage getLargeImage() {
     private static BufferedImage createImage(IMap map, float scale) {
       BufferedImage image =
           Imaging.getCompatibleImage(
-              (int) (map.getSizeInPixels().width * scale) + 1,
-              (int) (map.getSizeInPixels().height * scale) + 1);
+              (int) (map.getTileWidth() * scale) + 1,
+              (int) (map.getTileHeight() * scale) + 1);
       Graphics2D graphics = (Graphics2D) image.getGraphics();
 
       final float lineSize = Editor.preferences().getGridLineWidth() / scale;
       final Stroke stroke = new BasicStroke(lineSize);
       graphics.setColor(Editor.preferences().getGridColor());
-      for (int x = 0; x < map.getWidth(); x++) {
-        for (int y = 0; y < map.getHeight(); y++) {
-          Shape tile = map.getOrientation().getShape(x, y, map);
-          ShapeRenderer.renderOutlineTransformed(
-              graphics, tile, AffineTransform.getScaleInstance(scale, scale), stroke);
-        }
-      }
+      Shape tile = map.getOrientation().getShape(0, 0, map);
+      ShapeRenderer.renderOutlineTransformed(graphics, tile, AffineTransform.getScaleInstance(scale, scale), stroke);
 
       graphics.dispose();