remove autocloseable of system

Author: elebras1

examples/src/main/java/com/github/elebras1/flecs/examples/AdvancedSystemExample.java

@@ -149,15 +149,6 @@ public static void main(String[] args) {
             System.out.println("\nRunning one frame to show phase order:");
             world.progress(0.016f);
 
-            // Clean up
-            oneSecondSystem.close();
-            everyOtherFrame.close();
-            mtMoveSystem.close();
-            immediateSystem.close();
-            preUpdateSys.close();
-            onUpdateSys.close();
-            postUpdateSys.close();
-
             System.out.println("\n=== Advanced Systems Example Complete ===");
         }
     }

examples/src/main/java/com/github/elebras1/flecs/examples/SystemExample.java

@@ -117,11 +117,6 @@ public static void main(String[] args) {
                 });
             }
 
-            // Clean up systems
-            moveSystem.close();
-            debugSystem.close();
-            taskSystem.close();
-
             System.out.println("\n=== Systems Example Complete ===");
         }
     }

src/main/java/com/github/elebras1/flecs/FlecsSystem.java

@@ -5,68 +5,44 @@
 import java.lang.foreign.Arena;
 import java.lang.foreign.MemorySegment;
 
-public class FlecsSystem implements AutoCloseable {
+public class FlecsSystem {
 
     private final World world;
     private final long entityId;
     private final Entity entity;
-    private final Arena arena;
-    private boolean closed = false;
 
     FlecsSystem(World world, long entityId) {
         this.world = world;
         this.entityId = entityId;
         this.entity = world.obtainEntity(entityId);
-        this.arena = Arena.ofConfined();
     }
 
     public void run() {
-        this.checkClosed();
         flecs_h.ecs_run(this.world.nativeHandle(), this.entityId, 0.0f, MemorySegment.NULL);
     }
 
     public void run(float deltaTime) {
-        this.checkClosed();
         flecs_h.ecs_run(this.world.nativeHandle(), this.entityId, deltaTime, MemorySegment.NULL);
     }
 
     public long id() {
-        this.checkClosed();
         return this.entityId;
     }
 
     public Entity entity() {
-        this.checkClosed();
         return this.entity;
     }
 
     public void enable() {
-        this.checkClosed();
         this.entity.remove(FlecsConstants.EcsDisabled);
     }
 
     public void disable() {
-        this.checkClosed();
         this.entity.add(FlecsConstants.EcsDisabled);
     }
 
     public boolean isEnabled() {
-        this.checkClosed();
         return !this.entity.has(FlecsConstants.EcsDisabled);
     }
-
-    private void checkClosed() {
-        if (this.closed) {
-            throw new IllegalStateException("System has been closed");
-        }
-    }
-
-    @Override
-    public void close() {
-        if (!this.closed) {
-            this.arena.close();
-            this.closed = true;
-        }
-    }
 }
 

src/main/java/com/github/elebras1/flecs/SystemBuilder.java

@@ -9,8 +9,8 @@
 public class SystemBuilder {
 
     private final World world;
-    private final Arena arena;
     private final MemorySegment desc;
+    private final Arena arena;
     private int termCount = 0;
     private static final long TERM_SIZE = ecs_term_t.layout().byteSize();
     private Query.IterCallback iterCallback;
@@ -27,12 +27,10 @@ public SystemBuilder(World world) {
     public SystemBuilder(World world, String name) {
         this(world);
         MemorySegment nameSegment = this.arena.allocateFrom(name);
-        
-        try (Arena tempArena = Arena.ofConfined()) {
-            MemorySegment entityDescTemp = ecs_entity_desc_t.allocate(tempArena);
-            ecs_entity_desc_t.name(entityDescTemp, nameSegment);
-            ecs_system_desc_t.entity(this.desc, flecs_h.ecs_entity_init(world.nativeHandle(), entityDescTemp));
-        }
+
+        MemorySegment entityDescTemp = ecs_entity_desc_t.allocate(this.arena);
+        ecs_entity_desc_t.name(entityDescTemp, nameSegment);
+        ecs_system_desc_t.entity(this.desc, flecs_h.ecs_entity_init(world.nativeHandle(), entityDescTemp));
     }
 
     public SystemBuilder kind(long phase) {
@@ -243,6 +241,8 @@ private FlecsSystem build() {
 
         this.world.registerSystemCallbacks(systemId, this.iterCallback, this.runCallback, this.entityCallback);
 
+        this.arena.close();
+
         return new FlecsSystem(this.world, systemId);
     }
 }