Fix #822

Signed-off-by: Ross Allan <rallanpcl@gmail.com>

src/common/nallar/tickthreading/global/ThreadLocals.java

@@ -5,6 +5,8 @@
 import java.util.HashSet;
 import java.util.LinkedList;
 
+import nallar.tickthreading.util.BooleanThreadLocal;
+
 @SuppressWarnings ("UnusedDeclaration")
 public class ThreadLocals {
 	public static final ThreadLocal redPowerBlockUpdateSet = new HashSetThreadLocal();
@@ -55,16 +57,6 @@ protected Object initialValue() {
 		}
 	}
 
-	private static class BooleanThreadLocal extends ThreadLocal {
-		BooleanThreadLocal() {
-		}
-
-		@Override
-		protected Object initialValue() {
-			return Boolean.FALSE;
-		}
-	}
-
 	public static class MapChunkTempByteArrayThreadLocal extends ThreadLocal<byte[]> {
 		@Override
 		public byte[] initialValue() {

src/common/nallar/tickthreading/util/BooleanThreadLocal.java

@@ -30,7 +30,6 @@ public Boolean initialValue() {
 	@Override
 	public void set(Boolean value) {
 		if (value == get()) {
-			Log.severe("Pointless repeat set on threadlocal", new Throwable());
 			return;
 		}
 		super.set(value);
@@ -42,16 +41,14 @@ public void set(Boolean value) {
 					return;
 				}
 			} while (true);
-		} else if (value == Boolean.FALSE) {
+		} else {
 			do {
 				int old = $.getIntVolatile(this, index);
 				int next = old - 1;
 				if ($.compareAndSwapInt(this, index, old, next)) {
 					return;
 				}
 			} while (true);
-		} else {
-			throw new Error("Must use Boolean.TRUE/FALSE.");
 		}
 	}
 
@@ -74,16 +71,14 @@ public Boolean getAndSet(Boolean value) {
 					return oldValue;
 				}
 			} while (true);
-		} else if (value == Boolean.FALSE) {
+		} else {
 			do {
 				int old = $.getIntVolatile(this, index);
 				int next = old - 1;
 				if ($.compareAndSwapInt(this, index, old, next)) {
 					return oldValue;
 				}
 			} while (true);
-		} else {
-			throw new Error("Must use Boolean.TRUE/FALSE.");
 		}
 	}