X-Git-Url: http://dolda2000.com/gitweb/?a=blobdiff_plain;f=src%2Fjagi%2Fevent%2FDriver.java;h=8fe83a7e4b55bc7011bad467c7957f0d984e1c0a;hb=1ee6412bc01b15aa8fad07a8bbce694ca099a8cb;hp=59f54a3b72e634a700c2bad959d3780e477d4a55;hpb=12032480fba70f0521b0ec7ef7bc8db10fc87636;p=jagi.git

diff --git a/src/jagi/event/Driver.java b/src/jagi/event/Driver.java
index 59f54a3..8fe83a7 100644
--- a/src/jagi/event/Driver.java
+++ b/src/jagi/event/Driver.java
@@ -58,6 +58,7 @@ public class Driver {
 	final Map<Watcher, SelectionKey> watching = new IdentityHashMap<>();
 	final Heap<Watcher, Double> timeheap = new Heap<>(Comparator.naturalOrder());
 	final Map<Watcher, Object> paused = new IdentityHashMap<>();
+	final Collection<SelectionKey> cancelled = new HashSet<>();
 
 	SelectPool(SelectorProvider provider) {
 	    this.provider = provider;
@@ -112,14 +113,26 @@ public class Driver {
 			    return;
 			}
 			if(first != null)
-			    timeout = Math.max((long)Math.ceil((first - now) * 1000), 0);
+			    timeout = Math.max((long)Math.ceil((first - now) * 1000), 1);
 		    }
+		    Collection<SelectionKey> precancelled;
+		    synchronized(cancelled) {
+			precancelled = new ArrayList<>(cancelled);
+		    }
+		    if(!precancelled.isEmpty())
+			timeout = 1;
 		    poll.selectedKeys().clear();
 		    try {
 			poll.select(timeout);
 		    } catch(IOException e) {
 			throw(new RuntimeException(e));
 		    }
+		    if(!precancelled.isEmpty()) {
+			synchronized(cancelled) {
+			    cancelled.removeAll(precancelled);
+			    cancelled.notifyAll();
+			}
+		    }
 		    for(SelectionKey key : poll.selectedKeys())
 			handle((Watcher)key.attachment(), key.readyOps());
 		    now = time();
@@ -171,6 +184,11 @@ public class Driver {
 	void add(Watcher w, SelectableChannel ch) {
 	    if(watching.containsKey(w) || paused.containsKey(w) || timeheap.contains(w))
 		throw(new IllegalStateException(w + ": already registered"));
+	    try {
+		ch.configureBlocking(false);
+	    } catch(IOException e) {
+		throw(new RuntimeException(ch + ": could not make non-blocking", e));
+	    }
 	    int evs = w.events();
 	    double timeout = w.timeout();
 	    boolean hastime = timeout < Double.POSITIVE_INFINITY;
@@ -193,14 +211,28 @@ public class Driver {
 	    SelectionKey wc = watching.remove(w);
 	    Object tc = timeheap.remove(w);
 	    Object pc = paused.remove(w);
-	    if(wc != null)
-		wc.cancel();
+	    if(wc != null) {
+		synchronized(cancelled) {
+		    cancelled.add(wc);
+		    wc.cancel();
+		    poll.wakeup();
+		    boolean irq = false;
+		    while(cancelled.contains(wc)) {
+			try {
+			    cancelled.wait();
+			} catch(InterruptedException e) {
+			    irq = true;
+			}
+		    }
+		    if(irq)
+			Thread.currentThread().interrupt();
+		}
+	    }
 	    if(((wc != null) || (tc != null)) && (pc != null))
 		throw(new RuntimeException(w + ": inconsistent internal state"));
 	    if(wc == null)
 		throw(new IllegalStateException(w + ": not registered"));
 	    submit(() -> close(w));
-	    poll.wakeup();
 	}
 
 	void update(Watcher w) {