9
0
mirror of https://github.com/Samsuik/Sakura.git synced 2025-12-22 08:19:26 +00:00

Clean up store entity state

This commit is contained in:
Samsuik
2024-11-05 22:01:04 +00:00
parent 90526f9a54
commit b22464c2f5
2 changed files with 17 additions and 13 deletions

View File

@@ -6,49 +6,53 @@ Subject: [PATCH] Store Entity Data/State
diff --git a/src/main/java/me/samsuik/sakura/entity/EntityState.java b/src/main/java/me/samsuik/sakura/entity/EntityState.java diff --git a/src/main/java/me/samsuik/sakura/entity/EntityState.java b/src/main/java/me/samsuik/sakura/entity/EntityState.java
new file mode 100644 new file mode 100644
index 0000000000000000000000000000000000000000..10630c7e04a137ce766f4a45cb0f2e51bd967c98 index 0000000000000000000000000000000000000000..aeb5f128b369753cb20b7310fa382ddf2b6e8f0f
--- /dev/null --- /dev/null
+++ b/src/main/java/me/samsuik/sakura/entity/EntityState.java +++ b/src/main/java/me/samsuik/sakura/entity/EntityState.java
@@ -0,0 +1,37 @@ @@ -0,0 +1,41 @@
+package me.samsuik.sakura.entity; +package me.samsuik.sakura.entity;
+ +
+import net.minecraft.core.BlockPos; +import net.minecraft.core.BlockPos;
+import net.minecraft.world.entity.Entity; +import net.minecraft.world.entity.Entity;
+import net.minecraft.world.level.block.Blocks;
+import net.minecraft.world.phys.AABB; +import net.minecraft.world.phys.AABB;
+import net.minecraft.world.phys.Vec3; +import net.minecraft.world.phys.Vec3;
+import org.jspecify.annotations.NullMarked;
+ +
+import java.util.Optional; +import java.util.Optional;
+ +
+@NullMarked
+public record EntityState(Vec3 position, Vec3 momentum, AABB bb, Vec3 stuckSpeed, Optional<BlockPos> supportingPos, boolean onGround, float fallDistance) { +public record EntityState(Vec3 position, Vec3 momentum, AABB bb, Vec3 stuckSpeed, Optional<BlockPos> supportingPos, boolean onGround, float fallDistance) {
+ public static EntityState of(Entity entity) { + public static EntityState of(Entity entity) {
+ return new EntityState(entity.position(), entity.getDeltaMovement(), entity.getBoundingBox(), entity.stuckSpeedMultiplier(), entity.mainSupportingBlockPos, entity.onGround(), entity.fallDistance); + return new EntityState(
+ entity.position(), entity.getDeltaMovement(), entity.getBoundingBox(),
+ entity.stuckSpeedMultiplier(), entity.mainSupportingBlockPos,
+ entity.onGround(), entity.fallDistance
+ );
+ } + }
+ +
+ public void apply(Entity entity) { + public void apply(Entity entity) {
+ entity.setPos(this.position); + entity.setPos(this.position);
+ entity.setDeltaMovement(this.momentum); + entity.setDeltaMovement(this.momentum);
+ entity.setBoundingBox(this.bb); + entity.setBoundingBox(this.bb);
+ // null here is only safe for our use case (tnt and sand) + entity.makeStuckInBlock(Blocks.AIR.defaultBlockState(), this.stuckSpeed);
+ //noinspection DataFlowIssue
+ entity.makeStuckInBlock(null, this.stuckSpeed);
+ entity.onGround = this.onGround; + entity.onGround = this.onGround;
+ entity.mainSupportingBlockPos = this.supportingPos; + entity.mainSupportingBlockPos = this.supportingPos;
+ entity.fallDistance = this.fallDistance; + entity.fallDistance = this.fallDistance;
+ } + }
+ +
+ public void position(Entity entity) { + public void applyEntityPosition(Entity entity) {
+ entity.setPos(this.position); + entity.setPos(this.position);
+ entity.setBoundingBox(this.bb); + entity.setBoundingBox(this.bb);
+ } + }
+ +
+ public boolean isCurrentState(Entity entity) { + public boolean comparePositionAndMotion(Entity entity) {
+ return entity.position().equals(this.position) + return entity.position().equals(this.position)
+ && entity.getDeltaMovement().equals(this.momentum); + && entity.getDeltaMovement().equals(this.momentum);
+ // && entity.getBoundingBox().equals(bb); // uncomment if there are issues with morphed bounding boxes
+ } + }
+} +}
diff --git a/src/main/java/net/minecraft/world/entity/Entity.java b/src/main/java/net/minecraft/world/entity/Entity.java diff --git a/src/main/java/net/minecraft/world/entity/Entity.java b/src/main/java/net/minecraft/world/entity/Entity.java
index 2d8618541d2e673686485901ffc8e693545354a3..4a295091629acd9a540f1194b006f445e4432d4f 100644 index 2d8618541d2e673686485901ffc8e693545354a3..105b2d368891685642c6c5a7b08969a71085c9bb 100644
--- a/src/main/java/net/minecraft/world/entity/Entity.java --- a/src/main/java/net/minecraft/world/entity/Entity.java
+++ b/src/main/java/net/minecraft/world/entity/Entity.java +++ b/src/main/java/net/minecraft/world/entity/Entity.java
@@ -628,6 +628,25 @@ public abstract class Entity implements SyncedDataHolder, Nameable, EntityAccess @@ -628,6 +628,25 @@ public abstract class Entity implements SyncedDataHolder, Nameable, EntityAccess
@@ -71,7 +75,7 @@ index 2d8618541d2e673686485901ffc8e693545354a3..4a295091629acd9a540f1194b006f445
+ } + }
+ +
+ public final boolean compareState(Entity to) { + public final boolean compareState(Entity to) {
+ return to.entityState() != null && to.entityState().isCurrentState(this); + return to.entityState() != null && to.entityState().comparePositionAndMotion(this);
+ } + }
+ // Sakura end - store entity data/state + // Sakura end - store entity data/state

