use ReentrantLock in ThreadLocalBufferManager (#1252)

Author: pjfanning

src/main/java/com/fasterxml/jackson/core/util/ThreadLocalBufferManager.java

@@ -5,6 +5,7 @@
 
 import java.util.*;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.locks.ReentrantLock;
 
 /**
  * For issue [jackson-core#400] We keep a separate Set of all SoftReferences to BufferRecyclers
@@ -23,7 +24,7 @@ class ThreadLocalBufferManager
      * A lock to make sure releaseBuffers is only executed by one thread at a time
      * since it iterates over and modifies the allSoftBufRecyclers.
      */
-    private final Object RELEASE_LOCK = new Object();
+    private final ReentrantLock RELEASE_LOCK = new ReentrantLock();
 
     /**
      * A set of all SoftReferences to all BufferRecyclers to be able to release them on shutdown.
@@ -64,17 +65,20 @@ public static ThreadLocalBufferManager instance() {
      * It will clear all bufRecyclers from the SoftRefs and release all SoftRefs itself from our set.
      */
     public int releaseBuffers() {
-        synchronized (RELEASE_LOCK) {
-            int count = 0;
+        int count = 0;
+        RELEASE_LOCK.lock();
+        try {
             // does this need to be in sync block too? Looping over Map definitely has to but...
             removeSoftRefsClearedByGc(); // make sure the refQueue is empty
             for (SoftReference<BufferRecycler> ref : _trackedRecyclers.keySet()) {
                 ref.clear(); // possibly already cleared by gc, nothing happens in that case
                 ++count;
             }
             _trackedRecyclers.clear(); //release cleared SoftRefs
-            return count;
+        } finally {
+            RELEASE_LOCK.unlock();
         }
+        return count;
     }
 
     public SoftReference<BufferRecycler> wrapAndTrack(BufferRecycler br) {