Spigot has patched this issue inside MapIcon, meaning that we no longer need to maintain this patch; Spigots patch also fixes #668 in that it will verify the length of the array, as well as protect against a negative type value being fetched from the array. Only real change is that Spigots patch returns a MapIcon.Type.PLAYER, instead of the RED_MARKER as originally PR'd by Aikar.
		
			
				
	
	
		
			46 lines
		
	
	
	
		
			1.2 KiB
			
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			46 lines
		
	
	
	
		
			1.2 KiB
			
		
	
	
	
		
			Diff
		
	
	
	
	
	
From a5a7153cde37ce84714514cd647693113fdea266 Mon Sep 17 00:00:00 2001
 | 
						|
From: vemacs <d@nkmem.es>
 | 
						|
Date: Wed, 23 Nov 2016 12:53:43 -0500
 | 
						|
Subject: [PATCH] Misc Utils
 | 
						|
 | 
						|
 | 
						|
diff --git a/src/main/java/com/destroystokyo/paper/utils/CachedSizeConcurrentLinkedQueue.java b/src/main/java/com/destroystokyo/paper/utils/CachedSizeConcurrentLinkedQueue.java
 | 
						|
new file mode 100644
 | 
						|
index 00000000..d60ecbb1
 | 
						|
--- /dev/null
 | 
						|
+++ b/src/main/java/com/destroystokyo/paper/utils/CachedSizeConcurrentLinkedQueue.java
 | 
						|
@@ -0,0 +1,31 @@
 | 
						|
+package com.destroystokyo.paper.utils;
 | 
						|
+
 | 
						|
+import java.util.concurrent.ConcurrentLinkedQueue;
 | 
						|
+import java.util.concurrent.atomic.LongAdder;
 | 
						|
+
 | 
						|
+public class CachedSizeConcurrentLinkedQueue<E> extends ConcurrentLinkedQueue<E> {
 | 
						|
+    private final LongAdder cachedSize = new LongAdder();
 | 
						|
+
 | 
						|
+    @Override
 | 
						|
+    public boolean add(E e) {
 | 
						|
+        boolean result = super.add(e);
 | 
						|
+        if (result) {
 | 
						|
+            cachedSize.increment();
 | 
						|
+        }
 | 
						|
+        return result;
 | 
						|
+    }
 | 
						|
+
 | 
						|
+    @Override
 | 
						|
+    public E poll() {
 | 
						|
+        E result = super.poll();
 | 
						|
+        if (result != null) {
 | 
						|
+            cachedSize.decrement();
 | 
						|
+        }
 | 
						|
+        return result;
 | 
						|
+    }
 | 
						|
+
 | 
						|
+    @Override
 | 
						|
+    public int size() {
 | 
						|
+        return cachedSize.intValue();
 | 
						|
+    }
 | 
						|
+}
 | 
						|
-- 
 | 
						|
2.12.2
 | 
						|
 |