2003/06/16 07:58:56
[org.ibex.core.git] / src / org / xwt / Box.java
index 2ab32b9..738d102 100644 (file)
@@ -263,13 +263,13 @@ public final class Box extends JS.Scope {
     // Instance Data: IndexOf  ////////////////////////////////////////////////////////////
 
     /** The indexof() Function; created lazily */
-    public JS.Function indexof = null;
-    public JS.Function indexof() { if (indexof == null) indexof = new IndexOf(); return indexof; }
+    public JS.Callable indexof = null;
+    public JS.Callable indexof() { if (indexof == null) indexof = new IndexOf(); return indexof; }
 
     /** a trivial private class to serve as the box.indexof function object */
-    private class IndexOf extends JS.Function {
+    private class IndexOf extends JS.Callable {
         public IndexOf() { this.setSeal(true); }
-        public Object _call(JS.Array args) throws JS.Exn {
+        public Object call(JS.Array args) throws JS.Exn {
             if (args.length() != 1 || args.elementAt(0) == null || !(args.elementAt(0) instanceof Box)) return new Integer(-1);
             Box b = (Box)args.elementAt(0);
             if (b.getParent() != Box.this) {
@@ -424,7 +424,7 @@ public final class Box extends JS.Scope {
     }
 
     /** loads the image described by string str, possibly blocking for a network load */
-    static ImageDecoder getImage(String str, final Function callback) {
+    static ImageDecoder getImage(String str, final JS.Callable callback) {
 
         if (str.indexOf(':') == -1) {
             String s = str;
@@ -460,8 +460,8 @@ public final class Box extends JS.Scope {
                             if (ret != -1) bytesDownloaded += ret;
                             if (clear && callback != null) {
                                 clear = false;
-                                ThreadMessage.newthread(new JS.Function() {
-                                        public Object _call(JS.Array args_) throws JS.Exn {
+                                ThreadMessage.newthread(new JS.Callable() {
+                                        public Object call(JS.Array args_) throws JS.Exn {
                                             try {
                                                JS.Array args = new JS.Array();
                                                args.addElement(new Double(bytesDownloaded));
@@ -518,7 +518,7 @@ public final class Box extends JS.Scope {
         } else {
             image = getPicture(s);
             if (image == null) {
-                if (Log.on) Log.log(Box.class, "unable to load image " + s + " at " + JS.getCurrentFunctionSourceName());
+                if (Log.on) Log.log(Box.class, "unable to load image " + s + " at " + Context.getCurrentSourceNameAndLine());
                 return;
            }
             if (sizetoimage) syncSizeToImage();
@@ -539,7 +539,7 @@ public final class Box extends JS.Scope {
             if (border == null) {
                 ImageDecoder id = getImage(s, null);
                 if (id == null) {
-                    if (Log.on) Log.log(this, "unable to load border image " + s + " at " + JS.getCurrentFunctionSourceName());
+                    if (Log.on) Log.log(this, "unable to load border image " + s + " at " + Context.getCurrentSourceNameAndLine());
                     return;
                 }
                 int[] data = id.getData();
@@ -634,7 +634,7 @@ public final class Box extends JS.Scope {
     }
 
     /** creates a new box from an anonymous template; <tt>ids</tt> is passed through to Template.apply() */
-    Box(Template anonymous, Vec pboxes, Vec ptemplates, Function callback, int numerator, int denominator) {
+    Box(Template anonymous, Vec pboxes, Vec ptemplates, JS.Callable callback, int numerator, int denominator) {
         super(null);
         set(dmax, 0, Integer.MAX_VALUE);
         set(dmax, 1, Integer.MAX_VALUE);
@@ -646,7 +646,7 @@ public final class Box extends JS.Scope {
 
     /** creates a new box from an unresolved templatename and an importlist; use "box" for an untemplatized box */
     public Box(String templatename, String[] importlist) { this(templatename, importlist, null); }
-    public Box(String templatename, String[] importlist, Function callback) {
+    public Box(String templatename, String[] importlist, JS.Callable callback) {
         super(null);
         set(dmax, 0, Integer.MAX_VALUE);
         set(dmax, 1, Integer.MAX_VALUE);
@@ -711,12 +711,12 @@ public final class Box extends JS.Scope {
             for(Box cur = this; cur != null && (cur == this || cur == this.getParent()); cur = cur.getParent()) {
                 cur.dirty(pos(0) + min(oldsize(0) - bw, size(0) - bw),
                           pos(1),
-                          Math.abs(oldsize(0) - size(0)) + bw,
+                          java.lang.Math.abs(oldsize(0) - size(0)) + bw,
                           max(oldsize(1), size(1)));
                 cur.dirty(pos(0),
                           pos(1) + min(oldsize(1) - bh, size(1) - bh),
                           max(oldsize(0), size(0)),
-                          Math.abs(oldsize(1) - size(1)) + bh);
+                          java.lang.Math.abs(oldsize(1) - size(1)) + bh);
             }
             
         // SLOWPATH: dirty ourselves, as well as our former position on our parent
@@ -741,11 +741,11 @@ public final class Box extends JS.Scope {
         if (++surface.sizePosChangesSinceLastRender >= 500) {
             if (surface.sizePosChangesSinceLastRender == 500) {
                 if (Log.on) Log.log(this, "Warning, more than 500 SizeChange/PosChange traps triggered since last complete render");
-                if (Log.on) Log.log(this, "    interpreter is at " + JS.getCurrentFunctionSourceName());
+                if (Log.on) Log.log(this, "    interpreter is at " + Context.getCurrentSourceNameAndLine());
            /*
                 try {
                     Trap t = sizechange ? Trap.getTrap(this, "SizeChange") : Trap.getTrap(this, "PosChange");
-                    InterpretedFunction f = (InterpretedFunction)t.f;
+                    InterpretedJS.Callable f = (InterpretedJS.Callable)t.f;
                     if (Log.on) Log.log(this, "Current trap is at " + f.getSourceName() + ":" + f.getLineNumbers()[0]);
                 } catch (Throwable t) { }
            */
@@ -1062,7 +1062,7 @@ public final class Box extends JS.Scope {
     public Object get(int i) {
         if (redirect == null) return null;
         if (redirect != this) return redirect.get(i);
-        return i >= numChildren() ? null : getChild(i);
+        return i >= numChildren() || i < 0 ? null : getChild(i);
     }
 
     /**
@@ -1072,12 +1072,13 @@ public final class Box extends JS.Scope {
      *  WARNING: O(n) runtime, unless i == numChildren()
      */
     public void put(int i, Object value) {
+       if (i < 0) return;
 
         if (value != null && !(value instanceof Box)) {
-            if (Log.on) Log.log(this, "attempt to set a numerical property on a box to anything other than a box at " + JS.getCurrentFunctionSourceName());
+            if (Log.on) Log.log(this, "attempt to set a numerical property on a box to anything other than a box at " + Context.getCurrentSourceNameAndLine());
 
         } else if (redirect == null) {
-            if (Log.on) Log.log(this, "attempt to add/remove children to/from a node with a null redirect at " +  JS.getCurrentFunctionSourceName());
+            if (Log.on) Log.log(this, "attempt to add/remove children to/from a node with a null redirect at " +  Context.getCurrentSourceNameAndLine());
         } else if (redirect != this) {
             Box b = value == null ? (Box)redirect.get(i) : (Box)value;
             redirect.put(i, value);
@@ -1091,7 +1092,7 @@ public final class Box extends JS.Scope {
             }
 
         } else if (value instanceof RootProxy) {
-            if (Log.on) Log.log(this, "attempt to reparent a box via its proxy object at " + JS.getCurrentFunctionSourceName());
+            if (Log.on) Log.log(this, "attempt to reparent a box via its proxy object at " + Context.getCurrentSourceNameAndLine());
 
         } else {
             Box newnode = (Box)value;
@@ -1099,14 +1100,17 @@ public final class Box extends JS.Scope {
             // check if box being moved is currently target of a redirect
             for(Box cur = newnode.getParent(); cur != null; cur = cur.getParent())
                 if (cur.redirect == newnode) {
-                    if (Log.on) Log.log(this, "attempt to move a box that is the target of a redirect at "+ JS.getCurrentFunctionSourceName());
+                    if (Log.on) Log.log(this, "attempt to move a box that is the target of a redirect at "+
+                                       Context.getCurrentSourceNameAndLine());
                     return;
                 }
 
             // check for recursive ancestor violation
             for(Box cur = this; cur != null; cur = cur.getParent())
                 if (cur == newnode) {
-                    if (Log.on) Log.log(this, "attempt to make a node a parent of its own ancestor at " + JS.getCurrentFunctionSourceName());
+                    if (Log.on) Log.log(this, "attempt to make a node a parent of its own ancestor at " +
+                                       Context.getCurrentSourceNameAndLine());
+                    if (Log.on) Log.log(this, "box == " + this + "  ancestor == " + newnode);
                     return;
                 }
 
@@ -1185,13 +1189,13 @@ public final class Box extends JS.Scope {
         Object ret = super.get(name);
         if (name.startsWith("$") && ret == null)
             if (Log.on) Log.log(this, "WARNING: attempt to access " + name + ", but no child with id=\"" + name.substring(1) + "\" found; " +
-                                JS.getFileAndLine());
+                                Context.getCurrentSourceNameAndLine());
         return ret;
     }
 
     public Object[] keys() {
         Object[] ret = new Object[numChildren()];
-        for(int i=0; i<ret.length; i++) ret[i] = get(i);
+        for(int i=0; i<ret.length; i++) ret[i] = new Integer(i);
         return ret;
     }
 
@@ -1205,8 +1209,9 @@ public final class Box extends JS.Scope {
     public void put(Object name_, Object value, boolean ignoretraps, RootProxy rp) {
        if (name_ instanceof Number) { put(((Number)name_).intValue(), value); return; }
        String name = (String)name_;
+       if (name == null) return;  // FIXME, shouldn't be necessary
         if (name.startsWith("xwt_")) {
-            if (Log.on) Log.log(this, "attempt to set reserved property " + name + " at " + JS.getFileAndLine());
+            if (Log.on) Log.log(this, "attempt to set reserved property " + name + " at " + Context.getCurrentSourceNameAndLine());
             return;
        }
 
@@ -1231,18 +1236,22 @@ public final class Box extends JS.Scope {
         }
 
         if (name.charAt(0) == '_') {
-            if (value != null && !(value instanceof Function)) {
-                if (Log.on) Log.log(this, "attempt to put a non-function value to " + name + " at " + JS.getFileAndLine());
+            if (value != null && !(value instanceof JS.Callable)) {
+                if (Log.on) Log.log(this, "attempt to put a non function value (" + value + ") to " +
+                                   name + " at " + Context.getCurrentSourceNameAndLine());
+           } else if (value != null && !(value instanceof JS.CompiledFunction)) {
+                if (Log.on) Log.log(this, "attempt to put a non-compiled function value (" + value + ") to " +
+                                   name + " at " + Context.getCurrentSourceNameAndLine());
             } else if (name.charAt(1) == '_') {
                 name = name.substring(2).intern();
                 Trap t = Trap.getTrap(this, name);
                 if (t != null) t.delete();
-                if (value != null) Trap.addTrap(this, name, ((Function)value), true, rp);
+                if (value != null) Trap.addTrap(this, name, ((JS.CompiledFunction)value), true, rp);
             } else {
                 name = name.substring(1).intern();
                 Trap t = Trap.getTrap(this, name);
                 if (t != null) t.delete();
-                if (value != null) Trap.addTrap(this, name, ((Function)value), false, rp);
+                if (value != null) Trap.addTrap(this, name, ((JS.CompiledFunction)value), false, rp);
             }
             return;
         }