aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch')
-rw-r--r--patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch43
1 files changed, 43 insertions, 0 deletions
diff --git a/patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch b/patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch
new file mode 100644
index 0000000000..37691c2cd2
--- /dev/null
+++ b/patches/server/0371-Don-t-run-entity-collision-code-if-not-needed.patch
@@ -0,0 +1,43 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Spottedleaf <[email protected]>
+Date: Wed, 15 Apr 2020 17:56:07 -0700
+Subject: [PATCH] Don't run entity collision code if not needed
+
+Will not run if:
+Max entity cramming is disabled and the max collisions per entity is less than or equal to 0.
+Entity#isPushable() returns false, meaning all entities will not be able to collide with this
+entity anyways.
+The entity's current team collision rule causes them to NEVER collide.
+
+Co-authored-by: Owen1212055 <[email protected]>
+
+diff --git a/src/main/java/net/minecraft/world/entity/LivingEntity.java b/src/main/java/net/minecraft/world/entity/LivingEntity.java
+index 2d28d9a42e89b7efadd4e798c3da61565221a5bd..8577eb85cfe6d061e9e8b7e380db452a8c4ec858 100644
+--- a/src/main/java/net/minecraft/world/entity/LivingEntity.java
++++ b/src/main/java/net/minecraft/world/entity/LivingEntity.java
+@@ -3398,10 +3398,24 @@ public abstract class LivingEntity extends Entity implements Attackable {
+ if (this.level().isClientSide()) {
+ this.level().getEntities(EntityTypeTest.forClass(net.minecraft.world.entity.player.Player.class), this.getBoundingBox(), EntitySelector.pushableBy(this)).forEach(this::doPush);
+ } else {
++ // Paper start - don't run getEntities if we're not going to use its result
++ if (!this.isPushable()) {
++ return;
++ }
++ net.minecraft.world.scores.Team team = this.getTeam();
++ if (team != null && team.getCollisionRule() == net.minecraft.world.scores.Team.CollisionRule.NEVER) {
++ return;
++ }
++
++ int i = this.level().getGameRules().getInt(GameRules.RULE_MAX_ENTITY_CRAMMING);
++ if (i <= 0 && this.level().paperConfig().collisions.maxEntityCollisions <= 0) {
++ return;
++ }
++ // Paper end - don't run getEntities if we're not going to use its result
+ List<Entity> list = this.level().getEntities((Entity) this, this.getBoundingBox(), EntitySelector.pushableBy(this));
+
+ if (!list.isEmpty()) {
+- int i = this.level().getGameRules().getInt(GameRules.RULE_MAX_ENTITY_CRAMMING);
++ // Paper - moved up
+
+ if (i > 0 && list.size() > i - 1 && this.random.nextInt(4) == 0) {
+ int j = 0;