improve sign caching

src/client/java/fr/madu59/obe/client/mixin/blockentity/sign/SignTextMixin.java

@@ -0,0 +1,27 @@
+package fr.madu59.obe.client.mixin.blockentity.sign;
+
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Unique;
+
+import fr.madu59.obe.client.renderer.blockentity.ext.SignTextExt;
+import net.minecraft.world.level.block.entity.SignText;
+
+// SignText is immutable (edits return a new instance), so hasMessage can be computed once per instance
+// instead of streaming and stringifying every line on every frame
+@Mixin(SignText.class)
+public abstract class SignTextMixin implements SignTextExt {
+    @Unique private static final byte UNKNOWN = 0, EMPTY = 1, PRESENT = 2;
+    @Unique private byte hasMessage = UNKNOWN;
+    @Unique private byte hasFilteredMessage = UNKNOWN;
+
+    @Override
+    public boolean obe$hasMessage(boolean filtered) {
+        byte cached = filtered ? hasFilteredMessage : hasMessage;
+        if (cached == UNKNOWN) {
+            cached = ((SignText) (Object) this).hasMessage(filtered) ? PRESENT : EMPTY;
+            if (filtered) hasFilteredMessage = cached;
+            else hasMessage = cached;
+        }
+        return cached == PRESENT;
+    }
+}

src/client/java/fr/madu59/obe/client/renderer/blockentity/SpecialBlockEntityRenderingManager.java

@@ -3,13 +3,15 @@
 import java.util.List;
 
 import fr.madu59.obe.client.config.SettingsManager;
+import fr.madu59.obe.client.renderer.blockentity.ext.SignTextExt;
 import net.minecraft.client.Minecraft;
 import net.minecraft.world.item.ItemStack;
 import net.minecraft.world.level.block.entity.BeaconBlockEntity;
 import net.minecraft.world.level.block.entity.BlockEntity;
 import net.minecraft.world.level.block.entity.CampfireBlockEntity;
 import net.minecraft.world.level.block.entity.ShelfBlockEntity;
 import net.minecraft.world.level.block.entity.SignBlockEntity;
+import net.minecraft.world.level.block.entity.SignText;
 import net.minecraft.world.level.block.entity.SignTextSlot;
 
 public class SpecialBlockEntityRenderingManager {
@@ -41,6 +43,10 @@ private static boolean isContainerEmpty(List<ItemStack> items) {
 
     private static boolean isEmpty(SignBlockEntity be){
         boolean shouldFilter = Minecraft.getInstance().isTextFilteringEnabled();
-        return (be.getText(SignTextSlot.FRONT) == null || !be.getText(SignTextSlot.FRONT).hasMessage(shouldFilter)) && (be.getText(SignTextSlot.BACK) == null || !be.getText(SignTextSlot.BACK).hasMessage(shouldFilter));
+        return !hasMessage(be.getText(SignTextSlot.FRONT), shouldFilter) && !hasMessage(be.getText(SignTextSlot.BACK), shouldFilter);
+    }
+
+    private static boolean hasMessage(SignText text, boolean filtered){
+        return text != null && ((SignTextExt) text).obe$hasMessage(filtered);
     }
 }

src/client/java/fr/madu59/obe/client/renderer/blockentity/ext/SignTextExt.java

@@ -0,0 +1,5 @@
+package fr.madu59.obe.client.renderer.blockentity.ext;
+
+public interface SignTextExt {
+    boolean obe$hasMessage(boolean filtered);
+}

src/client/resources/obe.client.mixins.json

@@ -12,6 +12,7 @@
 		"blockentity.decoratedpot.DecoratedPotBlockEntityMixin",
 		"blockentity.shulkerbox.ShulkerBoxBlockEntityMixin",
 		"blockentity.lectern.LecternBlockEntityMixin",
+		"blockentity.sign.SignTextMixin",
 		"blockentity.compat.lootr.LootrChestBlockEntityMixin",
 		"blockentity.compat.lootr.LootrShulkerBoxBlockEntityMixin",
 		"blockentity.BlockEntityMixin",