2013-04-23 01:27:55 +00:00
|
|
|
From e5359d2e13ff1b81e120159a27421894c77aec33 Mon Sep 17 00:00:00 2001
|
2013-01-19 08:22:07 +00:00
|
|
|
From: Mike Primm <mike@primmhome.com>
|
|
|
|
Date: Wed, 16 Jan 2013 15:27:22 -0600
|
|
|
|
Subject: [PATCH] Alternate, sync-free-but-safe chunk reference cache
|
|
|
|
|
|
|
|
|
|
|
|
diff --git a/src/main/java/net/minecraft/server/World.java b/src/main/java/net/minecraft/server/World.java
|
2013-04-13 07:06:23 +00:00
|
|
|
index af85d8b..eb03aba 100644
|
2013-01-19 08:22:07 +00:00
|
|
|
--- a/src/main/java/net/minecraft/server/World.java
|
|
|
|
+++ b/src/main/java/net/minecraft/server/World.java
|
2013-04-13 07:06:23 +00:00
|
|
|
@@ -293,15 +293,13 @@ public abstract class World implements IBlockAccess {
|
2013-01-19 08:22:07 +00:00
|
|
|
|
|
|
|
// CraftBukkit start
|
|
|
|
public Chunk getChunkAt(int i, int j) {
|
|
|
|
- Chunk result = null;
|
|
|
|
- synchronized (this.chunkLock) {
|
|
|
|
- if (this.lastChunkAccessed == null || this.lastXAccessed != i || this.lastZAccessed != j) {
|
|
|
|
- this.lastChunkAccessed = this.chunkProvider.getOrCreateChunk(i, j);
|
|
|
|
- this.lastXAccessed = i;
|
|
|
|
- this.lastZAccessed = j;
|
|
|
|
- }
|
|
|
|
- result = this.lastChunkAccessed;
|
|
|
|
+ //synchronized (this.chunkLock) {
|
|
|
|
+ Chunk result = this.lastChunkAccessed; // Exploit fact that read is atomic
|
|
|
|
+ if (result == null || result.x != i || result.z != j) {
|
|
|
|
+ result = this.chunkProvider.getOrCreateChunk(i, j);
|
|
|
|
+ this.lastChunkAccessed = result; // Exploit fact that write is atomic
|
|
|
|
}
|
|
|
|
+ //}
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
// CraftBukkit end
|
|
|
|
--
|
2013-04-10 02:36:11 +00:00
|
|
|
1.8.2.1
|
2013-01-19 08:22:07 +00:00
|
|
|
|