View File

@@ -438,11 +438,11 @@ index ac43d1d91f7ac554f1267b3d76e22bd8f06d8f08..1f278f6ef7e39df676334e7e5853dec4
this.guardEntityTick(this::tickNonPassenger, entity); this.guardEntityTick(this::tickNonPassenger, entity);
gameprofilerfiller.pop(); gameprofilerfiller.pop();
diff --git a/src/main/java/net/minecraft/world/entity/Entity.java b/src/main/java/net/minecraft/world/entity/Entity.java diff --git a/src/main/java/net/minecraft/world/entity/Entity.java b/src/main/java/net/minecraft/world/entity/Entity.java
index 4a295091629acd9a540f1194b006f445e4432d4f..c8b33f6d5b820da4b9e16b7d251ca4aad1875aa8 100644 index 105b2d368891685642c6c5a7b08969a71085c9bb..a26a562e283b306d265b53612d1c04eccb74e962 100644
--- a/src/main/java/net/minecraft/world/entity/Entity.java --- a/src/main/java/net/minecraft/world/entity/Entity.java
+++ b/src/main/java/net/minecraft/world/entity/Entity.java +++ b/src/main/java/net/minecraft/world/entity/Entity.java
@@ -647,6 +647,23 @@ public abstract class Entity implements SyncedDataHolder, Nameable, EntityAccess @@ -647,6 +647,23 @@ public abstract class Entity implements SyncedDataHolder, Nameable, EntityAccess
return to.entityState() != null && to.entityState().isCurrentState(this); return to.entityState() != null && to.entityState().comparePositionAndMotion(this);
} }
// Sakura end - store entity data/state // Sakura end - store entity data/state
+ // Sakura start - merge cannon entities + // Sakura start - merge cannon entities