improve sign caching

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

@@ -0,0 +1,34 @@
+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.network.chat.Component;
+import net.minecraft.world.level.block.entity.SignText;
+
+// SignText is immutable (edits return a new instance), so whether it has a message 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 = EMPTY;
+            for (Component line : ((SignText) (Object) this).getMessages(filtered)) {
+                if (!line.getString().isEmpty()) {
+                    cached = PRESENT;
+                    break;
+                }
+            }
+            if (filtered) hasFilteredMessage = cached;
+            else hasMessage = cached;
+        }
+        return cached == PRESENT;
+    }
+}

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

@@ -12,6 +12,8 @@
 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 fr.madu59.obe.client.renderer.blockentity.ext.SignTextExt;
+import net.minecraft.world.level.block.entity.SignText;
 
 public class SpecialBlockEntityRenderingManager {
     public static boolean shouldSkipRendering(BlockEntity be) {
@@ -42,6 +44,11 @@ private static boolean isContainerEmpty(List<ItemStack> items) {
 
     private static boolean isEmpty(SignBlockEntity be){
         LocalPlayer player = Minecraft.getInstance().player;
-        return (be.getText(true) == null || !be.getText(true).hasMessage(player)) && (be.getText(false) == null || !be.getText(false).hasMessage(player));
+        boolean shouldFilter = player.isTextFilteringEnabled();
+        return !hasMessage(be.getText(true), shouldFilter) && !hasMessage(be.getText(false), shouldFilter);
+    }
+
+    private static boolean hasMessage(SignText text, boolean filtered){
+        return text != null && ((SignTextExt) text).obe$hasMessage(filtered);
     }
 }

src/main/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/main/resources/obe.mixins.json

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