Merge pull request #344 from Wyvest/main

cache `currentTimeMillis`

Author: tr7zw

src/main/java/dev/tr7zw/entityculling/EntityCullingModBase.java

@@ -36,6 +36,7 @@ public abstract class EntityCullingModBase extends EntityCullingVersionlessBase
     private int tickCounter = 0;
     public double lastTickTime = 0;
     private boolean freezeCamera = false;
+    public static volatile long tickTimeMs = System.currentTimeMillis();
     // preallocation sizes, adding a small buffer to hopefully avoid any resizing during runtime
     private static final int PREALLOC_BUFFER = 10;
     private int lastEntityCount = 0;
@@ -77,6 +78,7 @@ private void requestCull() {
     }
 
     public void clientTick() {
+        tickTimeMs = System.currentTimeMillis();
         // late init
         if (!lateInit) {
             lateInit = true;

src/main/java/dev/tr7zw/entityculling/mixin/BlockEntityRenderDispatcherMixin.java

@@ -74,7 +74,7 @@ public void tryExtractRenderState(BlockEntity blockEntity, float partialTicks,
             return;
         }
         if (blockEntity instanceof Cullable cullable) {
-            if (!cullable.isForcedVisible() && cullable.isCulled()) {
+            if (cullable.isCulled() && !cullable.isForcedVisible()) {
                 EntityCullingModBase.instance.skippedBlockEntities++;
                 info.setReturnValue(null);
                 if (EntityCullingModBase.instance.debugCollector.isRunning()) {
@@ -113,7 +113,7 @@ public <E extends BlockEntity> void render(E blockEntity, float f, com.mojang.bl
            return;
        }
     
-       if (!((Cullable) blockEntity).isForcedVisible() && ((Cullable) blockEntity).isCulled()) {
+       if (((Cullable) blockEntity).isCulled() && !((Cullable) blockEntity).isForcedVisible()) {
            EntityCullingModBase.instance.skippedBlockEntities++;
            info.cancel();
            return;

src/main/java/dev/tr7zw/entityculling/mixin/CullableMixin.java

@@ -5,6 +5,7 @@
 import net.minecraft.world.phys.*;
 import org.spongepowered.asm.mixin.Mixin;
 
+import dev.tr7zw.entityculling.EntityCullingModBase;
 import dev.tr7zw.entityculling.versionless.EntityCullingVersionlessBase;
 import dev.tr7zw.entityculling.access.Cullable;
 import net.minecraft.world.entity.Entity;
@@ -31,12 +32,12 @@ public class CullableMixin implements Cullable {
 
     @Override
     public void setTimeout() {
-        lasttime = System.currentTimeMillis() + 1000;
+        lasttime = EntityCullingModBase.tickTimeMs + 1000;
     }
 
     @Override
     public boolean isForcedVisible() {
-        return lasttime > System.currentTimeMillis();
+        return lasttime > EntityCullingModBase.tickTimeMs;
     }
 
     @Override

src/main/java/dev/tr7zw/entityculling/mixin/WorldRendererMixin.java

@@ -51,7 +51,7 @@ private void extractEntityRedir(Entity entity, float partialTick,
             return;
         }
         Cullable cullable = (Cullable) entity;
-        if (!cullable.isForcedVisible() && cullable.isCulled() && !NMSCullingHelper.ignoresCulling(entity)) {
+        if (cullable.isCulled() && !cullable.isForcedVisible() && !NMSCullingHelper.ignoresCulling(entity)) {
             EntityCullingModBase.instance.skippedEntities++;
             var state = new net.minecraft.client.renderer.entity.state.EntityRenderState();
 
@@ -158,7 +158,7 @@ private void renderEntity(Entity entity, double cameraX, double cameraY, double
           return;
       }
      Cullable cullable = (Cullable) entity;
-     if (!cullable.isForcedVisible() && cullable.isCulled() && !NMSCullingHelper.ignoresCulling(entity)) {
+     if (cullable.isCulled() && !cullable.isForcedVisible() && !NMSCullingHelper.ignoresCulling(entity)) {
         if (this.entityCulling$entityRenderDispatcher.getRenderer(entity) instanceof net.minecraft.client.renderer.entity.EntityRenderer entityRenderer
                 && entityRenderer instanceof dev.tr7zw.entityculling.access.EntityRendererInter entityRendererInter) {
             if (EntityCullingModBase.instance.config.renderNametagsThroughWalls && matrices != null