fix infinite loop
[org.ibex.core.git] / src / org / ibex / core / Ibex.java
index b44ba5a..7b27d14 100644 (file)
@@ -1,4 +1,7 @@
-// Copyright 2004 Adam Megacz, see the COPYING file for licensing [GPL]
+// Copyright 2000-2005 the Contributors, as shown in the revision logs.
+// Licensed under the GNU General Public License version 2 ("the License").
+// You may not use this file except in compliance with the License.
+
 package org.ibex.core;
 
 import java.io.*;
@@ -10,16 +13,17 @@ import org.ibex.net.*;
 import org.ibex.crypto.*;
 
 /** Singleton class that provides all functionality in the ibex.* namespace */
-public final class Ibex extends JS implements JS.Cloneable {
+public final class Ibex extends JS.Obj implements JS.Cloneable {
 
     // FIXME remove this
     private final JS rr;
+    private static final JS.Method METHOD = new JS.Method();
 
-    public Ibex(Stream rr) { try { this.rr = bless(rr);} catch(JSExn e) { throw new Error("should never happen"); } }
+    public Ibex(Stream rr) { try { this.rr = bless(rr);} catch(JSExn e) { throw new Error("should never happen: " + e); } }
 
     public JS resolveString(String str, boolean permitAbsolute) throws JSExn {
         if (str.indexOf("://") != -1) {
-            if (permitAbsolute) return (Stream)url2res(str);
+            if (permitAbsolute) return url2res(str);
             throw new JSExn("absolute URL " + str + " not permitted here");
         }
         // root-relative
@@ -29,35 +33,35 @@ public final class Ibex extends JS implements JS.Cloneable {
         while(str.indexOf('.') != -1) {
             String path = str.substring(0, str.indexOf('.'));
             str = str.substring(str.indexOf('.') + 1);
-            ret = (JS)ret.get(path);
+            ret = ret.get(JSU.S(path));
         }
-        if (!"".equals(str)) ret = (JS)ret.get(str);
+        if (!"".equals(str)) ret = ret.get(JSU.S(str));
         return ret;
     }
 
     /** lets us put multi-level get/put/call keys all in the same method */
-    private class Sub extends JS {
-        String key;
-        Sub(String key) { this.key = key; }
-        public void put(Object key, Object val) throws JSExn { Ibex.this.put(this.key + "." + key, val); }
-        public Object get(Object key) throws JSExn { return Ibex.this.get(this.key + "." + key); }
-        public Object call(Object a0, Object a1, Object a2, Object[] rest, int nargs) throws JSExn {
-            return Ibex.this.callMethod(this.key, a0, a1, a2, rest, nargs);
-        }
-        public Object callMethod(Object method, Object a0, Object a1, Object a2, Object[] rest, int nargs) throws JSExn {
-            return Ibex.this.callMethod(this.key + "." + method, a0, a1, a2, rest, nargs);
+    private class Sub extends JS.Obj {
+        JS key;
+        Sub(JS key) { this.key = key; }
+        public void put(JS key, JS val) throws JSExn { Ibex.this.put(JSU.S(JSU.toString(this.key) + "." + JSU.toString(key)), val); }
+        public JS get(JS key) throws JSExn { return Ibex.this.get(JSU.S(JSU.toString(this.key) + "." + JSU.toString(key))); }
+        public JS call(JS[] args) throws JSExn { return Ibex.this.call(this.key, args); }
+        public JS call(JS method, JS[] args) throws JSExn {
+            return Ibex.this.call(JSU.S(JSU.toString(this.key) + "." + JSU.toString(method)), args);
         }
     }
-    private Cache subCache = new Cache(20);
-    private Sub getSub(String s) {
-        Sub ret = (Sub)subCache.get(s);
-        if (ret == null) subCache.put(s, ret = new Sub(s));
+    private Cache subCache = new Cache(20, true);
+    private Sub getSub(JS key) {
+        Sub ret = (Sub)subCache.get(key);
+        if (ret == null) subCache.put(key, ret = new Sub(key));
         return ret;
     }
 
-    public Object get(Object name) throws JSExn {
-        if (name instanceof String && ((String)name).length() == 0) return rr;
-        //#switch(name)
+    public JS get(JS name) throws JSExn {
+        // FIXME: SHouldn't need this (just trap [""])
+        if (JSU.isString(name) && JSU.toString(name).length() == 0) return rr;
+        // FEATURE: Preprocessor hack to generate specialized JS instances (avoid all this string concatenation)
+        //#switch(JSU.toString(name))
         case "math": return ibexMath;
         case "string": return ibexString;
         case "date": return METHOD;
@@ -65,39 +69,39 @@ public final class Ibex extends JS implements JS.Cloneable {
         case "clone": return METHOD;
         case "bless": return METHOD;
         case "regexp": return METHOD;
-        case "ui": return getSub("ui");
-        case "ui.font": return getSub("ui.font");
+        case "ui": return getSub(name);
+        case "ui.font": return getSub(name);
         case "ui.font.wait": return METHOD;
         case "ui.font.width": return METHOD;
         case "ui.font.height": return METHOD;
-        case "ui.font.sansserif": return Main.builtin.get("fonts/vera/Vera.ttf");
-        case "ui.font.monospace": return Main.builtin.get("fonts/vera/VeraMono.ttf");
-        case "ui.font.serif": return Main.builtin.get("fonts/vera/VeraSe.ttf");
+        case "ui.font.sansserif": return Main.builtin.get(JSU.S("fonts/vera/Vera.ttf"));
+        case "ui.font.monospace": return Main.builtin.get(JSU.S("fonts/vera/VeraMono.ttf"));
+        case "ui.font.serif": return Main.builtin.get(JSU.S("fonts/vera/VeraSe.ttf"));
         case "ui.browser": return METHOD;
-        case "ui.mouse": return getSub("ui.mouse");
+        case "ui.mouse": return getSub(name);
         case "ui.mouse.button":
-            if (Surface.button1 && !Surface.button2 && !Surface.button3) return N(1);
-            else if (!Surface.button1 && Surface.button2 && !Surface.button3) return N(2);
-            else if (!Surface.button1 && !Surface.button2 && Surface.button3) return N(3);
-            else return ZERO;
-        case "ui.key": return getSub("ui.key");
-        case "ui.key.name": return getSub("ui.key.name");
-        case "ui.key.name.alt": return Platform.altKeyName();
-        case "ui.key.alt": return Surface.alt ? T : F;
-        case "ui.key.control": return Surface.control ? T : F;
-        case "ui.key.shift": return Surface.shift ? T : F;
-        case "ui.clipboard": return Platform.getClipBoard();
-        case "ui.maxdim": return N(Short.MAX_VALUE);
-        case "ui.screen": return getSub("ui.screen");
-        case "ui.screen.width": return N(Platform.getScreenWidth());
-        case "ui.screen.height": return N(Platform.getScreenHeight());
-        case "undocumented": return getSub("undocumented");
-        case "undocumented.initialOrigin": return Main.origin;
-        case "undocumented.initialTemplate": return Main.initialTemplate;
-        case "thread": return getSub("thread");
+            if (Surface.button1 && !Surface.button2 && !Surface.button3) return JSU.N(1);
+            else if (!Surface.button1 && Surface.button2 && !Surface.button3) return JSU.N(2);
+            else if (!Surface.button1 && !Surface.button2 && Surface.button3) return JSU.N(3);
+            else return JSU.ZERO;
+        case "ui.key": return getSub(name);
+        case "ui.key.name": return getSub(name);
+        case "ui.key.name.alt": return JSU.S(Platform.altKeyName());
+        case "ui.key.alt": return Surface.alt ? JSU.T : JSU.F;
+        case "ui.key.control": return Surface.control ? JSU.T : JSU.F;
+        case "ui.key.shift": return Surface.shift ? JSU.T : JSU.F;
+        case "ui.clipboard": return JSU.S((String)Platform.getClipBoard());
+        case "ui.maxdim": return JSU.N(Short.MAX_VALUE);
+        case "ui.screen": return getSub(name);
+        case "ui.screen.width": return JSU.N(Platform.getScreenWidth());
+        case "ui.screen.height": return JSU.N(Platform.getScreenHeight());
+        case "undocumented": return getSub(name);
+        case "undocumented.initialOrigin": return JSU.S(Main.origin);
+        case "undocumented.initialTemplate": return JSU.S(Main.initialTemplate);
+        case "thread": return getSub(name);
         case "thread.yield": return METHOD;
         case "thread.sleep": return METHOD;
-        case "stream": return getSub("stream");
+        case "stream": return getSub(name);
         case "stream.homedir": return url2res("file:" + System.getProperty("user.home"));
         case "stream.tempdir": return url2res("file:" + System.getProperty("java.io.tempdir"));
         case "stream.watch": return METHOD;
@@ -108,28 +112,28 @@ public final class Ibex extends JS implements JS.Cloneable {
         case "stream.parse.html": return METHOD;
         case "stream.parse.xml": return METHOD;
         case "stream.parse.utf8": return METHOD;
-        case "net": return getSub("net");
-        case "net.rpc": return getSub("net.rpc");
+        case "net": return getSub(name);
+        case "net.rpc": return getSub(name);
         case "net.rpc.xml": return METHOD;
         case "net.rpc.soap": return METHOD;
-        case "log": return getSub("log");
+        case "log": return getSub(name);
         case "log.debug": return METHOD;
         case "log.info": return METHOD;
         case "log.warn": return METHOD;
         case "log.error": return METHOD;
-        case "crypto": return getSub("crypto");
+        case "crypto": return getSub(name);
         case "crypto.rsa": return METHOD;
         case "crypto.md5": return METHOD;
         case "crypto.sha1": return METHOD;
         case "crypto.rc4": return METHOD;
         //#end
-        return super.get(name);
+        return null;
     }
 
-    public void put(Object name, final Object value) throws JSExn {
-        //#switch(name)
-        case "thread": Scheduler.add((Task)value); return;
-        case "ui.clipboard": Platform.setClipBoard((String)value); return;
+    public void put(JS name, JS value) throws JSExn {
+        //#switch(JSU.toString(name))
+        case "thread": Scheduler.add((Callable)value); return;
+        case "ui.clipboard": Platform.setClipBoard(JSU.toString(value)); return;
         case "ui.frame": Platform.createSurface((Box)value, true, true); return;
         case "ui.window": Platform.createSurface((Box)value, false, true); return;
         case "undocumented.proxyAuthorization":
@@ -140,41 +144,45 @@ public final class Ibex extends JS implements JS.Cloneable {
         throw new JSExn("attempted to put unknown property: ibex."+name);
     }
 
-    public Object callMethod(Object name, Object a, Object b, Object c, Object[] rest, int nargs) throws JSExn {
+    public JS call(JS method, JS[] args) throws JSExn {
         try {
-            //#switch(name)
-            case "date": return new JSDate(a, b, c, rest, nargs);
-            case "net.rpc.soap": return new SOAP((String)a, "", (String)b, (String)c);
+            //#switch(JSU.toString(method))
+            case "date": return new JSDate(args);
+            case "net.rpc.soap": return new SOAP(
+                    args.length < 1 ? null : JSU.toString(args[0]),
+                    "",
+                    args.length < 2 ? null : JSU.toString(args[1]),
+                    args.length < 3 ? null : JSU.toString(args[2]));
                 // FIXME support object dumping
-            case "log.debug":    JS.debug(a== null ? "**null**" : a.toString()); return null;
-            case "log.info":     JS.info(a== null ? "**null**" : a.toString()); return null;
-            case "log.warn":     JS.warn(a== null ? "**null**" : a.toString()); return null;
-            case "log.error":    JS.error(a== null ? "**null**" : a.toString()); return null;
+            case "log.debug":    JSU.debug(args.length < 1 ? "**null**" : JSU.str(args[0])); return null;
+            case "log.info":     JSU.info(args.length < 1 ? "**null**" : JSU.str(args[0])); return null;
+            case "log.warn":     JSU.warn(args.length < 1 ? "**null**" : JSU.str(args[0])); return null;
+            case "log.error":    JSU.error(args.length < 1 ? "**null**" : JSU.str(args[0])); return null;
             //#end
  
-            switch (nargs) {
+            switch (args.length) {
                 case 0:
-                    //#switch(name)
+                    //#switch(JSU.toString(method))
                     case "thread.yield": sleep(0); return null;
                     //#end
                     break;
                 case 1:
-                    //#switch(name)
+                    //#switch(JSU.toString(method))
                     case "clone":
-                        if(a == null) throw new JSExn("can't clone the null value");
-                        return ((JS)a).jsclone();
-                    case "bless": return bless((JS)a);
-                    case "ui.browser": Platform.newBrowserWindow((String)a); return null;
-                    case "stream.unzip": return new Stream.Zip((Stream)a);
-                    case "stream.uncab": return new Stream.Cab((Stream)a);
+                        if(args[0] == null) throw new JSExn("can't clone the null value");
+                        return new JS.Clone((JS)args[0]);
+                    case "bless": return bless((JS)args[0]);
+                    case "ui.browser": Platform.newBrowserWindow(JSU.toString(args[0])); return null;
+                    case "stream.unzip": return args[0] == null ? null : new Stream.Zip((Stream)args[0]);
+                       //case "stream.uncab": return a == null ? null : new Stream.Cab(a);
                     case "stream.cache":
-                        try { return new Stream.CachedStream((Stream)a, "resources", true); }
+                        try { return args[0] == null ? null : new Stream.CachedStream((Stream)args[0], "resources", true); }
                         catch (Stream.NotCacheableException e) { throw new JSExn("this resource cannot be cached"); }
                     case "stream.url": {
-                        String url = (String)a;
+                        String url = JSU.toString(args[0]);
                         if (url.startsWith("http://")) return new Stream.HTTP(url);
                         else if (url.startsWith("https://")) return new Stream.HTTP(url);
-                        else if (url.startsWith("data:")) return new Stream.ByteArray(Base64.decode(url.substring(5)), null);
+                        else if (url.startsWith("data:")) return new Stream.ByteArray(Encode.fromBase64(url.substring(5)), null);
                         else if (url.startsWith("utf8:")) return new Stream.ByteArray(url.substring(5).getBytes(), null);
                         else if (url.startsWith("file:")) {
                             // FIXME
@@ -182,46 +190,47 @@ public final class Ibex extends JS implements JS.Cloneable {
                         }
                         throw new JSExn("invalid resource specifier " + url);
                     }
-                    case "thread.sleep": sleep(JS.toInt(a)); return null;
-                    case "regexp": return new JSRegexp(a, null);
-                    case "net.rpc.xml": return new XMLRPC((String)a, "");
+                    case "thread.sleep": sleep(JSU.toInt(args[0])); return null;
+                    case "regexp": return new JSRegexp(args[0], null);
+                    case "net.rpc.xml": return new XMLRPC(JSU.toString(args[0]), "");
                     case "crypto.rsa": /* FEATURE */ return null;
                     case "crypto.md5": /* FEATURE */ return null;
                     case "crypto.sha1": /* FEATURE */ return null;
                     case "crypto.rc4": /* FEATURE */ return null;
                     case "stream.parse.html": throw new JSExn("not implemented yet"); //return null;
-                    case "stream.parse.xml": new XMLHelper((JS)b).doParse((JS)a); return null;
+                    case "stream.parse.xml": if(args[0] == null) return null; new XMLHelper(args[1]).doParse(args[0]); return null;
                         // FIXME backgrounding
-                    case "stream.parse.utf8": try { return new String(InputStreamToByteArray.convert(Stream.getInputStream(a))); }
+                    case "stream.parse.utf8": if(args[0] == null) return null;
+                                              try { return JSU.S(new String(InputStreamToByteArray.convert(args[0].getInputStream()))); }
                                               catch (Exception e) { Log.warn(this, e); }
                     //#end
                     break;
                 case 2:
-                    //#switch(name)
-                    case "stream.watch": return new Stream.ProgressWatcher((Stream)a, (JS)b);
-                    case "regexp": return new JSRegexp(a, b);
+                    //#switch(JSU.toString(method))
+                    case "stream.watch": return new Stream.ProgressWatcher((Stream)args[0], args[1]);
+                    case "regexp": return new JSRegexp(args[0], args[1]);
                     //#end
                 case 3:
-                    //#switch(name)
-                    case "ui.font.height": return N(Font.getFont((Stream)a, JS.toInt(b)).textheight((String)c));
+                    //#switch(JSU.toString(method))
+                    case "ui.font.height": return JSU.N(Font.getFont(args[0], JSU.toInt(args[1])).textheight(JSU.toString(args[3])));
                     case "ui.font.wait": throw new Error("FIXME: ibex.ui.font.wait not implemented");
-                    case "ui.font.width": return N(Font.getFont((Stream)a, JS.toInt(b)).textwidth((String)c));
+                    case "ui.font.width": return JSU.N(Font.getFont(args[0], JSU.toInt(args[1])).textwidth(JSU.toString(args[3])));
                     //#end
                     break;
             }
         } catch (RuntimeException e) {
             // FIXME: maybe JSExn should take a second argument, Exception
-            Log.warn(this, "ibex."+name+"() threw: " + e);
-            throw new JSExn("invalid argument for ibex object method "+name+"()");
+            Log.warn(this, "ibex."+method+"() threw: " + e);
+            throw new JSExn("invalid argument for ibex object method "+method+"()");
         }
 
-        throw new JSExn("invalid number of arguments ("+nargs+") for ibex object method "+name+"()");
+        throw new JSExn("invalid number of arguments ("+args.length+") for ibex object method "+method+"()");
     }
 
-    public Stream url2res(String url) throws JSExn {
+    public JS url2res(String url) throws JSExn {
         if (url.startsWith("http://")) return new Stream.HTTP(url);
         else if (url.startsWith("https://")) return new Stream.HTTP(url);
-        else if (url.startsWith("data:")) return new Stream.ByteArray(Base64.decode(url.substring(5)), null);
+        else if (url.startsWith("data:")) return new Stream.ByteArray(Encode.fromBase64(url.substring(5)), null);
         else if (url.startsWith("utf8:")) return new Stream.ByteArray(url.substring(5).getBytes(), null);
         else throw new JSExn("invalid resource specifier " + url);
         // FIXME support file:// via dialog boxes
@@ -229,44 +238,44 @@ public final class Ibex extends JS implements JS.Cloneable {
 
     public static void sleep(final int i) throws JSExn {
         try {
-            final JS.UnpauseCallback callback = JS.pause();
+            final Pausable callback = JSU.pause();
             // FEATURE use a single sleeper thread
             new Thread() { public void run() {
                 try { Thread.sleep(i); } catch (InterruptedException e) { }
                 Scheduler.add(callback);
             } }.start();
-        } catch (JS.NotPauseableException npe) {
+        } catch (Pausable.NotPausableException npe) {
             throw new JSExn("you cannot sleep or yield in the foreground thread");
         }
     }
     
-    public static final JSMath ibexMath = new JSMath() {
+    public static final JS ibexMath = new JS.Immutable() {
             // FEATURE: find a cleaner way to do this
-            private JS gs = new JSScope.Global();
-            public Object get(Object key) throws JSExn {
-                //#switch(key)
+            private JS gs = /*new JSScope.Global();*/ null; // FIXME: Global scope
+            public JS get(JS key) throws JSExn {
+                //#switch(JSU.toString(key))
                 case "isNaN": return METHOD;
                 case "isFinite": return METHOD;
                 case "NaN": return METHOD;
                 case "Infinity": return METHOD;
                 //#end
-                return super.get(key);
+                return JSU.MATH.get(key);
             }
-            public Object callMethod(Object name, Object a, Object b, Object c, Object[] rest, int nargs) throws JSExn {
-                //#switch(name)
-                case "isNaN": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "isFinite": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "NaN": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "Infinity": return gs.callMethod(name,a,b,c,rest,nargs);
+            public JS call(JS method, JS[] args) throws JSExn {
+                //#switch(JSU.toString(method))
+                case "isNaN": return gs.call(method, args);
+                case "isFinite": return gs.call(method, args);
+                case "NaN": return gs.call(method, args);
+                case "Infinity": return gs.call(method, args);
                 //#end
-                return super.callMethod(name,a,b,c,rest,nargs);
+                return JSU.MATH.call(method, args);
             }
     };
 
-    public static final JS ibexString = new JS() {
-            private JS gs = new JSScope.Global();
-            public Object get(Object key) throws JSExn {
-                //#switch(key)
+    public static final JS ibexString = new JS.Obj() {
+            private JS gs = /*new JSScope.Global();*/ null; // FIXME: Global scope
+            public JS get(JS key) throws JSExn {
+                //#switch(JSU.toString(key))
                 case "parseInt": return METHOD;
                 case "parseFloat": return METHOD;
                 case "decodeURI": return METHOD;
@@ -279,19 +288,19 @@ public final class Ibex extends JS implements JS.Cloneable {
                 //#end
                 return super.get(key);
             }
-            public Object callMethod(Object name, Object a, Object b, Object c, Object[] rest, int nargs) throws JSExn {
-                //#switch(name)
-                case "parseInt": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "parseFloat": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "decodeURI": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "decodeURIComponent": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "encodeURI": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "encodeURIComponent": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "escape": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "unescape": return gs.callMethod(name,a,b,c,rest,nargs);
-                case "fromCharCode": return gs.callMethod(name,a,b,c,rest,nargs);
+            public JS callMethod(JS method, JS[] args) throws JSExn {
+                //#switch(JSU.toString(method))
+                case "parseInt": return gs.call(method, args);
+                case "parseFloat": return gs.call(method, args);
+                case "decodeURI": return gs.call(method, args);
+                case "decodeURIComponent": return gs.call(method, args);
+                case "encodeURI": return gs.call(method, args);
+                case "encodeURIComponent": return gs.call(method, args);
+                case "escape": return gs.call(method, args);
+                case "unescape": return gs.call(method, args);
+                case "fromCharCode": return gs.call(method, args);
                 //#end
-                return super.callMethod(name,a,b,c,rest,nargs);
+                return super.call(method, args);
             }
     };
 
@@ -299,35 +308,45 @@ public final class Ibex extends JS implements JS.Cloneable {
         private class Wrapper extends XML.Exn { public JSExn wrapee; public Wrapper(JSExn jse) { super(""); wrapee = jse; } }
         private JS characters, whitespace, endElement, startElement;
         public XMLHelper(JS b) throws JSExn {
-            super(BUFFER_SIZE);
-            startElement = (JS)b.getAndTriggerTraps("startElement");
-            endElement   = (JS)b.getAndTriggerTraps("endElement");
-            characters   = (JS)b.getAndTriggerTraps("characters");
-            whitespace   = (JS)b.getAndTriggerTraps("whitespace");
+            super(BUFFER_SIZE, true);
+            startElement = b.getAndTriggerTraps(JSU.S("startElement"));
+            endElement   = b.getAndTriggerTraps(JSU.S("endElement"));
+            characters   = b.getAndTriggerTraps(JSU.S("characters"));
+            whitespace   = b.getAndTriggerTraps(JSU.S("whitespace"));
         }
 
-        public void startElement(XML.Element c) throws XML.Exn { try {
-                JS attrs = new JS.O();
+        private final JS[] callargs1= new JS[1], callargs2= new JS[2], callargs3= new JS[3];
+        public void startElement(Tree.Element c) throws XML.Exn { try {
+                Tree.Attributes a = c.getAttributes();
+                JS attrs = new JS.Obj();
                 // FIXME attribute URIs? add an additional hash?
-                for(int i=0; i<c.getAttrLen(); i++) attrs.put(c.getAttrKey(i), c.getAttrVal(i));
-                startElement.call(c.getLocalName(), attrs, c.getUri(), null, 3);
-        } catch (JSExn jse) { throw new Wrapper(jse); } }
+                for(int i=0; i<a.attrSize(); i++) attrs.put(JSU.S(a.getKey(i)), JSU.S(a.getVal(i)));
+                callargs3[0] = JSU.S(c.getLocalName());
+                callargs3[1] = attrs;
+                callargs3[2] = JSU.S(c.getUri());
+                startElement.call(callargs3);
+        } catch (JSExn jse) { throw new Wrapper(jse);
+        } finally { callargs3[0] = callargs3[1] = callargs3[2] = null; } }
 
-        public void endElement(XML.Element c) throws XML.Exn { try {
-                endElement.call(c.getLocalName(), c.getUri(), null, null, 2);
-        } catch (JSExn jse) { throw new Wrapper(jse); } }
+        public void endElement(Tree.Element c) throws XML.Exn { try {
+                callargs2[0] = JSU.S(c.getLocalName());
+                callargs2[1] = JSU.S(c.getUri());
+                endElement.call(callargs2);
+        } catch (JSExn jse) { throw new Wrapper(jse); } finally { callargs2[0] = callargs2[1] = null; } }
 
         public void characters(char[] ch, int start, int length) throws XML.Exn { try {
-                characters.call(new String(ch, start, length), null, null, null, 1);
-        } catch (JSExn jse) { throw new Wrapper(jse); } }
+                callargs1[0] = JSU.S(new String(ch, start, length));
+                characters.call(callargs1);
+        } catch (JSExn jse) { throw new Wrapper(jse); } finally { callargs1[0] = null; } }
 
         public void whitespace(char[] ch, int start, int length) throws XML.Exn { try {
-                whitespace.call(new String(ch, start, length), null, null, null, 1);
-        } catch (JSExn jse) { throw new Wrapper(jse); } }
+                callargs1[0] = JSU.S(new String(ch, start, length));
+                whitespace.call(callargs1);
+        } catch (JSExn jse) { throw new Wrapper(jse); } finally { callargs1[0] = null; } }
 
         public void doParse(JS s) throws JSExn {
             try { 
-                parse(new BufferedReader(new InputStreamReader(Stream.getInputStream(s))));
+                parse(new BufferedReader(new InputStreamReader(s.getInputStream())));
             } catch (Wrapper e) {
                 throw e.wrapee;
             } catch (XML.Exn e) {
@@ -342,63 +361,77 @@ public final class Ibex extends JS implements JS.Cloneable {
 
     // FEATURE: move this into builtin.xwar
     public Blessing bless(JS b) throws JSExn { return new Ibex.Blessing(b, this, null, null); }
-    // FIXME: Does this really need to extends JS.Clone?
-    public static class Blessing extends JS.Clone {
+    // JS:FIXME: This doesn't properly handle traps
+    public static class Blessing extends JS.Obj {
         private Ibex ibex;
         private Template t = null;
-        public Object parentkey = null;
+        public JS parentkey = null;
         public Blessing parent = null;
-        private Hash cache = new Hash();
-        public Blessing(JS clonee, Ibex ibex, Blessing parent, Object parentkey) throws JSExn {
-            super(clonee); this.ibex = ibex; this.parentkey = parentkey; this.parent = parent; }
-        public Object get(Object key) throws JSExn {
-            if (key.equals("")) return ((Object)getStatic());
-            if (cache.get(key) != null) return cache.get(key);
-            Object ret = new Blessing((JS)clonee.get(key), ibex, this, key);
+        public JS clonee;
+        private Basket.Map cache = new Basket.HashMap(); 
+        public Blessing(JS clonee, Ibex ibex, Blessing parent, JS parentkey) throws JSExn {
+            this.clonee = clonee; this.ibex = ibex; this.parentkey = parentkey; this.parent = parent; }
+        public JS get(JS key) throws JSExn {
+            if (JSU.isString(key) && JSU.toString(key).equals("")) return getStatic();
+            if (cache.get(key) != null) return (JS)cache.get(key);
+            JS ret = new Blessing(clonee.get(key), ibex, this, key);
             cache.put(key, ret);
             return ret;
         }
-        public static Blessing getBlessing(Object o) {
-            if (!(o instanceof JS)) return null;
-            JS js = (JS)o;
-            while (js instanceof JS.Clone && !(js instanceof Blessing)) js = ((JS.Clone)js).getClonee();
+        public static Blessing getBlessing(JS js) {
+            // CHECKME: is unclone() good enough or do we need getClonee() in Cloneable?
+            while (js instanceof JS.Clone && !(js instanceof Blessing)) js = js.unclone();
             if (!(js instanceof Blessing)) return null;
             return (Blessing)js;
         }
+        // FEATURE: This is a gross hack
         public InputStream getImage() throws JSExn {
             try {
-                InputStream in = Stream.getInputStream(this);
+                InputStream in = getInputStream();
                 if (in != null) return in;
             } catch (IOException e) { /* DELIBERATE */ }
             String[] exts = new String[] { ".png", ".jpeg", ".gif" };
             for (int i=0; i < exts.length; i++)
                 try {
-                    InputStream in = Stream.getInputStream(parent.get(parentkey + exts[i]));
+                    InputStream in = parent.get(JSU.S(JSU.toString(parentkey) + exts[i])).getInputStream();
                     if (in != null) return in;
                 } catch (IOException f) { /* DELIBERATE */ }
             return null;
         }
-        public JSScope getStatic() {
+        public JS getStatic() throws JSExn {
             try {
                 if (t == null) {
-                    JS res = (JS) parent.get(parentkey + ".t");
-                    t = Template.buildTemplate(res.unclone().toString(), res, ibex);
+                    // FEATURE: Might want to handle the ".t" part better
+                    JS res = parent.get(JSU.S(JSU.toString(parentkey) + ".t"));
+                    t = Template.buildTemplate(description(), res, ibex);
                 }
-                return t.staticScope;
+                return t != null ? t.staticObject : null;
             } catch (Exception e) {
                 Log.error(this, e);
                 return null;
             }
         }
-        public Object call(Object a, Object b, Object c, Object[] rest, int nargs) throws JSExn {
-            // GROSS hack
-            if (nargs != 1 && nargs != 9999) throw new JSExn("FIXME can only call with one arg");
+        private String description() {
+            String s = JSU.str(parentkey);
+            for(Blessing b = parent; b.parentkey != null; b = b.parent) s = JSU.str(b.parentkey) + "." + s;
+            return s;
+        }
+        public JS call(JS[] args) throws JSExn {
+            if (args.length != 1) throw new JSExn("can only call a template with one arg");
+            getStatic();
+            if (t == null) throw new JSExn("No such template " + JSU.str(parentkey));
+            if(!(args[0] instanceof Box)) throw new JSExn("can only apply templates to boxes");
+            t.apply((Box)args[0]);
+            return args[0];
+        }
+        // FEATURE: This is a gross hack
+        Template getTemplate()  throws JSExn {
             getStatic();
-            if (t == null) throw new JSExn("No such template " + parentkey);
-            if (nargs == 9999) return t;
-            t.apply((Box)a);
-            return a;
+            if (t == null) throw new JSExn("No such template " + JSU.str(parentkey));
+            return t;
         }
+        // JS:FIXME: Blessing shouldn't need to roll its own JS.Clone implementation
+        public InputStream getInputStream() throws JSExn, IOException { return clonee.getInputStream(); }
     }
 
 }