*) one possible deadlock (because of nested object locks) removed in class kelondroMap

git-svn-id: https://svn.berlios.de/svnroot/repos/yacy/trunk@61 6c8d7289-2bf4-0310-a012-ef5d649a1542

source/de/anomic/kelondro/kelondroMap.java

@@ -128,12 +128,11 @@ public class kelondroMap {
 
     class writeQueue extends Thread {
 
-        LinkedList queue;
+        private LinkedList queue = new LinkedList();
         boolean run;
         
         public writeQueue() {
             run = true;
-            queue = new LinkedList();
         }
         
         public void stack(String key) {
@@ -144,20 +143,28 @@ public class kelondroMap {
                 workoff(key);
         }
         
-        public synchronized void workoff() {
-            if (queue.size() > 0) workoff((String) queue.removeFirst());
+        public void workoff() {
+            String newKey = null;
+            synchronized (this.queue) {
+                if (this.queue.size() > 0) {
+                    newKey = (String) this.queue.removeFirst();                    
+                }
+			}
+            if (newKey != null) workoff(newKey);            
         }
 
-        public synchronized void dequeue(String key) {
+        public void dequeue(String key) {
             // take out one entry
-            ListIterator i = queue.listIterator();
-            String k;
-            while (i.hasNext()) {
-                k = (String) i.next();
-                if (k.equals(key)) {
-                    i.remove();
-                    return;
-                }
+            synchronized (this.queue) {
+	            ListIterator i = queue.listIterator();
+	            String k;
+	            while (i.hasNext()) {
+	                k = (String) i.next();
+	                if (k.equals(key)) {
+	                    i.remove();
+	                    return;
+	                }
+	            }
             }
         }