2004/01/03 04:27:27
[org.ibex.core.git] / src / org / xwt / js / Parser.java
index 9183c94..b67b59c 100644 (file)
@@ -5,7 +5,7 @@ import org.xwt.util.*;
 import java.io.*;
 
 /**
- *  Parses a stream of lexed tokens into a tree of CompiledFunctionImpl's.
+ *  Parses a stream of lexed tokens into a tree of JSFunction's.
  *
  *  There are three kinds of things we parse: blocks, statements, and
  *  expressions.
@@ -73,7 +73,7 @@ class Parser extends Lexer implements ByteCodes {
 
     /** for debugging */
     public static void main(String[] s) throws Exception {
-        CompiledFunctionImpl block = new JS.CompiledFunction("stdin", 0, new InputStreamReader(System.in), null);
+        JSFunction block = JSFunction.fromReader("stdin", 0, new InputStreamReader(System.in));
         if (block == null) return;
         System.out.println(block);
     }
@@ -83,6 +83,8 @@ class Parser extends Lexer implements ByteCodes {
 
     static byte[] precedence = new byte[MAX_TOKEN + 1];
     static boolean[] isRightAssociative = new boolean[MAX_TOKEN + 1];
+    // Use this as the precedence when we want anything up to the comma
+    private final static int NO_COMMA = 2;
     static {
         isRightAssociative[ASSIGN] =
             isRightAssociative[ASSIGN_BITOR] =
@@ -97,6 +99,8 @@ class Parser extends Lexer implements ByteCodes {
             isRightAssociative[ASSIGN_DIV] =
             isRightAssociative[ASSIGN_MOD] = true;
 
+        precedence[COMMA] = 1;
+        // 2 is intentionally left unassigned. we use minPrecedence==2 for comma separated lists
         precedence[ASSIGN] =
             precedence[ASSIGN_BITOR] =
             precedence[ASSIGN_BITXOR] =
@@ -108,24 +112,20 @@ class Parser extends Lexer implements ByteCodes {
             precedence[ASSIGN_SUB] =
             precedence[ASSIGN_MUL] =
             precedence[ASSIGN_DIV] =
-            precedence[ASSIGN_MOD] = 1;
-        precedence[HOOK] = 2;
-        precedence[COMMA] = 3;
-        precedence[OR] = precedence[AND] = precedence[BANG] = 4;
-        precedence[GT] = precedence[GE] = 5;
-        precedence[BITOR] = 6;
-        precedence[BITXOR] = 7;
-        precedence[BITAND] = 8;
-        precedence[EQ] = precedence[NE] = 9;
-        precedence[LT] = precedence[LE] = precedence[TYPEOF] = 10;
-        precedence[SHEQ] = precedence[SHNE] = 11;
+            precedence[ASSIGN_MOD] = 3;
+        precedence[HOOK] = 4;
+        precedence[OR] = 5;
+        precedence[AND] = 6;
+        precedence[BITOR] = 7;
+        precedence[BITXOR] = 8;
+        precedence[BITAND] = 9;
+        precedence[EQ] = precedence[NE] = precedence[SHEQ] = precedence[SHNE] = 10;
+        precedence[LT] = precedence[LE] = precedence[GT] = precedence[GE] = 11;
         precedence[LSH] = precedence[RSH] = precedence[URSH] = 12;
-        precedence[ADD] = precedence[SUB] = 13;
-        precedence[MUL] = precedence[DIV] = precedence[MOD] = 14;
-        precedence[BITNOT] =  15;
-        precedence[INC] = precedence[DEC] = 16;
-        precedence[LP] = 17;
-        precedence[DOT] = precedence[LB] = 18;
+        precedence[ADD] = precedence[SUB] = 12;
+        precedence[MUL] = precedence[DIV] = precedence[MOD] = 13;
+        precedence[BITNOT] =  precedence[BANG] = precedence[TYPEOF] = 14;
+        precedence[DOT] = precedence[LB] = precedence[LP] =  precedence[INC] = precedence[DEC] = 15;
     }
 
 
@@ -133,7 +133,16 @@ class Parser extends Lexer implements ByteCodes {
 
     /** gets a token and throws an exception if it is not <tt>code</tt> */
     private void consume(int code) throws IOException {
-        if (getToken() != code) throw pe("expected " + codeToString[code] + ", got " + (op == -1 ? "EOF" : codeToString[op]));
+        if (getToken() != code) {
+            if(code == NAME) switch(op) {
+                case RETURN: case TYPEOF: case BREAK: case CONTINUE: case TRY: case THROW:
+                case ASSERT: case NULL: case TRUE: case FALSE: case IN: case IF: case ELSE:
+                case SWITCH: case CASE: case DEFAULT: case WHILE: case VAR: case WITH:
+                case CATCH: case FINALLY:
+                    throw pe("Bad variable name; '" + codeToString[op].toLowerCase() + "' is a javascript keyword");
+            }
+            throw pe("expected " + codeToString[code] + ", got " + (op == -1 ? "EOF" : codeToString[op]));
+        }
     }
 
     /**
@@ -142,14 +151,14 @@ class Parser extends Lexer implements ByteCodes {
      *  bytecodes for that expression to <tt>appendTo</tt>; the
      *  appended bytecodes MUST grow the stack by exactly one element.
      */ 
-    private void startExpr(CompiledFunctionImpl appendTo, int minPrecedence) throws IOException {
+    private void startExpr(JSFunction appendTo, int minPrecedence) throws IOException {
         int saveParserLine = parserLine;
         _startExpr(appendTo, minPrecedence);
         parserLine = saveParserLine;
     }
-    private void _startExpr(CompiledFunctionImpl appendTo, int minPrecedence) throws IOException {
+    private void _startExpr(JSFunction appendTo, int minPrecedence) throws IOException {
         int tok = getToken();
-        CompiledFunctionImpl b = appendTo;
+        JSFunction b = appendTo;
 
         switch (tok) {
         case -1: throw pe("expected expression");
@@ -157,34 +166,45 @@ class Parser extends Lexer implements ByteCodes {
         // all of these simply push values onto the stack
         case NUMBER: b.add(parserLine, LITERAL, number); break;
         case STRING: b.add(parserLine, LITERAL, string); break;
-        case THIS: b.add(parserLine, TOPSCOPE, null); break;
         case NULL: b.add(parserLine, LITERAL, null); break;
-        case TRUE: case FALSE: b.add(parserLine, LITERAL, new Boolean(tok == TRUE)); break;
+        case TRUE: case FALSE: b.add(parserLine, LITERAL, JS.B(tok == TRUE)); break;
+
+        // (.foo) syntax
+        case DOT: {
+            consume(NAME);
+            b.add(parserLine, TOPSCOPE);
+            b.add(parserLine, LITERAL, "");
+            b.add(parserLine, GET);
+            b.add(parserLine, LITERAL, string);
+            b.add(parserLine, GET);
+            continueExpr(b, minPrecedence);
+            break;
+        }
 
         case LB: {
-            b.add(parserLine, ARRAY, new Integer(0));                       // push an array onto the stack
-            int size0 = b.size();
+            b.add(parserLine, ARRAY, JS.ZERO);                       // push an array onto the stack
+            int size0 = b.size;
             int i = 0;
             if (peekToken() != RB)
                 while(true) {                                               // iterate over the initialization values
-                    int size = b.size();
-                    b.add(parserLine, LITERAL, new Integer(i++));           // push the index in the array to place it into
+                    int size = b.size;
+                    b.add(parserLine, LITERAL, JS.N(i++));           // push the index in the array to place it into
                     if (peekToken() == COMMA || peekToken() == RB)
                         b.add(parserLine, LITERAL, null);                   // for stuff like [1,,2,]
                     else
-                        startExpr(b, -1);                                   // push the value onto the stack
+                        startExpr(b, NO_COMMA);                             // push the value onto the stack
                     b.add(parserLine, PUT);                                 // put it into the array
                     b.add(parserLine, POP);                                 // discard the value remaining on the stack
                     if (peekToken() == RB) break;
                     consume(COMMA);
                 }
-            b.set(size0 - 1, new Integer(i));                               // back at the ARRAY instruction, write the size of the array
+            b.set(size0 - 1, JS.N(i));                               // back at the ARRAY instruction, write the size of the array
             consume(RB);
             break;
         }
         case SUB: {  // negative literal (like "3 * -1")
             consume(NUMBER);
-            b.add(parserLine, LITERAL, new Double(number.doubleValue() * -1));
+            b.add(parserLine, LITERAL, JS.N(number.doubleValue() * -1));
             break;
         }
         case LP: {  // grouping (not calling)
@@ -194,14 +214,19 @@ class Parser extends Lexer implements ByteCodes {
         }
         case INC: case DEC: {  // prefix (not postfix)
             startExpr(b, precedence[tok]);
-            int prev = b.size() - 1;
+            int prev = b.size - 1;
             if (b.get(prev) == GET && b.getArg(prev) != null)
                 b.set(prev, LITERAL, b.getArg(prev));
             else if(b.get(prev) == GET)
                 b.pop();
             else
                 throw pe("prefixed increment/decrement can only be performed on a valid assignment target");
-            b.add(parserLine, tok, Boolean.TRUE);
+            b.add(parserLine, GET_PRESERVE, Boolean.TRUE);
+            b.add(parserLine, LITERAL, JS.N(1));
+            b.add(parserLine, tok == INC ? ADD : SUB, JS.N(2));
+            b.add(parserLine, PUT, null);
+            b.add(parserLine, SWAP, null);
+            b.add(parserLine, POP, null);
             break;
         }
         case BANG: case BITNOT: case TYPEOF: {
@@ -218,7 +243,7 @@ class Parser extends Lexer implements ByteCodes {
                     getToken();
                     b.add(parserLine, LITERAL, string);                          // grab the key
                     consume(COLON);
-                    startExpr(b, -1);                                            // grab the value
+                    startExpr(b, NO_COMMA);                                      // grab the value
                     b.add(parserLine, PUT);                                      // put the value into the object
                     b.add(parserLine, POP);                                      // discard the remaining value
                     if (peekToken() == RC) break;
@@ -231,13 +256,13 @@ class Parser extends Lexer implements ByteCodes {
         case NAME: {
             b.add(parserLine, TOPSCOPE);
             b.add(parserLine, LITERAL, string);
-            continueExprAfterAssignable(b);
+            continueExprAfterAssignable(b,minPrecedence);
             break;
         }
         case FUNCTION: {
             consume(LP);
             int numArgs = 0;
-            CompiledFunctionImpl b2 = new JS.CompiledFunction(sourceName, parserLine, null, null);
+            JSFunction b2 = new JSFunction(sourceName, parserLine, null);
             b.add(parserLine, NEWFUNCTION, b2);
 
             // function prelude; arguments array is already on the stack
@@ -248,12 +273,13 @@ class Parser extends Lexer implements ByteCodes {
             b2.add(parserLine, PUT);
 
             while(peekToken() != RP) {                                    // run through the list of argument names
+                numArgs++;
                 if (peekToken() == NAME) {
                     consume(NAME);                                        // a named argument
                     String varName = string;
                     
                     b2.add(parserLine, DUP);                              // dup the args array 
-                    b2.add(parserLine, GET, new Integer(numArgs));   // retrieve it from the arguments array
+                    b2.add(parserLine, GET, JS.N(numArgs - 1));   // retrieve it from the arguments array
                     b2.add(parserLine, TOPSCOPE);
                     b2.add(parserLine, SWAP);
                     b2.add(parserLine, DECLARE, varName);                  // declare the name
@@ -264,22 +290,20 @@ class Parser extends Lexer implements ByteCodes {
                 }
                 if (peekToken() == RP) break;
                 consume(COMMA);
-                numArgs++;
             }
             consume(RP);
 
+            b2.numFormalArgs = numArgs;
             b2.add(parserLine, POP);                                      // pop off the arguments array
             b2.add(parserLine, POP);                                      // pop off TOPSCOPE
             
            if(peekToken() != LC)
-                throw pe("Functions must have a block surrounded by curly brackets");
+                throw pe("JSFunctions must have a block surrounded by curly brackets");
                 
             parseBlock(b2, null);                                   // the function body
 
-            if(b2.get(b2.size()-1) != RETURN) {
-                b2.add(parserLine, LITERAL, null);                        // in case we "fall out the bottom", return NULL
-                b2.add(parserLine, RETURN);
-            }
+            b2.add(parserLine, LITERAL, null);                        // in case we "fall out the bottom", return NULL
+            b2.add(parserLine, RETURN);
 
             break;
         }
@@ -298,52 +322,66 @@ class Parser extends Lexer implements ByteCodes {
      *  expression that modifies the assignable.  This method always
      *  decreases the stack depth by exactly one element.
      */
-    private void continueExprAfterAssignable(CompiledFunctionImpl b) throws IOException {
+    private void continueExprAfterAssignable(JSFunction b,int minPrecedence) throws IOException {
         int saveParserLine = parserLine;
-        _continueExprAfterAssignable(b);
+        _continueExprAfterAssignable(b,minPrecedence);
         parserLine = saveParserLine;
     }
-    private void _continueExprAfterAssignable(CompiledFunctionImpl b) throws IOException {
+    private void _continueExprAfterAssignable(JSFunction b,int minPrecedence) throws IOException {
         if (b == null) throw new Error("got null b; this should never happen");
         int tok = getToken();
+        if (minPrecedence != -1 && (precedence[tok] < minPrecedence || (precedence[tok] == minPrecedence && !isRightAssociative[tok])))
+            // force the default case
+            tok = -1;
         switch(tok) {
         case ASSIGN_BITOR: case ASSIGN_BITXOR: case ASSIGN_BITAND: case ASSIGN_LSH: case ASSIGN_RSH: case ASSIGN_URSH:
-        case ASSIGN_ADD: case ASSIGN_SUB: case ASSIGN_MUL: case ASSIGN_DIV: case ASSIGN_MOD: {
+        case ASSIGN_MUL: case ASSIGN_DIV: case ASSIGN_MOD: case ASSIGN_ADD: case ASSIGN_SUB: {
             b.add(parserLine, GET_PRESERVE);
-            startExpr(b, -1);
-            b.add(parserLine, tok - 1);
+            startExpr(b,  precedence[tok]);
+            int size = b.size;
+            if (tok == ASSIGN_ADD || tok == ASSIGN_SUB) {
+                b.add(parserLine, tok);
+            }
+            // tok-1 is always s/^ASSIGN_// (0 is BITOR, 1 is ASSIGN_BITOR, etc) 
+            b.add(parserLine, tok - 1, tok-1==ADD ? JS.N(2) : null);
             b.add(parserLine, PUT);
             b.add(parserLine, SWAP);
             b.add(parserLine, POP);
+            if (tok == ASSIGN_ADD || tok == ASSIGN_SUB) b.set(size, tok, JS.N(b.size - size));
             break;
         }
         case INC: case DEC: { // postfix
-            b.add(parserLine, GET_PRESERVE);
-            b.add(parserLine, LITERAL, new Integer(1));
-            b.add(parserLine, tok == INC ? ADD : SUB);
-            b.add(parserLine, PUT);
-            b.add(parserLine, SWAP);
-            b.add(parserLine, POP);
-            b.add(parserLine, LITERAL, new Integer(1));
-            b.add(parserLine, tok == INC ? SUB : ADD);
+            b.add(parserLine, GET_PRESERVE, Boolean.TRUE);
+            b.add(parserLine, LITERAL, JS.N(1));
+            b.add(parserLine, tok == INC ? ADD : SUB, JS.N(2));
+            b.add(parserLine, PUT, null);
+            b.add(parserLine, SWAP, null);
+            b.add(parserLine, POP, null);
+            b.add(parserLine, LITERAL, JS.N(1));
+            b.add(parserLine, tok == INC ? SUB : ADD, null);   // undo what we just did, since this is postfix
             break;
         }
         case ASSIGN: {
-            startExpr(b, -1);
+            startExpr(b, precedence[tok]);
             b.add(parserLine, PUT);
             b.add(parserLine, SWAP);
             b.add(parserLine, POP);
             break;
         }
         case LP: {
-            int n = parseArgs(b);
-            b.add(parserLine,CALLMETHOD,new Integer(n));
+
+            // Method calls are implemented by doing a GET_PRESERVE
+            // first.  If the object supports method calls, it will
+            // return JS.METHOD
+            int n = parseArgs(b, 2);
+            b.add(parserLine, GET_PRESERVE);
+            b.add(parserLine, CALLMETHOD, JS.N(n));
             break;
         }
         default: {
             pushBackToken();
-            if(b.get(b.size()-1) == LITERAL && b.getArg(b.size()-1) != null)
-                b.set(b.size()-1,GET,b.getArg(b.size()-1));
+            if(b.get(b.size-1) == LITERAL && b.getArg(b.size-1) != null)
+                b.set(b.size-1,GET,b.getArg(b.size-1));
             else
                 b.add(parserLine, GET);
             return;
@@ -363,12 +401,12 @@ class Parser extends Lexer implements ByteCodes {
      *  If any bytecodes are appended, they will not alter the stack
      *  depth.
      */
-    private void continueExpr(CompiledFunctionImpl b, int minPrecedence) throws IOException {
+    private void continueExpr(JSFunction b, int minPrecedence) throws IOException {
         int saveParserLine = parserLine;
         _continueExpr(b, minPrecedence);
         parserLine = saveParserLine;
     }
-    private void _continueExpr(CompiledFunctionImpl b, int minPrecedence) throws IOException {
+    private void _continueExpr(JSFunction b, int minPrecedence) throws IOException {
         if (b == null) throw new Error("got null b; this should never happen");
         int tok = getToken();
         if (tok == -1) return;
@@ -379,49 +417,72 @@ class Parser extends Lexer implements ByteCodes {
 
         switch (tok) {
         case LP: {  // invocation (not grouping)
-            int n = parseArgs(b);
-            b.add(parserLine, CALL, new Integer(n));
+            int n = parseArgs(b, 1);
+            b.add(parserLine, CALL, JS.N(n));
             break;
         }
         case BITOR: case BITXOR: case BITAND: case SHEQ: case SHNE: case LSH:
-        case RSH: case URSH: case ADD: case MUL: case DIV: case MOD:
+        case RSH: case URSH: case MUL: case DIV: case MOD:
         case GT: case GE: case EQ: case NE: case LT: case LE: case SUB: {
             startExpr(b, precedence[tok]);
             b.add(parserLine, tok);
             break;
         }
+        case ADD: {
+            int count=1;
+            int nextTok;
+            do {
+                startExpr(b,precedence[tok]);
+                count++;
+                nextTok = getToken();
+            } while(nextTok == tok);
+            pushBackToken();
+            b.add(parserLine, tok, JS.N(count));
+            break;
+        }
         case OR: case AND: {
-            b.add(parserLine, tok == AND ? b.JF : b.JT, new Integer(0));       // test to see if we can short-circuit
-            int size = b.size();
+            b.add(parserLine, tok == AND ? b.JF : b.JT, JS.ZERO);       // test to see if we can short-circuit
+            int size = b.size;
             startExpr(b, precedence[tok]);                                     // otherwise check the second value
-            b.add(parserLine, JMP, new Integer(2));                            // leave the second value on the stack and jump to the end
+            b.add(parserLine, JMP, JS.N(2));                            // leave the second value on the stack and jump to the end
             b.add(parserLine, LITERAL, tok == AND ?
-                  new Boolean(false) : new Boolean(true));                     // target of the short-circuit jump is here
-            b.set(size - 1, new Integer(b.size() - size));                     // write the target of the short-circuit jump
+                  JS.B(false) : JS.B(true));                     // target of the short-circuit jump is here
+            b.set(size - 1, JS.N(b.size - size));                     // write the target of the short-circuit jump
             break;
         }
         case DOT: {
-            consume(NAME);
+            // support foo..bar syntax for foo[""].bar
+            if (peekToken() == DOT) {
+                string = "";
+            } else {
+                consume(NAME);
+            }
             b.add(parserLine, LITERAL, string);
-            continueExprAfterAssignable(b);
+            continueExprAfterAssignable(b,minPrecedence);
             break;
         }
         case LB: { // subscripting (not array constructor)
             startExpr(b, -1);
             consume(RB);
-            continueExprAfterAssignable(b);
+            continueExprAfterAssignable(b,minPrecedence);
             break;
         }
         case HOOK: {
-            b.add(parserLine, JF, new Integer(0));                // jump to the if-false expression
-            int size = b.size();
-            startExpr(b, -1);                                     // write the if-true expression
-            b.add(parserLine, JMP, new Integer(0));               // if true, jump *over* the if-false expression     
-            b.set(size - 1, new Integer(b.size() - size + 1));    // now we know where the target of the jump is
+            b.add(parserLine, JF, JS.ZERO);                // jump to the if-false expression
+            int size = b.size;
+            startExpr(b, minPrecedence);                          // write the if-true expression
+            b.add(parserLine, JMP, JS.ZERO);               // if true, jump *over* the if-false expression     
+            b.set(size - 1, JS.N(b.size - size + 1));    // now we know where the target of the jump is
             consume(COLON);
-            size = b.size();
-            startExpr(b, -1);                                     // write the if-false expression
-            b.set(size - 1, new Integer(b.size() - size + 1));    // this is the end; jump to here
+            size = b.size;
+            startExpr(b, minPrecedence);                          // write the if-false expression
+            b.set(size - 1, JS.N(b.size - size + 1));    // this is the end; jump to here
+            break;
+        }
+        case COMMA: {
+            // pop the result of the previous expression, it is ignored
+            b.add(parserLine,POP);
+            startExpr(b,-1);
             break;
         }
         default: {
@@ -434,12 +495,14 @@ class Parser extends Lexer implements ByteCodes {
     }
     
     // parse a set of comma separated function arguments, assume LP has already been consumed
-    private int parseArgs(CompiledFunctionImpl b) throws IOException {
+    // if swap is true, (because the function is already on the stack) we will SWAP after each argument to keep it on top
+    private int parseArgs(JSFunction b, int pushdown) throws IOException {
         int i = 0;
         while(peekToken() != RP) {
             i++;
             if (peekToken() != COMMA) {
-                startExpr(b, -1);
+                startExpr(b, NO_COMMA);
+                b.add(parserLine, SWAP, JS.N(pushdown));
                 if (peekToken() == RP) break;
             }
             consume(COMMA);
@@ -449,13 +512,13 @@ class Parser extends Lexer implements ByteCodes {
     }
     
     /** Parse a block of statements which must be surrounded by LC..RC. */
-    void parseBlock(CompiledFunctionImpl b) throws IOException { parseBlock(b, null); }
-    void parseBlock(CompiledFunctionImpl b, String label) throws IOException {
+    void parseBlock(JSFunction b) throws IOException { parseBlock(b, null); }
+    void parseBlock(JSFunction b, String label) throws IOException {
         int saveParserLine = parserLine;
         _parseBlock(b, label);
         parserLine = saveParserLine;
     }
-    void _parseBlock(CompiledFunctionImpl b, String label) throws IOException {
+    void _parseBlock(JSFunction b, String label) throws IOException {
         if (peekToken() == -1) return;
         else if (peekToken() != LC) parseStatement(b, null);
         else {
@@ -466,12 +529,12 @@ class Parser extends Lexer implements ByteCodes {
     }
 
     /** Parse a single statement, consuming the RC or SEMI which terminates it. */
-    void parseStatement(CompiledFunctionImpl b, String label) throws IOException {
+    void parseStatement(JSFunction b, String label) throws IOException {
         int saveParserLine = parserLine;
         _parseStatement(b, label);
         parserLine = saveParserLine;
     }
-    void _parseStatement(CompiledFunctionImpl b, String label) throws IOException {
+    void _parseStatement(JSFunction b, String label) throws IOException {
         int tok = peekToken();
         if (tok == -1) return;
         switch(tok = getToken()) {
@@ -498,7 +561,7 @@ class Parser extends Lexer implements ByteCodes {
                 b.add(parserLine, DECLARE, string);               // declare it
                 if (peekToken() == ASSIGN) {                     // if there is an '=' after the variable name
                     consume(ASSIGN);
-                    startExpr(b, -1);
+                    startExpr(b, NO_COMMA);
                     b.add(parserLine, PUT);                      // assign it
                     b.add(parserLine, POP);                      // clean the stack
                 } else {
@@ -516,40 +579,40 @@ class Parser extends Lexer implements ByteCodes {
             startExpr(b, -1);
             consume(RP);
             
-            b.add(parserLine, JF, new Integer(0));                    // if false, jump to the else-block
-            int size = b.size();
+            b.add(parserLine, JF, JS.ZERO);                    // if false, jump to the else-block
+            int size = b.size;
             parseStatement(b, null);
             
             if (peekToken() == ELSE) {
                 consume(ELSE);
-                b.add(parserLine, JMP, new Integer(0));               // if we took the true-block, jump over the else-block
-                b.set(size - 1, new Integer(b.size() - size + 1));
-                size = b.size();
+                b.add(parserLine, JMP, JS.ZERO);               // if we took the true-block, jump over the else-block
+                b.set(size - 1, JS.N(b.size - size + 1));
+                size = b.size;
                 parseStatement(b, null);
             }
-            b.set(size - 1, new Integer(b.size() - size + 1));        // regardless of which branch we took, b[size] needs to point here
+            b.set(size - 1, JS.N(b.size - size + 1));        // regardless of which branch we took, b[size] needs to point here
             break;
         }
         case WHILE: {
             consume(LP);
             if (label != null) b.add(parserLine, LABEL, label);
             b.add(parserLine, LOOP);
-            int size = b.size();
+            int size = b.size;
             b.add(parserLine, POP);                                   // discard the first-iteration indicator
             startExpr(b, -1);
-            b.add(parserLine, JT, new Integer(2));                    // if the while() clause is true, jump over the BREAK
+            b.add(parserLine, JT, JS.N(2));                    // if the while() clause is true, jump over the BREAK
             b.add(parserLine, BREAK);
             consume(RP);
             parseStatement(b, null);
             b.add(parserLine, CONTINUE);                              // if we fall out of the end, definately continue
-            b.set(size - 1, new Integer(b.size() - size + 1));        // end of the loop
+            b.set(size - 1, JS.N(b.size - size + 1));        // end of the loop
             break;
         }
         case SWITCH: {
             consume(LP);
             if (label != null) b.add(parserLine, LABEL, label);
             b.add(parserLine, LOOP);
-            int size0 = b.size();
+            int size0 = b.size;
             startExpr(b, -1);
             consume(RP);
             consume(LC);
@@ -560,10 +623,10 @@ class Parser extends Lexer implements ByteCodes {
                     startExpr(b, -1);
                     consume(COLON);
                     b.add(parserLine, EQ);                         // check if we should do this case-block
-                    b.add(parserLine, JF, new Integer(0));         // if not, jump to the next one
-                    int size = b.size();
+                    b.add(parserLine, JF, JS.ZERO);         // if not, jump to the next one
+                    int size = b.size;
                     while(peekToken() != CASE && peekToken() != DEFAULT && peekToken() != RC) parseStatement(b, null);
-                    b.set(size - 1, new Integer(1 + b.size() - size));
+                    b.set(size - 1, JS.N(1 + b.size - size));
                 } else if (peekToken() == DEFAULT) {
                     consume(DEFAULT);
                     consume(COLON);
@@ -575,44 +638,44 @@ class Parser extends Lexer implements ByteCodes {
                 } else {
                     throw pe("expected CASE, DEFAULT, or RC; got " + codeToString[peekToken()]);
                 }
-            b.set(size0 - 1, new Integer(b.size() - size0 + 1));      // end of the loop
+            b.set(size0 - 1, JS.N(b.size - size0 + 1));      // end of the loop
             break;
         }
             
         case DO: {
             if (label != null) b.add(parserLine, LABEL, label);
             b.add(parserLine, LOOP);
-            int size = b.size();
+            int size = b.size;
             parseStatement(b, null);
             consume(WHILE);
             consume(LP);
             startExpr(b, -1);
-            b.add(parserLine, JT, new Integer(2));                  // check the while() clause; jump over the BREAK if true
+            b.add(parserLine, JT, JS.N(2));                  // check the while() clause; jump over the BREAK if true
             b.add(parserLine, BREAK);
             b.add(parserLine, CONTINUE);
             consume(RP);
             consume(SEMI);
-            b.set(size - 1, new Integer(b.size() - size + 1));      // end of the loop; write this location to the LOOP instruction
+            b.set(size - 1, JS.N(b.size - size + 1));      // end of the loop; write this location to the LOOP instruction
             break;
         }
             
         case TRY: {
             b.add(parserLine, TRY); // try bytecode causes a TryMarker to be pushed
-            int tryInsn = b.size() - 1;
+            int tryInsn = b.size - 1;
             // parse the expression to be TRYed
             parseStatement(b, null); 
             // pop the try  marker. this is pushed when the TRY bytecode is executed                              
             b.add(parserLine, POP);
             // jump forward to the end of the catch block, start of the finally block                             
             b.add(parserLine, JMP);                                  
-            int successJMPInsn = b.size() - 1;
+            int successJMPInsn = b.size - 1;
             
             if (peekToken() != CATCH && peekToken() != FINALLY)
                 throw pe("try without catch or finally");
             
             int catchJMPDistance = -1;
             if (peekToken() == CATCH) {
-                catchJMPDistance = b.size() - tryInsn;
+                catchJMPDistance = b.size - tryInsn;
                 String exceptionVar;
                 getToken();
                 consume(LP);
@@ -633,12 +696,12 @@ class Parser extends Lexer implements ByteCodes {
             }
             
             // jump here if no exception was thrown
-            b.set(successJMPInsn, new Integer(b.size() - successJMPInsn)); 
+            b.set(successJMPInsn, JS.N(b.size - successJMPInsn)); 
                         
             int finallyJMPDistance = -1;
             if (peekToken() == FINALLY) {
                 b.add(parserLine, LITERAL, null); // null FinallyData
-                finallyJMPDistance = b.size() - tryInsn;
+                finallyJMPDistance = b.size - tryInsn;
                 consume(FINALLY);
                 parseStatement(b, null);
                 b.add(parserLine,FINALLY_DONE); 
@@ -667,7 +730,7 @@ class Parser extends Lexer implements ByteCodes {
                     
                 b.add(parserLine, LOOP);                                 // we actually only add this to ensure that BREAK works
                 b.add(parserLine, POP);                                  // discard the first-iteration indicator
-                int size = b.size();
+                int size = b.size;
                 consume(NAME);
                 consume(IN);
                 startExpr(b, -1);
@@ -676,12 +739,12 @@ class Parser extends Lexer implements ByteCodes {
                 b.add(parserLine, GET);
                 consume(RP);
                     
-                b.add(parserLine, LITERAL, new Integer(1));              // decrement the length
+                b.add(parserLine, LITERAL, JS.N(1));              // decrement the length
                 b.add(parserLine, SUB);
                 b.add(parserLine, DUP);
-                b.add(parserLine, LITERAL, new Integer(0));              // see if we've exhausted all the elements
+                b.add(parserLine, LITERAL, JS.ZERO);              // see if we've exhausted all the elements
                 b.add(parserLine, LT);
-                b.add(parserLine, JF, new Integer(2));
+                b.add(parserLine, JF, JS.N(2));
                 b.add(parserLine, BREAK);                                // if we have, then BREAK
                 b.add(parserLine, GET_PRESERVE);                         // get the key out of the keys array
                 b.add(parserLine, LITERAL, varName);
@@ -689,7 +752,7 @@ class Parser extends Lexer implements ByteCodes {
                 parseStatement(b, null);                                 // do some stuff
                 b.add(parserLine, CONTINUE);                             // continue if we fall out the bottom
 
-                b.set(size - 1, new Integer(b.size() - size + 1));       // BREAK to here
+                b.set(size - 1, JS.N(b.size - size + 1));       // BREAK to here
                 b.add(parserLine, OLDSCOPE);                             // restore the scope
                     
             } else {
@@ -697,8 +760,8 @@ class Parser extends Lexer implements ByteCodes {
                 b.add(parserLine, NEWSCOPE);                             // grab a fresh scope
                     
                 parseStatement(b, null);                                 // initializer
-                CompiledFunctionImpl e2 =                                    // we need to put the incrementor before the test
-                    new JS.CompiledFunction(sourceName, parserLine, null, null);  // so we save the test here
+                JSFunction e2 =                                    // we need to put the incrementor before the test
+                    new JSFunction(sourceName, parserLine, null);  // so we save the test here
                 if (peekToken() != SEMI)
                     startExpr(e2, -1);
                 else
@@ -706,23 +769,23 @@ class Parser extends Lexer implements ByteCodes {
                 consume(SEMI);
                 if (label != null) b.add(parserLine, LABEL, label);
                 b.add(parserLine, LOOP);
-                int size2 = b.size();
+                int size2 = b.size;
                     
-                b.add(parserLine, JT, new Integer(0));                   // if we're on the first iteration, jump over the incrementor
-                int size = b.size();
+                b.add(parserLine, JT, JS.ZERO);                   // if we're on the first iteration, jump over the incrementor
+                int size = b.size;
                 if (peekToken() != RP) {                                 // do the increment thing
                     startExpr(b, -1);
                     b.add(parserLine, POP);
                 }
-                b.set(size - 1, new Integer(b.size() - size + 1));
+                b.set(size - 1, JS.N(b.size - size + 1));
                 consume(RP);
                     
                 b.paste(e2);                                             // ok, *now* test if we're done yet
-                b.add(parserLine, JT, new Integer(2));                   // break out if we don't meet the test
+                b.add(parserLine, JT, JS.N(2));                   // break out if we don't meet the test
                 b.add(parserLine, BREAK);
                 parseStatement(b, null);
                 b.add(parserLine, CONTINUE);                             // if we fall out the bottom, CONTINUE
-                b.set(size2 - 1, new Integer(b.size() - size2 + 1));     // end of the loop
+                b.set(size2 - 1, JS.N(b.size - size2 + 1));     // end of the loop
                     
                 b.add(parserLine, OLDSCOPE);                             // get our scope back
             }