wisplite/distant-horizons-core-sharded
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

reminder comment about potential LodQuadTree concurrent issue

Browse Source
This commit is contained in:
James Seibel
2025-01-11 21:06:47 -06:00
parent 4f47c78acc
commit 9ede311c40
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
core/src/main/java/com/seibel/distanthorizons/core/render/LodQuadTree.java
+2 -2
View File
@@ -643,7 +643,7 @@ public class LodQuadTree extends QuadTree<LodRenderSection> implements IDebugRen
*/
public void reloadPos(long pos)
{
this.cachedRenderSourceByPos.invalidate(pos);
this.cachedRenderSourceByPos.invalidate(pos); // TODO will this cause issues? we may need to lock this invalidation if the cached data source is currently in use
this.sectionsToReload.add(pos);
// the adjacent locations also need to be updated to make sure lighting
@@ -652,7 +652,7 @@ public class LodQuadTree extends QuadTree<LodRenderSection> implements IDebugRen
for (EDhDirection direction : EDhDirection.ADJ_DIRECTIONS)
{
long adjacentPos = DhSectionPos.getAdjacentPos(pos, direction);
this.cachedRenderSourceByPos.invalidate(adjacentPos);
this.cachedRenderSourceByPos.invalidate(adjacentPos); // TODO will this cause issues? we may need to lock this invalidation if the cached data source is currently in use
this.sectionsToReload.add(adjacentPos);
}
}