70 lines
		
	
	
	
		
			5 KiB
			
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			70 lines
		
	
	
	
		
			5 KiB
			
		
	
	
	
		
			Diff
		
	
	
	
	
	
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
 | 
						|
From: Aikar <aikar@aikar.co>
 | 
						|
Date: Mon, 28 Mar 2016 19:55:45 -0400
 | 
						|
Subject: [PATCH] Only process BlockPhysicsEvent if a plugin has a listener
 | 
						|
 | 
						|
Saves on some object allocation and processing when no plugin listens to this
 | 
						|
 | 
						|
diff --git a/src/main/java/net/minecraft/server/MinecraftServer.java b/src/main/java/net/minecraft/server/MinecraftServer.java
 | 
						|
index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644
 | 
						|
--- a/src/main/java/net/minecraft/server/MinecraftServer.java
 | 
						|
+++ b/src/main/java/net/minecraft/server/MinecraftServer.java
 | 
						|
@@ -0,0 +0,0 @@ public abstract class MinecraftServer extends ReentrantBlockableEventLoop<TickTa
 | 
						|
 
 | 
						|
         while (iterator.hasNext()) {
 | 
						|
             ServerLevel worldserver = (ServerLevel) iterator.next();
 | 
						|
+            worldserver.hasPhysicsEvent =  org.bukkit.event.block.BlockPhysicsEvent.getHandlerList().getRegisteredListeners().length > 0; // Paper
 | 
						|
 
 | 
						|
             this.profiler.push(() -> {
 | 
						|
                 return worldserver + " " + worldserver.dimension().location();
 | 
						|
diff --git a/src/main/java/net/minecraft/server/level/ServerLevel.java b/src/main/java/net/minecraft/server/level/ServerLevel.java
 | 
						|
index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644
 | 
						|
--- a/src/main/java/net/minecraft/server/level/ServerLevel.java
 | 
						|
+++ b/src/main/java/net/minecraft/server/level/ServerLevel.java
 | 
						|
@@ -0,0 +0,0 @@ public class ServerLevel extends Level implements WorldGenLevel {
 | 
						|
     // CraftBukkit start
 | 
						|
     public final LevelStorageSource.LevelStorageAccess convertable;
 | 
						|
     public final UUID uuid;
 | 
						|
+    public boolean hasPhysicsEvent = true; // Paper
 | 
						|
 
 | 
						|
     @Override public LevelChunk getChunkIfLoaded(int x, int z) { // Paper - this was added in world too but keeping here for NMS ABI
 | 
						|
         return this.chunkSource.getChunk(x, z, false);
 | 
						|
diff --git a/src/main/java/net/minecraft/world/level/Level.java b/src/main/java/net/minecraft/world/level/Level.java
 | 
						|
index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644
 | 
						|
--- a/src/main/java/net/minecraft/world/level/Level.java
 | 
						|
+++ b/src/main/java/net/minecraft/world/level/Level.java
 | 
						|
@@ -0,0 +0,0 @@ public abstract class Level implements LevelAccessor, AutoCloseable {
 | 
						|
                 // CraftBukkit start
 | 
						|
                 iblockdata1.updateIndirectNeighbourShapes(this, blockposition, k, j - 1); // Don't call an event for the old block to limit event spam
 | 
						|
                 CraftWorld world = ((ServerLevel) this).getWorld();
 | 
						|
-                if (world != null) {
 | 
						|
+                if (world != null && ((ServerLevel)this).hasPhysicsEvent) { // Paper
 | 
						|
                     BlockPhysicsEvent event = new BlockPhysicsEvent(world.getBlockAt(blockposition.getX(), blockposition.getY(), blockposition.getZ()), CraftBlockData.fromData(iblockdata));
 | 
						|
                     this.getCraftServer().getPluginManager().callEvent(event);
 | 
						|
 
 | 
						|
diff --git a/src/main/java/net/minecraft/world/level/block/BushBlock.java b/src/main/java/net/minecraft/world/level/block/BushBlock.java
 | 
						|
index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644
 | 
						|
--- a/src/main/java/net/minecraft/world/level/block/BushBlock.java
 | 
						|
+++ b/src/main/java/net/minecraft/world/level/block/BushBlock.java
 | 
						|
@@ -0,0 +0,0 @@ public abstract class BushBlock extends Block {
 | 
						|
     public BlockState updateShape(BlockState state, Direction direction, BlockState neighborState, LevelAccessor world, BlockPos pos, BlockPos neighborPos) {
 | 
						|
         // CraftBukkit start
 | 
						|
         if (!state.canSurvive(world, pos)) {
 | 
						|
-            if (!org.bukkit.craftbukkit.event.CraftEventFactory.callBlockPhysicsEvent(world, pos).isCancelled()) {
 | 
						|
+            if (!(world instanceof net.minecraft.server.level.ServerLevel && ((net.minecraft.server.level.ServerLevel) world).hasPhysicsEvent) || !org.bukkit.craftbukkit.event.CraftEventFactory.callBlockPhysicsEvent(world, pos).isCancelled()) { // Paper
 | 
						|
                 return Blocks.AIR.defaultBlockState();
 | 
						|
             }
 | 
						|
         }
 | 
						|
diff --git a/src/main/java/net/minecraft/world/level/block/DoublePlantBlock.java b/src/main/java/net/minecraft/world/level/block/DoublePlantBlock.java
 | 
						|
index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644
 | 
						|
--- a/src/main/java/net/minecraft/world/level/block/DoublePlantBlock.java
 | 
						|
+++ b/src/main/java/net/minecraft/world/level/block/DoublePlantBlock.java
 | 
						|
@@ -0,0 +0,0 @@ public class DoublePlantBlock extends BushBlock {
 | 
						|
 
 | 
						|
     protected static void preventDropFromBottomPart(Level world, BlockPos pos, BlockState state, Player player) {
 | 
						|
         // CraftBukkit start
 | 
						|
-        if (org.bukkit.craftbukkit.event.CraftEventFactory.callBlockPhysicsEvent(world, pos).isCancelled()) {
 | 
						|
+        if (((net.minecraft.server.level.ServerLevel)world).hasPhysicsEvent && org.bukkit.craftbukkit.event.CraftEventFactory.callBlockPhysicsEvent(world, pos).isCancelled()) { // Paper
 | 
						|
             return;
 | 
						|
         }
 | 
						|
         // CraftBukkit end
 |