propose-patch

[org.ibex.core.git] / src / org / xwt / Scheduler.java

diff --git a/src/org/xwt/Scheduler.java b/src/org/xwt/Scheduler.java
index 6867376..9e939a3 100644 (file)
--- a/src/org/xwt/Scheduler.java
+++ b/src/org/xwt/Scheduler.java
@@ -1,4 +1,4 @@
-// Copyright 2003 Adam Megacz, see the COPYING file for licensing [GPL]
+// Copyright 2004 Adam Megacz, see the COPYING file for licensing [GPL]
 package org.xwt;
 
 import java.util.*;
@@ -8,33 +8,26 @@ import org.xwt.util.*;
 /** Implements cooperative multitasking */
 public class Scheduler {
 
-    // FIXME: prepending events messes with keysate -- make a "no re-ordering" invariant?
-
     // Public API Exposed to org.xwt /////////////////////////////////////////////////
 
     private static Scheduler singleton;
     public static interface Task { public abstract void perform() throws Exception; }
-
-    /** adds a task to the back of the queue */
     public static void add(Task t) { singleton.runnable.append(t); }
-
-    /** adds a task to the front of the queue (guaranteed to run next) */
-    public static void addAtFront(Task t) { singleton.runnable.prepend(t); }
-
     public static void init() { if (singleton == null) (singleton = Platform.getScheduler()).run(); }
 
     private static Task current = null;
-    public static Task current() { return current; }
 
-    /** synchronizd so that we can safely call it from an event-delivery thread, in-context */
     private static volatile boolean rendering = false;
     private static volatile boolean again = false;
+
+    /** synchronizd so that we can safely call it from an event-delivery thread, in-context */
     public static void renderAll() {
         if (rendering) { again = true; return; }
         synchronized(Scheduler.class) {
             try {
                 rendering = true;
                 do {
+                    // FEATURE: this could be cleaner
                     again = false;
                     for(int i=0; i<Surface.allSurfaces.size(); i++) {
                         Surface s = ((Surface)Surface.allSurfaces.elementAt(i));
@@ -68,10 +61,11 @@ public class Scheduler {
         while(true) {
             current = (Task)runnable.remove(true);
             try {
+                // FIXME hideous
                 synchronized(this) {
                     for(int i=0; i<Surface.allSurfaces.size(); i++) {
                         Surface s = (Surface)Surface.allSurfaces.elementAt(i);
-                        if (current instanceof JSFunction) {
+                        if (current instanceof JS) {
                             s._mousex = Integer.MAX_VALUE;
                             s._mousey = Integer.MAX_VALUE;
                         } else {
@@ -84,7 +78,8 @@ public class Scheduler {
                 renderAll();
             } catch (JSExn e) {
                 Log.info(Scheduler.class, "a JavaScript thread spawned with xwt.thread() threw an exception:");
-                Log.info(Scheduler.class, e);
+                Log.info(Scheduler.class, "JS Exception: " + e.getObject() + "\n" + e.backtrace());
+                Log.info(Scheduler.class,e);
             } catch (Exception e) {
                 Log.info(Scheduler.class, "a Task threw an exception which was caught by the scheduler:");
                 Log.info(Scheduler.class, e);