Improve immediate presentation of meshes generated in blocking mode.

Previously even when meshes were generated in the blocking defer mode "Immediate" (zero frame) they would not become visible if they were part of a previously invisible section and thus not in a render list already. This change makes it so that sections that are added to a blocking-type task queue are also forcefully added to the render list since they are very likely to contain geometry by the end of the frame.

Author: douira

common/src/main/java/net/caffeinemc/mods/sodium/client/render/chunk/lists/ChunkRenderList.java

@@ -126,7 +126,7 @@ private void sortSections(int relativeCameraSectionX, int relativeCameraSectionY
         }
     }
 
-    public void add(RenderSection render) {
+    public void add(RenderSection render, boolean forceRenderGeometry) {
         if (this.size >= RenderRegion.REGION_SIZE) {
             throw new ArrayIndexOutOfBoundsException("Render list is full");
         }
@@ -136,7 +136,7 @@ public void add(RenderSection render) {
         int index = render.getSectionIndex();
         int flags = render.getFlags();
 
-        if (((flags >>> RenderSectionFlags.HAS_BLOCK_GEOMETRY) & 1) == 1) {
+        if (((flags >>> RenderSectionFlags.HAS_BLOCK_GEOMETRY) & 1) == 1 || forceRenderGeometry) {
             this.sectionsWithGeometryMap[index >> 6] |= 1L << (index & 0b111111);
             if (this.addedSectionsAreSorted) {
                 this.sectionsWithGeometry[this.sectionsWithGeometryCount] = (byte) index;

common/src/main/java/net/caffeinemc/mods/sodium/client/render/chunk/lists/SectionCollector.java

@@ -36,19 +36,8 @@ public SectionCollector(int frame, TaskQueueType importantRebuildQueueType, Task
 
     @Override
     public void visit(RenderSection section) {
-        // only process section (and associated render list) if it has content that needs rendering
-        if (section.getFlags() != 0) {
-            RenderRegion region = section.getRegion();
-            ChunkRenderList renderList = region.getRenderList();
-
-            if (renderList.getLastVisibleFrame() != this.frame) {
-                renderList.reset(this.frame, this.orderIsSorted());
-
-                this.renderLists.add(renderList);
-            }
-
-            renderList.add(section);
-        }
+        // when a chunk with no geometry to render is added to a zero-frame defer rebuild queue that means it will very likely receive geometry to render this frame. This flag makes sure it gets added to the render list now so that it doesn't remain invisible in frame, even though we blocked the frame to generate its mesh.
+        boolean predictNewGeometryThisFrame = false;
 
         // always add to rebuild lists though, because it might just not be built yet
         var pendingUpdate = section.getPendingUpdate();
@@ -61,14 +50,29 @@ public void visit(RenderSection section) {
                 this.needsRevisitForPendingUpdates = true;
                 return;
             }
-            
+
             var queueType = ChunkUpdateTypes.getQueueType(pendingUpdate, this.importantRebuildQueueType, this.importantSortQueueType);
             Queue<RenderSection> queue = this.sortedTaskLists.get(queueType);
 
             if (queue.size() < queueType.queueSizeLimit()) {
                 queue.add(section);
+                predictNewGeometryThisFrame = queueType == TaskQueueType.ZERO_FRAME_DEFER;
             }
         }
+
+        // only process section (and associated render list) if it has content that needs rendering
+        if (section.getFlags() != 0 || predictNewGeometryThisFrame) {
+            RenderRegion region = section.getRegion();
+            ChunkRenderList renderList = region.getRenderList();
+
+            if (renderList.getLastVisibleFrame() != this.frame) {
+                renderList.reset(this.frame, this.orderIsSorted());
+
+                this.renderLists.add(renderList);
+            }
+
+            renderList.add(section, predictNewGeometryThisFrame);
+        }
     }
 
     @Override