Handling the case NAME DOT THIS for inner classes
[IRC.git] / Robust / src / IR / Tree / SemanticCheck.java
index bf7f8f51166fa8aa86db32b195267658e96ce071..5d96ada540fe3cd60cbef22df604c129548bfc5a 100644 (file)
@@ -9,121 +9,166 @@ public class SemanticCheck {
   TypeUtil typeutil;
   Stack loopstack;
   HashSet toanalyze;
-  HashSet completed;
+  HashMap<ClassDescriptor, Integer> completed;
 
+  public static final int NOCHECK=0;
+  public static final int REFERENCE=1;
+  public static final int INIT=2;
+
+  boolean checkAll;
+
+  public boolean hasLayout(ClassDescriptor cd) {
+    return completed.get(cd)!=null&&completed.get(cd)==INIT;
+  }
 
   public SemanticCheck(State state, TypeUtil tu) {
+    this(state, tu, true);
+  }
+
+  public SemanticCheck(State state, TypeUtil tu, boolean checkAll) {
     this.state=state;
     this.typeutil=tu;
     this.loopstack=new Stack();
     this.toanalyze=new HashSet();
-    this.completed=new HashSet();
+    this.completed=new HashMap<ClassDescriptor, Integer>();
+    this.checkAll=checkAll;
   }
 
-  public ClassDescriptor getClass(String classname) {
-    ClassDescriptor cd=typeutil.getClass(classname, toanalyze);
-    checkClass(cd);
+  public ClassDescriptor getClass(ClassDescriptor context, String classname) {
+    return getClass(context, classname, INIT);
+  }
+  public ClassDescriptor getClass(ClassDescriptor context, String classnameIn, int fullcheck) {
+    ClassDescriptor cd=typeutil.getClass(context, classnameIn, toanalyze);
+    checkClass(cd, fullcheck);
     return cd;
   }
 
-  private void checkClass(ClassDescriptor cd) {
-    if (!completed.contains(cd)) {
-      completed.add(cd);
-      
-      //Set superclass link up
-      if (cd.getSuper()!=null) {
-       cd.setSuper(getClass(cd.getSuper()));
-    if(cd.getSuperDesc().isInterface()) {
-      throw new Error("Error! Class " + cd.getSymbol() + " extends interface " + cd.getSuper());
-    }
-       // Link together Field, Method, and Flag tables so classes
-       // inherit these from their superclasses
-       cd.getFieldTable().setParent(cd.getSuperDesc().getFieldTable());
-       cd.getMethodTable().setParent(cd.getSuperDesc().getMethodTable());
-       cd.getFlagTable().setParent(cd.getSuperDesc().getFlagTable());
-      }
-      // Link together Field, Method tables do classes inherit these from 
-      // their ancestor interfaces
-      Vector<String> sifv = cd.getSuperInterface();
-      for(int i = 0; i < sifv.size(); i++) {
-       ClassDescriptor superif = getClass(sifv.elementAt(i));
-       if(!superif.isInterface()) {
-         throw new Error("Error! Class " + cd.getSymbol() + " implements non-interface " + superif.getSymbol());
-       }
-       cd.addSuperInterfaces(superif);
-       cd.getFieldTable().addParentIF(superif.getFieldTable());
-       cd.getMethodTable().addParentIF(superif.getMethodTable());
-      }
-      
-      /* Check to see that fields are well typed */
-      for(Iterator field_it=cd.getFields(); field_it.hasNext();) {
-       FieldDescriptor fd=(FieldDescriptor)field_it.next();
-       checkField(cd,fd);
+  public void checkClass(ClassDescriptor cd) {
+    checkClass(cd, INIT);
+  }
+
+  public void checkClass(ClassDescriptor cd, int fullcheck) {
+    if (!completed.containsKey(cd)||completed.get(cd)<fullcheck) {
+      int oldstatus=completed.containsKey(cd)?completed.get(cd):0;
+      completed.put(cd, fullcheck);
+
+      if (fullcheck>=REFERENCE&&oldstatus<INIT) {
+        //Set superclass link up
+        if (cd.getSuper()!=null) {
+         ClassDescriptor superdesc=getClass(cd, cd.getSuper(), fullcheck);
+         if (superdesc.isInnerClass()) {
+           cd.setAsInnerClass();
+         }
+         if (superdesc.isInterface()) {
+           if (cd.getInline()) {
+             cd.setSuper(null);
+             cd.getSuperInterface().add(superdesc.getSymbol());
+           } else {
+             throw new Error("Error! Class " + cd.getSymbol() + " extends interface " + cd.getSuper());
+           }
+         } else {
+           cd.setSuperDesc(superdesc);
+
+           // Link together Field, Method, and Flag tables so classes
+           // inherit these from their superclasses
+           if (oldstatus<REFERENCE) {
+             cd.getFieldTable().setParent(cd.getSuperDesc().getFieldTable());
+             cd.getMethodTable().setParent(cd.getSuperDesc().getMethodTable());
+             cd.getFlagTable().setParent(cd.getSuperDesc().getFlagTable());
+           }
+         }
+        }
+        // Link together Field, Method tables do classes inherit these from
+        // their ancestor interfaces
+        Vector<String> sifv = cd.getSuperInterface();
+        for(int i = 0; i < sifv.size(); i++) {
+          ClassDescriptor superif = getClass(cd, sifv.elementAt(i), fullcheck);
+          if(!superif.isInterface()) {
+            throw new Error("Error! Class " + cd.getSymbol() + " implements non-interface " + superif.getSymbol());
+          }
+          if (oldstatus<REFERENCE) {
+            cd.addSuperInterfaces(superif);
+            cd.getMethodTable().addParentIF(superif.getMethodTable());
+            cd.getFieldTable().addParentIF(superif.getFieldTable());
+          }
+        }
       }
-      
-      for(Iterator method_it=cd.getMethods(); method_it.hasNext();) {
-        MethodDescriptor md=(MethodDescriptor)method_it.next();
-        checkMethod(cd,md);
-        if(md.isDefaultConstructor() && (cd.getSuperDesc() != null)) {
-          // add the construction of it super class, can only be super()
-          NameDescriptor nd=new NameDescriptor("super");
-          MethodInvokeNode min=new MethodInvokeNode(nd);
-          BlockExpressionNode ben=new BlockExpressionNode(min);
-          BlockNode bn = state.getMethodBody(md);
-          bn.addFirstBlockStatement(ben);
+      if (oldstatus<INIT&&fullcheck>=INIT) {
+       if (cd.isInnerClass()) {
+          Modifiers fdmodifiers=new Modifiers();
+         FieldDescriptor enclosingfd=new FieldDescriptor(fdmodifiers,new TypeDescriptor(cd.getSurroundingDesc()),"this___enclosing",null,false);
+         cd.addField(enclosingfd);
+       }
+
+        /* Check to see that fields are well typed */
+        for(Iterator field_it=cd.getFields(); field_it.hasNext(); ) {
+          FieldDescriptor fd=(FieldDescriptor)field_it.next();
+         try {
+          checkField(cd,fd);
+         } catch (Error e) {
+           System.out.println("Class/Field in "+cd+":"+fd);
+           throw e;
+         }
+        }
+        for(Iterator method_it=cd.getMethods(); method_it.hasNext(); ) {
+          MethodDescriptor md=(MethodDescriptor)method_it.next();
+         try {
+          checkMethod(cd,md);
+         } catch (Error e) {
+           System.out.println("Class/Method in "+cd+":"+md);
+           throw e;
+         }
         }
       }
     }
   }
 
   public void semanticCheck() {
-    SymbolTable classtable=state.getClassSymbolTable();
+    SymbolTable classtable = state.getClassSymbolTable();
     toanalyze.addAll(classtable.getValueSet());
     toanalyze.addAll(state.getTaskSymbolTable().getValueSet());
 
     // Do methods next
-    while(!toanalyze.isEmpty()) {
-      Object obj=toanalyze.iterator().next();
+    while (!toanalyze.isEmpty()) {
+      Object obj = toanalyze.iterator().next();
       if (obj instanceof TaskDescriptor) {
-       toanalyze.remove(obj);
-       TaskDescriptor td=(TaskDescriptor)obj;
-       try {
-         checkTask(td);
-       } catch( Error e ) {
-           System.out.println( "Error in "+td );
-           throw e;
-       }
+        toanalyze.remove(obj);
+        TaskDescriptor td = (TaskDescriptor) obj;
+        try {
+          checkTask(td);
+        } catch (Error e) {
+          System.out.println("Error in " + td);
+          throw e;
+        }
       } else {
-       ClassDescriptor cd=(ClassDescriptor)obj;
-       toanalyze.remove(cd);
-       //need to initialize typeutil object here...only place we can
-       //get class descriptors without first calling getclass
-       getClass(cd.getSymbol());
-       for(Iterator method_it=cd.getMethods(); method_it.hasNext();) {
-         MethodDescriptor md=(MethodDescriptor)method_it.next();
-         try {
-           checkMethodBody(cd,md);
-         } catch( Error e ) {
-           System.out.println( "Error in "+md );
-           throw e;
-         }
-       }
+        ClassDescriptor cd = (ClassDescriptor) obj;
+        toanalyze.remove(cd);
+
+        // need to initialize typeutil object here...only place we can
+        // get class descriptors without first calling getclass
+        getClass(cd, cd.getSymbol());
+        for (Iterator method_it = cd.getMethods(); method_it.hasNext(); ) {
+          MethodDescriptor md = (MethodDescriptor) method_it.next();
+          try {
+            checkMethodBody(cd, md);
+          } catch (Error e) {
+            System.out.println("Error in " + md);
+            throw e;
+          }
+        }
       }
     }
   }
 
-  public void checkTypeDescriptor(TypeDescriptor td) {
+  private void checkTypeDescriptor(ClassDescriptor cd, TypeDescriptor td) {
     if (td.isPrimitive())
       return;       /* Done */
     else if (td.isClass()) {
       String name=td.toString();
-      int index = name.lastIndexOf('.');
-      if(index != -1) {
-        name = name.substring(index+1);
-      }
-      ClassDescriptor field_cd=getClass(name);
+      ClassDescriptor field_cd=checkAll?getClass(cd, name):getClass(cd, name, REFERENCE);
+
       if (field_cd==null)
-       throw new Error("Undefined class "+name);
+        throw new Error("Undefined class "+name);
       td.setClassDescriptor(field_cd);
       return;
     } else if (td.isTag())
@@ -133,7 +178,7 @@ public class SemanticCheck {
   }
 
   public void checkField(ClassDescriptor cd, FieldDescriptor fd) {
-    checkTypeDescriptor(fd.getType());
+    checkTypeDescriptor(cd, fd.getType());
   }
 
   public void checkConstraintCheck(TaskDescriptor td, SymbolTable nametable, Vector ccs) {
@@ -143,8 +188,8 @@ public class SemanticCheck {
       ConstraintCheck cc=(ConstraintCheck) ccs.get(i);
 
       for(int j=0; j<cc.numArgs(); j++) {
-       ExpressionNode en=cc.getArg(j);
-       checkExpressionNode(td,nametable,en,null);
+        ExpressionNode en=cc.getArg(j);
+        checkExpressionNode(td,nametable,en,null);
       }
     }
   }
@@ -158,36 +203,36 @@ public class SemanticCheck {
       //Make sure the variable is declared as a parameter to the task
       VarDescriptor vd=(VarDescriptor)td.getParameterTable().get(varname);
       if (vd==null)
-       throw new Error("Parameter "+varname+" in Flag Effects not declared in "+td);
+        throw new Error("Parameter "+varname+" in Flag Effects not declared in "+td);
       fe.setVar(vd);
 
       //Make sure it correspods to a class
       TypeDescriptor type_d=vd.getType();
       if (!type_d.isClass())
-       throw new Error("Cannot have non-object argument for flag_effect");
+        throw new Error("Cannot have non-object argument for flag_effect");
 
       ClassDescriptor cd=type_d.getClassDesc();
       for(int j=0; j<fe.numEffects(); j++) {
-       FlagEffect flag=fe.getEffect(j);
-       String name=flag.getName();
-       FlagDescriptor flag_d=(FlagDescriptor)cd.getFlagTable().get(name);
-       //Make sure the flag is declared
-       if (flag_d==null)
-         throw new Error("Flag descriptor "+name+" undefined in class: "+cd.getSymbol());
-       if (flag_d.getExternal())
-         throw new Error("Attempting to modify external flag: "+name);
-       flag.setFlag(flag_d);
+        FlagEffect flag=fe.getEffect(j);
+        String name=flag.getName();
+        FlagDescriptor flag_d=(FlagDescriptor)cd.getFlagTable().get(name);
+        //Make sure the flag is declared
+        if (flag_d==null)
+          throw new Error("Flag descriptor "+name+" undefined in class: "+cd.getSymbol());
+        if (flag_d.getExternal())
+          throw new Error("Attempting to modify external flag: "+name);
+        flag.setFlag(flag_d);
       }
       for(int j=0; j<fe.numTagEffects(); j++) {
-       TagEffect tag=fe.getTagEffect(j);
-       String name=tag.getName();
-
-       Descriptor d=(Descriptor)nametable.get(name);
-       if (d==null)
-         throw new Error("Tag descriptor "+name+" undeclared");
-       else if (!(d instanceof TagVarDescriptor))
-         throw new Error(name+" is not a tag descriptor");
-       tag.setTag((TagVarDescriptor)d);
+        TagEffect tag=fe.getTagEffect(j);
+        String name=tag.getName();
+
+        Descriptor d=(Descriptor)nametable.get(name);
+        if (d==null)
+          throw new Error("Tag descriptor "+name+" undeclared");
+        else if (!(d instanceof TagVarDescriptor))
+          throw new Error(name+" is not a tag descriptor");
+        tag.setTag((TagVarDescriptor)d);
       }
     }
   }
@@ -196,15 +241,15 @@ public class SemanticCheck {
     for(int i=0; i<td.numParameters(); i++) {
       /* Check that parameter is well typed */
       TypeDescriptor param_type=td.getParamType(i);
-      checkTypeDescriptor(param_type);
+      checkTypeDescriptor(null, param_type);
 
       /* Check the parameter's flag expression is well formed */
       FlagExpressionNode fen=td.getFlag(td.getParameter(i));
       if (!param_type.isClass())
-       throw new Error("Cannot have non-object argument to a task");
+        throw new Error("Cannot have non-object argument to a task");
       ClassDescriptor cd=param_type.getClassDesc();
       if (fen!=null)
-       checkFlagExpressionNode(cd, fen);
+        checkFlagExpressionNode(cd, fen);
     }
 
     checkFlagEffects(td, td.getFlagEffects(),td.getParameterTable());
@@ -220,7 +265,7 @@ public class SemanticCheck {
       FlagOpNode fon=(FlagOpNode)fen;
       checkFlagExpressionNode(cd, fon.getLeft());
       if (fon.getRight()!=null)
-       checkFlagExpressionNode(cd, fon.getRight());
+        checkFlagExpressionNode(cd, fon.getRight());
       break;
     }
 
@@ -230,7 +275,7 @@ public class SemanticCheck {
       String name=fn.getFlagName();
       FlagDescriptor fd=(FlagDescriptor)cd.getFlagTable().get(name);
       if (fd==null)
-       throw new Error("Undeclared flag: "+name);
+        throw new Error("Undeclared flag: "+name);
       fn.setFlag(fd);
       break;
     }
@@ -244,18 +289,18 @@ public class SemanticCheck {
     /* Check for abstract methods */
     if(md.isAbstract()) {
       if(!cd.isAbstract() && !cd.isInterface()) {
-       throw new Error("Error! The non-abstract Class " + cd.getSymbol() + " contains an abstract method " + md.getSymbol());
+        throw new Error("Error! The non-abstract Class " + cd.getSymbol() + " contains an abstract method " + md.getSymbol());
       }
     }
 
     /* Check return type */
     if (!md.isConstructor() && !md.isStaticBlock())
-      if (!md.getReturnType().isVoid())
-       checkTypeDescriptor(md.getReturnType());
-
+      if (!md.getReturnType().isVoid()) {
+        checkTypeDescriptor(cd, md.getReturnType());
+      }
     for(int i=0; i<md.numParameters(); i++) {
       TypeDescriptor param_type=md.getParamType(i);
-      checkTypeDescriptor(param_type);
+      checkTypeDescriptor(cd, param_type);
     }
     /* Link the naming environments */
     if (!md.isStatic())     /* Fields aren't accessible directly in a static method, so don't link in this table */
@@ -265,19 +310,27 @@ public class SemanticCheck {
       VarDescriptor thisvd=new VarDescriptor(new TypeDescriptor(cd),"this");
       md.setThis(thisvd);
     }
+    if(md.isDefaultConstructor() && (cd.getSuperDesc() != null)) {
+      // add the construction of it super class, can only be super()
+      NameDescriptor nd=new NameDescriptor("super");
+      MethodInvokeNode min=new MethodInvokeNode(nd);
+      BlockExpressionNode ben=new BlockExpressionNode(min);
+      BlockNode bn = state.getMethodBody(md);
+      bn.addFirstBlockStatement(ben);
+    }
   }
 
   public void checkMethodBody(ClassDescriptor cd, MethodDescriptor md) {
     ClassDescriptor superdesc=cd.getSuperDesc();
     if (superdesc!=null) {
       Set possiblematches=superdesc.getMethodTable().getSet(md.getSymbol());
-      for(Iterator methodit=possiblematches.iterator(); methodit.hasNext();) {
-       MethodDescriptor matchmd=(MethodDescriptor)methodit.next();
-       if (md.matches(matchmd)) {
-         if (matchmd.getModifiers().isFinal()) {
-           throw new Error("Try to override final method in method:"+md+" declared in  "+cd);
-         }
-       }
+      for(Iterator methodit=possiblematches.iterator(); methodit.hasNext(); ) {
+        MethodDescriptor matchmd=(MethodDescriptor)methodit.next();
+        if (md.matches(matchmd)) {
+          if (matchmd.getModifiers().isFinal()) {
+            throw new Error("Try to override final method in method:"+md+" declared in  "+cd);
+          }
+        }
       }
     }
     BlockNode bn=state.getMethodBody(md);
@@ -310,7 +363,7 @@ public class SemanticCheck {
     case Kind.IfStatementNode:
       checkIfStatementNode(md, nametable, (IfStatementNode)bsn);
       return;
-      
+
     case Kind.SwitchStatementNode:
       checkSwitchStatementNode(md, nametable, (SwitchStatementNode)bsn);
       return;
@@ -340,11 +393,12 @@ public class SemanticCheck {
       return;
 
     case Kind.ContinueBreakNode:
-       checkContinueBreakNode(md, nametable, (ContinueBreakNode) bsn);
-       return;
+      checkContinueBreakNode(md, nametable, (ContinueBreakNode) bsn);
+      return;
 
     case Kind.SESENode:
     case Kind.GenReachNode:
+    case Kind.GenDefReachNode:
       // do nothing, no semantic check for SESEs
       return;
     }
@@ -358,7 +412,7 @@ public class SemanticCheck {
 
   void checkDeclarationNode(Descriptor md, SymbolTable nametable,  DeclarationNode dn) {
     VarDescriptor vd=dn.getVarDescriptor();
-    checkTypeDescriptor(vd.getType());
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), vd.getType());
     Descriptor d=nametable.get(vd.getSymbol());
     if ((d==null)||
         (d instanceof FieldDescriptor)) {
@@ -394,13 +448,13 @@ public class SemanticCheck {
   }
 
   void checkContinueBreakNode(Descriptor md, SymbolTable nametable, ContinueBreakNode cbn) {
-      if (loopstack.empty())
-         throw new Error("continue/break outside of loop");
-      Object o = loopstack.peek();
-      if(o instanceof LoopNode) {
-        LoopNode ln=(LoopNode)o;
-        cbn.setLoop(ln);
-      }
+    if (loopstack.empty())
+      throw new Error("continue/break outside of loop");
+    Object o = loopstack.peek();
+    if(o instanceof LoopNode) {
+      LoopNode ln=(LoopNode)o;
+      cbn.setLoop(ln);
+    }
   }
 
   void checkReturnNode(Descriptor d, SymbolTable nametable, ReturnNode rn) {
@@ -409,11 +463,11 @@ public class SemanticCheck {
     MethodDescriptor md=(MethodDescriptor)d;
     if (rn.getReturnExpression()!=null)
       if (md.getReturnType()==null)
-       throw new Error("Constructor can't return something.");
+        throw new Error("Constructor can't return something.");
       else if (md.getReturnType().isVoid())
-       throw new Error(md+" is void");
+        throw new Error(md+" is void");
       else
-       checkExpressionNode(md, nametable, rn.getReturnExpression(), md.getReturnType());
+        checkExpressionNode(md, nametable, rn.getReturnExpression(), md.getReturnType());
     else
     if (md.getReturnType()!=null&&!md.getReturnType().isVoid())
       throw new Error("Need to return something for "+md);
@@ -432,10 +486,10 @@ public class SemanticCheck {
     if (isn.getFalseBlock()!=null)
       checkBlockNode(md, nametable, isn.getFalseBlock());
   }
-  
+
   void checkSwitchStatementNode(Descriptor md, SymbolTable nametable, SwitchStatementNode ssn) {
     checkExpressionNode(md, nametable, ssn.getCondition(), new TypeDescriptor(TypeDescriptor.INT));
-    
+
     BlockNode sbn = ssn.getSwitchBody();
     boolean hasdefault = false;
     for(int i = 0; i < sbn.size(); i++) {
@@ -446,7 +500,7 @@ public class SemanticCheck {
       hasdefault = containdefault;
     }
   }
-  
+
   boolean checkSwitchBlockNode(Descriptor md, SymbolTable nametable, SwitchBlockNode sbn) {
     Vector<SwitchLabelNode> slnv = sbn.getSwitchConditions();
     int defaultb = 0;
@@ -466,21 +520,21 @@ public class SemanticCheck {
       return (defaultb > 0);
     }
   }
-  
+
   void checkConstantExpressionNode(Descriptor md, SymbolTable nametable, ExpressionNode en, TypeDescriptor td) {
     switch(en.kind()) {
     case Kind.FieldAccessNode:
       checkFieldAccessNode(md,nametable,(FieldAccessNode)en,td);
       return;
-     
+
     case Kind.LiteralNode:
       checkLiteralNode(md,nametable,(LiteralNode)en,td);
       return;
-      
+
     case Kind.NameNode:
       checkNameNode(md,nametable,(NameNode)en,td);
       return;
-      
+
     case Kind.OpNode:
       checkOpNode(md, nametable, (OpNode)en, td);
       return;
@@ -533,7 +587,7 @@ public class SemanticCheck {
     case Kind.TertiaryNode:
       checkTertiaryNode(md, nametable, (TertiaryNode)en, td);
       return;
-      
+
     case Kind.InstanceOfNode:
       checkInstanceOfNode(md, nametable, (InstanceOfNode) en, td);
       return;
@@ -541,7 +595,7 @@ public class SemanticCheck {
     case Kind.ArrayInitializerNode:
       checkArrayInitializerNode(md, nametable, (ArrayInitializerNode) en, td);
       return;
-     
+
     case Kind.ClassTypeNode:
       checkClassTypeNode(md, nametable, (ClassTypeNode) en, td);
       return;
@@ -550,26 +604,25 @@ public class SemanticCheck {
   }
 
   void checkClassTypeNode(Descriptor md, SymbolTable nametable, ClassTypeNode tn, TypeDescriptor td) {
-    checkTypeDescriptor(tn.getType());
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), tn.getType());
   }
-  
+
   void checkCastNode(Descriptor md, SymbolTable nametable, CastNode cn, TypeDescriptor td) {
     /* Get type descriptor */
     if (cn.getType()==null) {
       NameDescriptor typenamed=cn.getTypeName().getName();
-      String typename=typenamed.toString();
-      TypeDescriptor ntd=new TypeDescriptor(getClass(typename));
+      TypeDescriptor ntd=new TypeDescriptor(getClass(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), typenamed.toString()));
       cn.setType(ntd);
     }
 
     /* Check the type descriptor */
     TypeDescriptor cast_type=cn.getType();
-    checkTypeDescriptor(cast_type);
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null),cast_type);
 
     /* Type check */
     if (td!=null) {
       if (!typeutil.isSuperorType(td,cast_type))
-       throw new Error("Cast node returns "+cast_type+", but need "+td);
+        throw new Error("Cast node returns "+cast_type+", but need "+td);
     }
 
     ExpressionNode en=cn.getExpression();
@@ -583,25 +636,52 @@ public class SemanticCheck {
     if (typeutil.isCastable(etd, cast_type))
       return;
 
+    //rough hack to handle interfaces...should clean up
+    if (etd.isClass()&&cast_type.isClass()) {
+      ClassDescriptor cdetd=etd.getClassDesc();
+      ClassDescriptor cdcast_type=cast_type.getClassDesc();
+
+      if (cdetd.isInterface()&&!cdcast_type.getModifier().isFinal())
+       return;
+      
+      if (cdcast_type.isInterface()&&!cdetd.getModifier().isFinal())
+       return;
+    }
     /* Different branches */
     /* TODO: change if add interfaces */
     throw new Error("Cast will always fail\n"+cn.printNode(0));
   }
 
+  //FieldDescriptor checkFieldAccessNodeForParentNode( Descriptor md, SymbolTable na )
   void checkFieldAccessNode(Descriptor md, SymbolTable nametable, FieldAccessNode fan, TypeDescriptor td) {
     ExpressionNode left=fan.getExpression();
     checkExpressionNode(md,nametable,left,null);
     TypeDescriptor ltd=left.getType();
+    if (!ltd.isArray())
+      checkClass(ltd.getClassDesc(), INIT);
     String fieldname=fan.getFieldName();
 
     FieldDescriptor fd=null;
     if (ltd.isArray()&&fieldname.equals("length"))
       fd=FieldDescriptor.arrayLength;
-    else
+    else {
       fd=(FieldDescriptor) ltd.getClassDesc().getFieldTable().get(fieldname);
-
+    }
     if(ltd.isClassNameRef()) {
       // the field access is using a class name directly
+      if (fd==null) {
+       ClassDescriptor surroundingCls=ltd.getClassDesc().getSurroundingDesc();
+       
+       while(surroundingCls!=null) {
+         fd=(FieldDescriptor) surroundingCls.getFieldTable().get(fieldname);
+         if (fd!=null) {
+           fan.left=new ClassTypeNode(new TypeDescriptor(surroundingCls));
+           break;
+         }
+         surroundingCls=surroundingCls.getSurroundingDesc();
+       }
+      }
+
       if(ltd.getClassDesc().isEnum()) {
         int value = ltd.getClassDesc().getEnumConstant(fieldname);
         if(-1 == value) {
@@ -621,10 +701,49 @@ public class SemanticCheck {
       } else {
         throw new Error("Dereference of the non-static field "+ fieldname + " in "+fan.printNode(0)+" in "+md);
       }
-    } 
+    }
+
+    if (fd==null){
+       if((md instanceof MethodDescriptor) && false == ((MethodDescriptor)md).isStaticBlock()) {
+               ClassDescriptor cd = ((MethodDescriptor)md).getClassDesc();
+               FieldAccessNode theFieldNode =  fieldAccessExpression( cd, fieldname, fan.getNumLine() );
+               if( null != theFieldNode ) {
+                       //fan = theFieldNode;
+                       checkFieldAccessNode( md, nametable, theFieldNode, td );
+                       fan.setField( theFieldNode.getField() );
+                       fan.setExpression( theFieldNode.getExpression() );
+                       //TypeDescriptor td1 = fan.getType();
+                       //td1.toString();
+                       return;         
+               }       
+               }
+       throw new Error("Unknown field "+fieldname + " in "+fan.printNode(0)+" in "+md);
+      }
+    if (fd==null) {
+      ClassDescriptor surroundingCls=ltd.getClassDesc().getSurroundingDesc();
+      int numencloses=1;
+      while(surroundingCls!=null) {
+       fd=(FieldDescriptor)surroundingCls.getFieldTable().get(fieldname);
+       if (fd!=null) {
+         surroundingCls=ltd.getClassDesc().getSurroundingDesc();
+         FieldAccessNode ftmp=fan;
+         for(;numencloses>0;numencloses--) {
+           FieldAccessNode fnew=new FieldAccessNode(ftmp.left, "this___enclosing");
+           fnew.setField((FieldDescriptor)surroundingCls.getFieldTable().get("this___enclosing"));
+           ftmp.left=fnew;
+           ftmp=fnew;
+           surroundingCls=surroundingCls.getSurroundingDesc();
+         }
+         break;
+       }
+       surroundingCls=surroundingCls.getSurroundingDesc();
+       numencloses++;
+      }
+
+      if (fd==null)
+       throw new Error("Unknown field "+fieldname + " in "+fan.printNode(0)+" in "+md);
+    }
 
-    if (fd==null)
-      throw new Error("Unknown field "+fieldname + " in "+fan.printNode(0)+" in "+md);
 
     if (fd.getType().iswrapper()) {
       FieldAccessNode fan2=new FieldAccessNode(left, fieldname);
@@ -639,13 +758,13 @@ public class SemanticCheck {
       FieldDescriptor fdwr=(FieldDescriptor) ltdwr.getClassDesc().getFieldTable().get(fieldnamewr);
       fan.setField(fdwr);
       if (fdwr==null)
-         throw new Error("Unknown field "+fieldnamewr + " in "+fan.printNode(0)+" in "+md);
+        throw new Error("Unknown field "+fieldnamewr + " in "+fan.printNode(0)+" in "+md);
     } else {
       fan.setField(fd);
     }
     if (td!=null) {
       if (!typeutil.isSuperorType(td,fan.getType()))
-       throw new Error("Field node returns "+fan.getType()+", but need "+td);
+        throw new Error("Field node returns "+fan.getType()+", but need "+td);
     }
   }
 
@@ -661,7 +780,7 @@ public class SemanticCheck {
 
     if (td!=null)
       if (!typeutil.isSuperorType(td,aan.getType()))
-       throw new Error("Field node returns "+aan.getType()+", but need "+td);
+        throw new Error("Field node returns "+aan.getType()+", but need "+td);
   }
 
   void checkLiteralNode(Descriptor md, SymbolTable nametable, LiteralNode ln, TypeDescriptor td) {
@@ -682,23 +801,23 @@ public class SemanticCheck {
     } else if (o instanceof Character) {
       ln.setType(new TypeDescriptor(TypeDescriptor.CHAR));
     } else if (o instanceof String) {
-      ln.setType(new TypeDescriptor(getClass(TypeUtil.StringClass)));
+      ln.setType(new TypeDescriptor(getClass(null, TypeUtil.StringClass)));
     }
 
     if (td!=null)
       if (!typeutil.isSuperorType(td,ln.getType())) {
         Long l = ln.evaluate();
-        if((ln.getType().isByte() || ln.getType().isShort() 
-            || ln.getType().isChar() || ln.getType().isInt()) 
-            && (l != null) 
-            && (td.isByte() || td.isShort() || td.isChar() 
-                || td.isInt() || td.isLong())) {
+        if((ln.getType().isByte() || ln.getType().isShort()
+            || ln.getType().isChar() || ln.getType().isInt())
+           && (l != null)
+           && (td.isByte() || td.isShort() || td.isChar()
+               || td.isInt() || td.isLong())) {
           long lnvalue = l.longValue();
-          if((td.isByte() && ((lnvalue > 127) || (lnvalue < -128))) 
-              || (td.isShort() && ((lnvalue > 32767) || (lnvalue < -32768)))
-              || (td.isChar() && ((lnvalue > 65535) || (lnvalue < 0)))
-              || (td.isInt() && ((lnvalue > 2147483647) || (lnvalue < -2147483648)))
-              || (td.isLong() && ((lnvalue > 9223372036854775807L) || (lnvalue < -9223372036854775808L)))) {
+          if((td.isByte() && ((lnvalue > 127) || (lnvalue < -128)))
+             || (td.isShort() && ((lnvalue > 32767) || (lnvalue < -32768)))
+             || (td.isChar() && ((lnvalue > 65535) || (lnvalue < 0)))
+             || (td.isInt() && ((lnvalue > 2147483647) || (lnvalue < -2147483648)))
+             || (td.isLong() && ((lnvalue > 9223372036854775807L) || (lnvalue < -9223372036854775808L)))) {
             throw new Error("Field node returns "+ln.getType()+", but need "+td+" in "+md);
           }
         } else {
@@ -707,6 +826,49 @@ public class SemanticCheck {
       }
   }
 
+  FieldDescriptor recurseSurroundingClasses( ClassDescriptor icd, String varname ) {
+       if( null == icd || false == icd.isInnerClass() )
+               return null;
+       
+       ClassDescriptor surroundingDesc = icd.getSurroundingDesc();
+       if( null == surroundingDesc )
+               return null;
+       
+       SymbolTable fieldTable = surroundingDesc.getFieldTable();
+       FieldDescriptor fd = ( FieldDescriptor ) fieldTable.get( varname );
+       if( null != fd )
+               return fd;
+       return recurseSurroundingClasses( surroundingDesc, varname );
+  }
+
+  FieldAccessNode fieldAccessExpression( ClassDescriptor icd, String varname, int linenum ) {
+       FieldDescriptor fd = recurseSurroundingClasses( icd, varname );
+       if( null == fd )
+               return null;
+
+       ClassDescriptor cd = fd.getClassDescriptor();
+       int depth = 1;
+       int startingDepth = icd.getInnerDepth();
+
+       if( true == cd.isInnerClass() ) 
+               depth = cd.getInnerDepth();
+
+       String composed = "this";
+       NameDescriptor runningDesc = new NameDescriptor( "this" );;
+       
+       for ( int index = startingDepth; index > depth; --index ) {
+               composed = "this$" + String.valueOf( index - 1  );      
+               runningDesc = new NameDescriptor( runningDesc, composed );
+       }
+       if( false == cd.isInnerClass() )
+               runningDesc = new NameDescriptor( runningDesc, "this$" + String.valueOf(0) ); //all the way up.
+       NameDescriptor idDesc = new NameDescriptor( runningDesc, varname );
+       
+       
+       FieldAccessNode theFieldNode = ( FieldAccessNode )translateNameDescriptorintoExpression( idDesc, linenum );
+       return theFieldNode;
+  }
+
   void checkNameNode(Descriptor md, SymbolTable nametable, NameNode nn, TypeDescriptor td) {
     NameDescriptor nd=nn.getName();
     if (nd.getBase()!=null) {
@@ -719,23 +881,33 @@ public class SemanticCheck {
       String varname=nd.toString();
       if(varname.equals("this")) {
         // "this"
-        nn.setVar((VarDescriptor)nametable.get("this")); 
+        nn.setVar((VarDescriptor)nametable.get("this"));
         return;
       }
       Descriptor d=(Descriptor)nametable.get(varname);
       if (d==null) {
         ClassDescriptor cd = null;
+       //check the inner class case first.
+       if((md instanceof MethodDescriptor) && false == ((MethodDescriptor)md).isStaticBlock()) {
+               cd = ((MethodDescriptor)md).getClassDesc();
+               FieldAccessNode theFieldNode =  fieldAccessExpression( cd, varname, nn.getNumLine() );
+               if( null != theFieldNode ) {
+                       nn.setExpression(( ExpressionNode )theFieldNode);
+                       checkExpressionNode(md,nametable,( ExpressionNode )theFieldNode,td);
+                       return;         
+               }               
+       }
         if((md instanceof MethodDescriptor) && ((MethodDescriptor)md).isStaticBlock()) {
           // this is a static block, all the accessed fields should be static field
           cd = ((MethodDescriptor)md).getClassDesc();
           SymbolTable fieldtbl = cd.getFieldTable();
           FieldDescriptor fd=(FieldDescriptor)fieldtbl.get(varname);
-          if((fd == null) || (!fd.isStatic())){
+          if((fd == null) || (!fd.isStatic())) {
             // no such field in the class, check if this is a class
             if(varname.equals("this")) {
               throw new Error("Error: access this obj in a static block");
             }
-            cd=getClass(varname);
+            cd=getClass(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), varname);
             if(cd != null) {
               // this is a class name
               nn.setClassDesc(cd);
@@ -765,49 +937,49 @@ public class SemanticCheck {
               throw new Error("Name "+varname+" should not be used in " + md);
             }
           }
-          cd=getClass(varname);
+          cd=getClass(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), varname);
           if(cd != null) {
             // this is a class name
             nn.setClassDesc(cd);
             return;
           } else {
-            throw new Error("Name "+varname+" undefined in: "+md);
+            throw new Error("Name "+varname+" undefined in: "+md);         
           }
         }
       }
       if (d instanceof VarDescriptor) {
-       nn.setVar(d);
+        nn.setVar(d);
       } else if (d instanceof FieldDescriptor) {
-       FieldDescriptor fd=(FieldDescriptor)d;
-       if (fd.getType().iswrapper()) {
-         String id=nd.getIdentifier();
-         NameDescriptor base=nd.getBase();
-         NameNode n=new NameNode(nn.getName());
-         n.setNumLine(nn.getNumLine());
-         n.setField(fd);
-         n.setVar((VarDescriptor)nametable.get("this"));        /* Need a pointer to this */
-         FieldAccessNode fan=new FieldAccessNode(n,"value");
-         fan.setNumLine(n.getNumLine());
-         FieldDescriptor fdval=(FieldDescriptor) fd.getType().getClassDesc().getFieldTable().get("value");
-         fan.setField(fdval);
-         nn.setExpression(fan);
-       } else {
-         nn.setField(fd);
-         nn.setVar((VarDescriptor)nametable.get("this"));        /* Need a pointer to this */
-       }
+        FieldDescriptor fd=(FieldDescriptor)d;
+        if (fd.getType().iswrapper()) {
+          String id=nd.getIdentifier();
+          NameDescriptor base=nd.getBase();
+          NameNode n=new NameNode(nn.getName());
+          n.setNumLine(nn.getNumLine());
+          n.setField(fd);
+          n.setVar((VarDescriptor)nametable.get("this"));        /* Need a pointer to this */
+          FieldAccessNode fan=new FieldAccessNode(n,"value");
+          fan.setNumLine(n.getNumLine());
+          FieldDescriptor fdval=(FieldDescriptor) fd.getType().getClassDesc().getFieldTable().get("value");
+          fan.setField(fdval);
+          nn.setExpression(fan);
+        } else {
+          nn.setField(fd);
+          nn.setVar((VarDescriptor)nametable.get("this"));        /* Need a pointer to this */
+        }
       } else if (d instanceof TagVarDescriptor) {
-       nn.setVar(d);
+        nn.setVar(d);
       } else throw new Error("Wrong type of descriptor");
       if (td!=null)
-       if (!typeutil.isSuperorType(td,nn.getType()))
-         throw new Error("Field node returns "+nn.getType()+", but need "+td);
+        if (!typeutil.isSuperorType(td,nn.getType()))
+          throw new Error("Field node returns "+nn.getType()+", but need "+td);
     }
   }
 
   void checkOffsetNode(Descriptor md, SymbolTable nameTable, OffsetNode ofn, TypeDescriptor td) {
     TypeDescriptor ltd=ofn.td;
-    checkTypeDescriptor(ltd);
-    
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null),ltd);
+
     String fieldname = ofn.fieldname;
     FieldDescriptor fd=null;
     if (ltd.isArray()&&fieldname.equals("length")) {
@@ -824,9 +996,7 @@ public class SemanticCheck {
 
     if (td!=null) {
       if (!typeutil.isSuperorType(td, ofn.getType())) {
-       System.out.println(td);
-       System.out.println(ofn.getType());
-       throw new Error("Type of rside not compatible with type of lside"+ofn.printNode(0));
+        throw new Error("Type of rside not compatible with type of lside"+ofn.printNode(0));
       }
     }
   }
@@ -834,63 +1004,28 @@ public class SemanticCheck {
 
   void checkTertiaryNode(Descriptor md, SymbolTable nametable, TertiaryNode tn, TypeDescriptor td) {
     checkExpressionNode(md, nametable, tn.getCond(), new TypeDescriptor(TypeDescriptor.BOOLEAN));
-    checkExpressionNode(md, nametable, tn.getTrueExpr(), td );
-    checkExpressionNode(md, nametable, tn.getFalseExpr(), td );
+    checkExpressionNode(md, nametable, tn.getTrueExpr(), td);
+    checkExpressionNode(md, nametable, tn.getFalseExpr(), td);
   }
 
   void checkInstanceOfNode(Descriptor md, SymbolTable nametable, InstanceOfNode tn, TypeDescriptor td) {
     if (td!=null&&!td.isBoolean())
       throw new Error("Expecting type "+td+"for instanceof expression");
-    
-    checkTypeDescriptor(tn.getExprType());
+
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), tn.getExprType());
     checkExpressionNode(md, nametable, tn.getExpr(), null);
   }
 
   void checkArrayInitializerNode(Descriptor md, SymbolTable nametable, ArrayInitializerNode ain, TypeDescriptor td) {
     Vector<TypeDescriptor> vec_type = new Vector<TypeDescriptor>();
     for( int i = 0; i < ain.numVarInitializers(); ++i ) {
-      checkExpressionNode(md, nametable, ain.getVarInitializer(i), td==null?td:td.dereference());
+      checkExpressionNode(md, nametable, ain.getVarInitializer(i), td.dereference());
       vec_type.add(ain.getVarInitializer(i).getType());
     }
-    // descide the type of this variableInitializerNode
-    TypeDescriptor out_type = vec_type.elementAt(0);
-    for(int i = 1; i < vec_type.size(); i++) {
-      TypeDescriptor tmp_type = vec_type.elementAt(i);
-      if(out_type == null) {
-        if(tmp_type != null) {
-          out_type = tmp_type;
-        }
-      } else if(out_type.isNull()) {
-        if(!tmp_type.isNull() ) {
-          if(!tmp_type.isArray()) {
-            throw new Error("Error: mixed type in var initializer list");
-          } else {
-            out_type = tmp_type;
-          }
-        }
-      } else if(out_type.isArray()) {
-        if(tmp_type.isArray()) {
-          if(tmp_type.getArrayCount() > out_type.getArrayCount()) {
-            out_type = tmp_type;
-          }
-        } else if((tmp_type != null) && (!tmp_type.isNull())) {
-          throw new Error("Error: mixed type in var initializer list");
-        }
-      } else if(out_type.isInt()) {
-        if(!tmp_type.isInt()) {
-          throw new Error("Error: mixed type in var initializer list");
-        }
-      } else if(out_type.isString()) {
-        if(!tmp_type.isString()) {
-          throw new Error("Error: mixed type in var initializer list");
-        }
-      }
-    }
-    if(out_type != null) {
-      out_type = out_type.makeArray(state);
-      //out_type.setStatic();
-    }
-    ain.setType(out_type);
+    if (td==null)
+      throw new Error();
+
+    ain.setType(td);
   }
 
   void checkAssignmentNode(Descriptor md, SymbolTable nametable, AssignmentNode an, TypeDescriptor td) {
@@ -899,7 +1034,7 @@ public class SemanticCheck {
         (an.getOperation().getBaseOp().getOp()!=Operation.POSTINC&&
          an.getOperation().getBaseOp().getOp()!=Operation.POSTDEC))
       postinc=false;
-    if (!postinc)      
+    if (!postinc)
       checkExpressionNode(md, nametable, an.getSrc(),td);
     //TODO: Need check on validity of operation here
     if (!((an.getDest() instanceof FieldAccessNode)||
@@ -911,27 +1046,27 @@ public class SemanticCheck {
     /* We want parameter variables to tasks to be immutable */
     if (md instanceof TaskDescriptor) {
       if (an.getDest() instanceof NameNode) {
-       NameNode nn=(NameNode)an.getDest();
-       if (nn.getVar()!=null) {
-         if (((TaskDescriptor)md).getParameterTable().contains(nn.getVar().getSymbol()))
-           throw new Error("Can't modify parameter "+nn.getVar()+ " to task "+td.getSymbol());
-       }
+        NameNode nn=(NameNode)an.getDest();
+        if (nn.getVar()!=null) {
+          if (((TaskDescriptor)md).getParameterTable().contains(nn.getVar().getSymbol()))
+            throw new Error("Can't modify parameter "+nn.getVar()+ " to task "+td.getSymbol());
+        }
       }
     }
 
     if (an.getDest().getType().isString()&&an.getOperation().getOp()==AssignOperation.PLUSEQ) {
       //String add
-      ClassDescriptor stringcl=getClass(TypeUtil.StringClass);
+      ClassDescriptor stringcl=getClass(null, TypeUtil.StringClass);
       TypeDescriptor stringtd=new TypeDescriptor(stringcl);
       NameDescriptor nd=new NameDescriptor("String");
       NameDescriptor valuend=new NameDescriptor(nd, "valueOf");
 
       if (!(an.getSrc().getType().isString()&&(an.getSrc() instanceof OpNode))) {
-       MethodInvokeNode rightmin=new MethodInvokeNode(valuend);
-       rightmin.setNumLine(an.getSrc().getNumLine());
-       rightmin.addArgument(an.getSrc());
-       an.right=rightmin;
-       checkExpressionNode(md, nametable, an.getSrc(), null);
+        MethodInvokeNode rightmin=new MethodInvokeNode(valuend);
+        rightmin.setNumLine(an.getSrc().getNumLine());
+        rightmin.addArgument(an.getSrc());
+        an.right=rightmin;
+        checkExpressionNode(md, nametable, an.getSrc(), null);
       }
     }
 
@@ -946,8 +1081,8 @@ public class SemanticCheck {
             dt = dt.dereference();
             st = st.dereference();
           } while(dt.isArray());
-          if((st.isByte() || st.isShort() || st.isChar() || st.isInt()) 
-              && (dt.isByte() || dt.isShort() || dt.isChar() || dt.isInt() || dt.isLong())) {
+          if((st.isByte() || st.isShort() || st.isChar() || st.isInt())
+             && (dt.isByte() || dt.isShort() || dt.isChar() || dt.isInt() || dt.isLong())) {
             return;
           } else {
             throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
@@ -955,15 +1090,15 @@ public class SemanticCheck {
         }
       } else {
         Long l = an.getSrc().evaluate();
-        if((st.isByte() || st.isShort() || st.isChar() || st.isInt()) 
-            && (l != null) 
-            && (dt.isByte() || dt.isShort() || dt.isChar() || dt.isInt() || dt.isLong())) {
+        if((st.isByte() || st.isShort() || st.isChar() || st.isInt())
+           && (l != null)
+           && (dt.isByte() || dt.isShort() || dt.isChar() || dt.isInt() || dt.isLong())) {
           long lnvalue = l.longValue();
-          if((dt.isByte() && ((lnvalue > 127) || (lnvalue < -128))) 
-              || (dt.isShort() && ((lnvalue > 32767) || (lnvalue < -32768)))
-              || (dt.isChar() && ((lnvalue > 65535) || (lnvalue < 0)))
-              || (dt.isInt() && ((lnvalue > 2147483647) || (lnvalue < -2147483648)))
-              || (dt.isLong() && ((lnvalue > 9223372036854775807L) || (lnvalue < -9223372036854775808L)))) {
+          if((dt.isByte() && ((lnvalue > 127) || (lnvalue < -128)))
+             || (dt.isShort() && ((lnvalue > 32767) || (lnvalue < -32768)))
+             || (dt.isChar() && ((lnvalue > 65535) || (lnvalue < 0)))
+             || (dt.isInt() && ((lnvalue > 2147483647) || (lnvalue < -2147483648)))
+             || (dt.isLong() && ((lnvalue > 9223372036854775807L) || (lnvalue < -9223372036854775808L)))) {
             throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
           }
         } else {
@@ -974,7 +1109,7 @@ public class SemanticCheck {
   }
 
   void checkLoopNode(Descriptor md, SymbolTable nametable, LoopNode ln) {
-      loopstack.push(ln);
+    loopstack.push(ln);
     if (ln.getType()==LoopNode.WHILELOOP||ln.getType()==LoopNode.DOWHILELOOP) {
       checkExpressionNode(md, nametable, ln.getCondition(), new TypeDescriptor(TypeDescriptor.BOOLEAN));
       checkBlockNode(md, nametable, ln.getBody());
@@ -984,8 +1119,8 @@ public class SemanticCheck {
       BlockNode bn=ln.getInitializer();
       bn.getVarTable().setParent(nametable);
       for(int i=0; i<bn.size(); i++) {
-       BlockStatementNode bsn=bn.get(i);
-       checkBlockStatementNode(md, bn.getVarTable(),bsn);
+        BlockStatementNode bsn=bn.get(i);
+        checkBlockStatementNode(md, bn.getVarTable(),bsn);
       }
       //check the condition
       checkExpressionNode(md, bn.getVarTable(), ln.getCondition(), new TypeDescriptor(TypeDescriptor.BOOLEAN));
@@ -995,98 +1130,134 @@ public class SemanticCheck {
     loopstack.pop();
   }
 
-
-  void checkCreateObjectNode(Descriptor md, SymbolTable nametable, CreateObjectNode con, TypeDescriptor td) {
-    TypeDescriptor[] tdarray=new TypeDescriptor[con.numArgs()];
-    for(int i=0; i<con.numArgs(); i++) {
-      ExpressionNode en=con.getArg(i);
-      checkExpressionNode(md,nametable,en,null);
-      tdarray[i]=en.getType();
+  void InnerClassAddParamToCtor( MethodDescriptor md, ClassDescriptor cd, SymbolTable nametable, 
+                                CreateObjectNode con, TypeDescriptor td ) {
+       
+       TypeDescriptor cdsType = new TypeDescriptor( cd );
+       ExpressionNode conExp = con.getSurroundingClassExpression();
+       //System.out.println( "The surrounding class expression si " + con );
+       if( null == conExp ) {
+               if( md.isStatic() ) {
+                       throw new Error("trying to instantiate inner class: " +  con.getType() + " in a static scope" );
+               }
+               VarDescriptor thisVD = md.getThis();
+               if( null == thisVD ) {
+                       throw new Error( "this pointer is not defined in a non static scope" ); 
+               }                       
+               if( cdsType.equals( thisVD.getType() ) == false ) {
+                       throw new Error( "the type of this pointer is different than the type expected for inner class constructor. Initializing the inner class: "                             +  con.getType() + " in the wrong scope" );             
+               }       
+               //make this into an expression node.
+               NameNode nThis=new NameNode( new NameDescriptor( "this" ) );
+               con.addArgument( nThis );
+       }
+       else {
+               //REVISIT : here i am storing the expression as an expressionNode which does not implement type, there is no way for me to semantic check this argument.
+               con.addArgument( conExp );
+       }
+       //System.out.println( " the modified createObjectNode is " + con.printNode( 0 ) + "\n" );
+}
+  void checkCreateObjectNode(Descriptor md, SymbolTable nametable, CreateObjectNode con,
+                             TypeDescriptor td) {
+    TypeDescriptor[] tdarray = new TypeDescriptor[con.numArgs()];
+    for (int i = 0; i < con.numArgs(); i++) {
+      ExpressionNode en = con.getArg(i);
+      checkExpressionNode(md, nametable, en, null);
+      tdarray[i] = en.getType();
     }
 
-    TypeDescriptor typetolookin=con.getType();
-    checkTypeDescriptor(typetolookin);
+    TypeDescriptor typetolookin = con.getType();
+    checkTypeDescriptor(((md instanceof MethodDescriptor)?((MethodDescriptor)md).getClassDesc():null), typetolookin);
+
+    if (td != null && !typeutil.isSuperorType(td, typetolookin))
+      throw new Error(typetolookin + " isn't a " + td);
 
-    if (td!=null&&!typeutil.isSuperorType(td, typetolookin))
-      throw new Error(typetolookin + " isn't a "+td);
-    
     /* Check Array Initializers */
-    if((con.getArrayInitializer() != null)) {
-      checkArrayInitializerNode(md, nametable, con.getArrayInitializer(), td);
+    if ((con.getArrayInitializer() != null)) {
+      checkArrayInitializerNode(md, nametable, con.getArrayInitializer(), typetolookin);
     }
 
     /* Check flag effects */
-    if (con.getFlagEffects()!=null) {
-      FlagEffects fe=con.getFlagEffects();
-      ClassDescriptor cd=typetolookin.getClassDesc();
-
-      for(int j=0; j<fe.numEffects(); j++) {
-       FlagEffect flag=fe.getEffect(j);
-       String name=flag.getName();
-       FlagDescriptor flag_d=(FlagDescriptor)cd.getFlagTable().get(name);
-       //Make sure the flag is declared
-       if (flag_d==null)
-         throw new Error("Flag descriptor "+name+" undefined in class: "+cd.getSymbol());
-       if (flag_d.getExternal())
-         throw new Error("Attempting to modify external flag: "+name);
-       flag.setFlag(flag_d);
+    if (con.getFlagEffects() != null) {
+      FlagEffects fe = con.getFlagEffects();
+      ClassDescriptor cd = typetolookin.getClassDesc();
+
+      for (int j = 0; j < fe.numEffects(); j++) {
+        FlagEffect flag = fe.getEffect(j);
+        String name = flag.getName();
+        FlagDescriptor flag_d = (FlagDescriptor) cd.getFlagTable().get(name);
+        // Make sure the flag is declared
+        if (flag_d == null)
+          throw new Error("Flag descriptor " + name + " undefined in class: " + cd.getSymbol());
+        if (flag_d.getExternal())
+          throw new Error("Attempting to modify external flag: " + name);
+        flag.setFlag(flag_d);
       }
-      for(int j=0; j<fe.numTagEffects(); j++) {
-       TagEffect tag=fe.getTagEffect(j);
-       String name=tag.getName();
-
-       Descriptor d=(Descriptor)nametable.get(name);
-       if (d==null)
-         throw new Error("Tag descriptor "+name+" undeclared");
-       else if (!(d instanceof TagVarDescriptor))
-         throw new Error(name+" is not a tag descriptor");
-       tag.setTag((TagVarDescriptor)d);
+      for (int j = 0; j < fe.numTagEffects(); j++) {
+        TagEffect tag = fe.getTagEffect(j);
+        String name = tag.getName();
+
+        Descriptor d = (Descriptor) nametable.get(name);
+        if (d == null)
+          throw new Error("Tag descriptor " + name + " undeclared");
+        else if (!(d instanceof TagVarDescriptor))
+          throw new Error(name + " is not a tag descriptor");
+        tag.setTag((TagVarDescriptor) d);
       }
     }
 
-    if ((!typetolookin.isClass())&&(!typetolookin.isArray()))
-      throw new Error("Can't allocate primitive type:"+con.printNode(0));
+    if ((!typetolookin.isClass()) && (!typetolookin.isArray()))
+      throw new Error("Can't allocate primitive type:" + con.printNode(0));
 
     if (!typetolookin.isArray()) {
-      //Array's don't need constructor calls
-      ClassDescriptor classtolookin=typetolookin.getClassDesc();
-
-      Set methoddescriptorset=classtolookin.getMethodTable().getSet(typetolookin.getSymbol());
-      MethodDescriptor bestmd=null;
-NextMethod:
-      for(Iterator methodit=methoddescriptorset.iterator(); methodit.hasNext();) {
-       MethodDescriptor currmd=(MethodDescriptor)methodit.next();
-       /* Need correct number of parameters */
-       if (con.numArgs()!=currmd.numParameters())
-         continue;
-       for(int i=0; i<con.numArgs(); i++) {
-         if (!typeutil.isSuperorType(currmd.getParamType(i),tdarray[i]))
-           continue NextMethod;
-       }
-       /* Local allocations can't call global allocator */
-       if (!con.isGlobal()&&currmd.isGlobal())
-         continue;
-
-       /* Method okay so far */
-       if (bestmd==null)
-         bestmd=currmd;
-       else {
-         if (typeutil.isMoreSpecific(currmd,bestmd)) {
-           bestmd=currmd;
-         } else if (con.isGlobal()&&match(currmd, bestmd)) {
-           if (currmd.isGlobal()&&!bestmd.isGlobal())
-             bestmd=currmd;
-           else if (currmd.isGlobal()&&bestmd.isGlobal())
-             throw new Error();
-         } else if (!typeutil.isMoreSpecific(bestmd, currmd)) {
-           throw new Error("No method is most specific:"+bestmd+" and "+currmd);
-         }
+      // Array's don't need constructor calls
+      ClassDescriptor classtolookin = typetolookin.getClassDesc();
+      checkClass(classtolookin, INIT);
+      if( classtolookin.isInnerClass() ) {
+       InnerClassAddParamToCtor( (MethodDescriptor)md, ((MethodDescriptor)md).getClassDesc() , nametable, con, td );
+       tdarray = new TypeDescriptor[con.numArgs()];
+       for (int i = 0; i < con.numArgs(); i++) {
+               ExpressionNode en = con.getArg(i);
+               checkExpressionNode(md, nametable, en, null);
+               tdarray[i] = en.getType();
+        }
+      }
+      Set methoddescriptorset = classtolookin.getMethodTable().getSet(classtolookin.getSymbol());
+      MethodDescriptor bestmd = null;
+NextMethod: for (Iterator methodit = methoddescriptorset.iterator(); methodit.hasNext(); ) {
+        MethodDescriptor currmd = (MethodDescriptor) methodit.next();
+        /* Need correct number of parameters */
+        if (con.numArgs() != currmd.numParameters())
+          continue;
+        for (int i = 0; i < con.numArgs(); i++) {
+          if (!typeutil.isSuperorType(currmd.getParamType(i), tdarray[i]))
+            continue NextMethod;
+        }
+        /* Local allocations can't call global allocator */
+        if (!con.isGlobal() && currmd.isGlobal())
+          continue;
+
+        /* Method okay so far */
+        if (bestmd == null)
+          bestmd = currmd;
+        else {
+          if (typeutil.isMoreSpecific(currmd, bestmd)) {
+            bestmd = currmd;
+          } else if (con.isGlobal() && match(currmd, bestmd)) {
+            if (currmd.isGlobal() && !bestmd.isGlobal())
+              bestmd = currmd;
+            else if (currmd.isGlobal() && bestmd.isGlobal())
+              throw new Error();
+          } else if (!typeutil.isMoreSpecific(bestmd, currmd)) {
+            throw new Error("No method is most specific:" + bestmd + " and " + currmd);
+          }
 
-         /* Is this more specific than bestmd */
-       }
+          /* Is this more specific than bestmd */
+        }
+      }
+      if (bestmd == null) {
+        throw new Error("No method found for " + con.printNode(0) + " in " + md);
       }
-      if (bestmd==null)
-       throw new Error("No method found for "+con.printNode(0)+" in "+md);
       con.setConstructor(bestmd);
     }
   }
@@ -1100,7 +1271,7 @@ NextMethod:
       throw new Error();
     for(int i=0; i<md1.numParameters(); i++) {
       if (!md2.getParamType(i).equals(md1.getParamType(i)))
-       return false;
+        return false;
     }
     if (!md2.getReturnType().equals(md1.getReturnType()))
       return false;
@@ -1116,11 +1287,11 @@ NextMethod:
   ExpressionNode translateNameDescriptorintoExpression(NameDescriptor nd, int numLine) {
     String id=nd.getIdentifier();
     NameDescriptor base=nd.getBase();
-    if (base==null){
+    if (base==null) {
       NameNode nn=new NameNode(nd);
       nn.setNumLine(numLine);
       return nn;
-    }else{
+    } else {
       FieldAccessNode fan=new FieldAccessNode(translateNameDescriptorintoExpression(base,numLine),id);
       fan.setNumLine(numLine);
       return fan;
@@ -1141,6 +1312,7 @@ NextMethod:
       ExpressionNode en=min.getArg(i);
       checkExpressionNode(md,nametable,en,null);
       tdarray[i]=en.getType();
+
       if(en.getType().isClass() && en.getType().getClassDesc().isEnum()) {
         tdarray[i] = new TypeDescriptor(TypeDescriptor.INT);
       }
@@ -1149,15 +1321,12 @@ NextMethod:
     if (min.getExpression()!=null) {
       checkExpressionNode(md,nametable,min.getExpression(),null);
       typetolookin=min.getExpression().getType();
-      //if (typetolookin==null)
-      //throw new Error(md+" has null return type");
-
     } else if (min.getBaseName()!=null) {
       String rootname=min.getBaseName().getRoot();
       if (rootname.equals("super")) {
-       ClassDescriptor supercd=((MethodDescriptor)md).getClassDesc().getSuperDesc();
-       typetolookin=new TypeDescriptor(supercd);
-       min.setSuper();
+        ClassDescriptor supercd=((MethodDescriptor)md).getClassDesc().getSuperDesc();
+        typetolookin=new TypeDescriptor(supercd);
+        min.setSuper();
       } else if (rootname.equals("this")) {
         if(isstatic) {
           throw new Error("use this object in static method md = "+ md.toString());
@@ -1165,32 +1334,28 @@ NextMethod:
         ClassDescriptor cd=((MethodDescriptor)md).getClassDesc();
         typetolookin=new TypeDescriptor(cd);
       } else if (nametable.get(rootname)!=null) {
-       //we have an expression
-       min.setExpression(translateNameDescriptorintoExpression(min.getBaseName(),min.getNumLine()));
-       checkExpressionNode(md, nametable, min.getExpression(), null);
-       typetolookin=min.getExpression().getType();
+        //we have an expression
+        min.setExpression(translateNameDescriptorintoExpression(min.getBaseName(),min.getNumLine()));
+        checkExpressionNode(md, nametable, min.getExpression(), null);
+        typetolookin=min.getExpression().getType();
       } else {
-       if(!min.getBaseName().getSymbol().equals("System.out")) {
-         ExpressionNode nn = translateNameDescriptorintoExpression(min.getBaseName(),min.getNumLine());
-         checkExpressionNode(md, nametable, nn, null);
-         typetolookin = nn.getType();
-         if(!((nn.kind()== Kind.NameNode) && (((NameNode)nn).getField() == null)
-              && (((NameNode)nn).getVar() == null) && (((NameNode)nn).getExpression() == null))) {
-           // this is not a pure class name, need to add to 
-           min.setExpression(nn);
-         }
-       } else {
-         //we have a type
-         ClassDescriptor cd = null;
-         //if (min.getBaseName().getSymbol().equals("System.out"))
-         cd=getClass("System");
-         /*else {
-            cd=getClass(min.getBaseName().getSymbol());
-           }*/
-         if (cd==null)
-           throw new Error("md = "+ md.toString()+ "  "+min.getBaseName()+" undefined");
-         typetolookin=new TypeDescriptor(cd);
-       }
+        if(!min.getBaseName().getSymbol().equals("System.out")||state.JNI) {
+          ExpressionNode nn = translateNameDescriptorintoExpression(min.getBaseName(),min.getNumLine());
+          checkExpressionNode(md, nametable, nn, null);
+          typetolookin = nn.getType();
+          if(!((nn.kind()== Kind.NameNode) && (((NameNode)nn).getField() == null)
+               && (((NameNode)nn).getVar() == null) && (((NameNode)nn).getExpression() == null))) {
+            // this is not a pure class name, need to add to
+            min.setExpression(nn);
+          }
+        } else {
+          //we have a type
+          ClassDescriptor cd = getClass(null, "System");
+
+          if (cd==null)
+            throw new Error("md = "+ md.toString()+ "  "+min.getBaseName()+" undefined");
+          typetolookin=new TypeDescriptor(cd);
+        }
       }
     } else if ((md instanceof MethodDescriptor)&&min.getMethodName().equals("super")) {
       ClassDescriptor supercd=((MethodDescriptor)md).getClassDesc().getSuperDesc();
@@ -1203,37 +1368,36 @@ NextMethod:
       throw new Error("Unknown method call to "+min.getMethodName()+"in task"+md.getSymbol());
     }
     if (!typetolookin.isClass())
-      throw new Error("Error with method call to "+min.getMethodName());
+      throw new Error("Error with method call to "+min.getMethodName()+" in class "+typetolookin);
     ClassDescriptor classtolookin=typetolookin.getClassDesc();
-    //System.out.println("Method name="+min.getMethodName());
-
+    checkClass(classtolookin, REFERENCE);
     Set methoddescriptorset=classtolookin.getMethodTable().getSet(min.getMethodName());
     MethodDescriptor bestmd=null;
 NextMethod:
-    for(Iterator methodit=methoddescriptorset.iterator(); methodit.hasNext();) {
+    for(Iterator methodit=methoddescriptorset.iterator(); methodit.hasNext(); ) {
       MethodDescriptor currmd=(MethodDescriptor)methodit.next();
       /* Need correct number of parameters */
       if (min.numArgs()!=currmd.numParameters())
-       continue;
+        continue;
       for(int i=0; i<min.numArgs(); i++) {
-       if (!typeutil.isSuperorType(currmd.getParamType(i),tdarray[i]))
-         if(((!tdarray[i].isArray() &&(tdarray[i].isInt() || tdarray[i].isLong())) 
-             && currmd.getParamType(i).isClass() && currmd.getParamType(i).getClassDesc().getSymbol().equals("Object"))) {
-           // primitive parameters vs object
-         } else {
-           continue NextMethod;
-         }
+        if (!typeutil.isSuperorType(currmd.getParamType(i),tdarray[i]))
+          if(((!tdarray[i].isArray() &&(tdarray[i].isInt() || tdarray[i].isLong()))
+              && currmd.getParamType(i).isClass() && currmd.getParamType(i).getClassDesc().getSymbol().equals("Object"))) {
+            // primitive parameters vs object
+          } else {
+            continue NextMethod;
+          }
       }
       /* Method okay so far */
       if (bestmd==null)
-       bestmd=currmd;
+        bestmd=currmd;
       else {
-       if (typeutil.isMoreSpecific(currmd,bestmd)) {
-         bestmd=currmd;
-       } else if (!typeutil.isMoreSpecific(bestmd, currmd))
-         throw new Error("No method is most specific:"+bestmd+" and "+currmd);
+        if (typeutil.isMoreSpecific(currmd,bestmd)) {
+          bestmd=currmd;
+        } else if (!typeutil.isMoreSpecific(bestmd, currmd))
+          throw new Error("No method is most specific:"+bestmd+" and "+currmd);
 
-       /* Is this more specific than bestmd */
+        /* Is this more specific than bestmd */
       }
     }
     if (bestmd==null)
@@ -1243,35 +1407,35 @@ NextMethod:
     if ((td!=null)&&(min.getType()!=null)&&!typeutil.isSuperorType(td,  min.getType()))
       throw new Error(min.getType()+ " is not equal to or a subclass of "+td);
     /* Check whether we need to set this parameter to implied this */
-    if (! isstatic && !bestmd.isStatic()) {
+    if (!isstatic && !bestmd.isStatic()) {
       if (min.getExpression()==null) {
-       ExpressionNode en=new NameNode(new NameDescriptor("this"));
-       min.setExpression(en);
-       checkExpressionNode(md, nametable, min.getExpression(), null);
+        ExpressionNode en=new NameNode(new NameDescriptor("this"));
+        min.setExpression(en);
+        checkExpressionNode(md, nametable, min.getExpression(), null);
       }
     }
-    
+
     /* Check if we need to wrap primitive paratmeters to objects */
     for(int i=0; i<min.numArgs(); i++) {
       if(!tdarray[i].isArray() && (tdarray[i].isInt() || tdarray[i].isLong())
-        && min.getMethod().getParamType(i).isClass() && min.getMethod().getParamType(i).getClassDesc().getSymbol().equals("Object")) {
-       // Shall wrap this primitive parameter as a object
-       ExpressionNode exp = min.getArg(i);
-       TypeDescriptor ptd = null;
-       NameDescriptor nd=null;
-       if(exp.getType().isInt()) {
-         nd = new NameDescriptor("Integer");
-         ptd = state.getTypeDescriptor(nd);
-       } else if(exp.getType().isLong()) {
-         nd = new NameDescriptor("Long");
-         ptd = state.getTypeDescriptor(nd);
-       }
-       boolean isglobal = false;
-       String disjointId = null;
-       CreateObjectNode con=new CreateObjectNode(ptd, isglobal, disjointId);
-       con.addArgument(exp);
-       checkExpressionNode(md, nametable, con, null);
-       min.setArgument(con, i);
+         && min.getMethod().getParamType(i).isClass() && min.getMethod().getParamType(i).getClassDesc().getSymbol().equals("Object")) {
+        // Shall wrap this primitive parameter as a object
+        ExpressionNode exp = min.getArg(i);
+        TypeDescriptor ptd = null;
+        NameDescriptor nd=null;
+        if(exp.getType().isInt()) {
+          nd = new NameDescriptor("Integer");
+          ptd = state.getTypeDescriptor(nd);
+        } else if(exp.getType().isLong()) {
+          nd = new NameDescriptor("Long");
+          ptd = state.getTypeDescriptor(nd);
+        }
+        boolean isglobal = false;
+        String disjointId = null;
+        CreateObjectNode con=new CreateObjectNode(ptd, isglobal, disjointId);
+        con.addArgument(exp);
+        checkExpressionNode(md, nametable, con, null);
+        min.setArgument(con, i);
       }
     }
   }
@@ -1282,7 +1446,7 @@ NextMethod:
     if (on.getRight()!=null)
       checkExpressionNode(md, nametable, on.getRight(), null);
     TypeDescriptor ltd=on.getLeft().getType();
-    TypeDescriptor rtd=on.getRight()!=null ? on.getRight().getType() : null;
+    TypeDescriptor rtd=on.getRight()!=null?on.getRight().getType():null;
     TypeDescriptor lefttype=null;
     TypeDescriptor righttype=null;
     Operation op=on.getOp();
@@ -1291,12 +1455,12 @@ NextMethod:
     case Operation.LOGIC_OR:
     case Operation.LOGIC_AND:
       if (!(rtd.isBoolean()))
-       throw new Error();
+        throw new Error();
       on.setRightType(rtd);
 
     case Operation.LOGIC_NOT:
       if (!(ltd.isBoolean()))
-       throw new Error();
+        throw new Error();
       //no promotion
       on.setLeftType(ltd);
 
@@ -1307,13 +1471,13 @@ NextMethod:
       // 5.6.2 Binary Numeric Promotion
       //TODO unboxing of reference objects
       if (ltd.isDouble())
-       throw new Error();
+        throw new Error();
       else if (ltd.isFloat())
-       throw new Error();
+        throw new Error();
       else if (ltd.isLong())
-       lefttype=new TypeDescriptor(TypeDescriptor.LONG);
+        lefttype=new TypeDescriptor(TypeDescriptor.LONG);
       else
-       lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        lefttype=new TypeDescriptor(TypeDescriptor.INT);
       on.setLeftType(lefttype);
       on.setType(lefttype);
       break;
@@ -1324,16 +1488,16 @@ NextMethod:
       // 5.6.2 Binary Numeric Promotion
       //TODO unboxing of reference objects
       if (ltd.isDouble()||rtd.isDouble())
-       throw new Error();
+        throw new Error();
       else if (ltd.isFloat()||rtd.isFloat())
-       throw new Error();
+        throw new Error();
       else if (ltd.isLong()||rtd.isLong())
-       lefttype=new TypeDescriptor(TypeDescriptor.LONG);
+        lefttype=new TypeDescriptor(TypeDescriptor.LONG);
       // 090205 hack for boolean
       else if (ltd.isBoolean()||rtd.isBoolean())
-       lefttype=new TypeDescriptor(TypeDescriptor.BOOLEAN);
+        lefttype=new TypeDescriptor(TypeDescriptor.BOOLEAN);
       else
-       lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        lefttype=new TypeDescriptor(TypeDescriptor.INT);
       righttype=lefttype;
 
       on.setLeftType(lefttype);
@@ -1343,7 +1507,7 @@ NextMethod:
 
     case Operation.ISAVAILABLE:
       if (!(ltd.isPtr())) {
-       throw new Error("Can't use isavailable on non-pointers/non-parameters.");
+        throw new Error("Can't use isavailable on non-pointers/non-parameters.");
       }
       lefttype=ltd;
       on.setLeftType(lefttype);
@@ -1355,25 +1519,25 @@ NextMethod:
       // 5.6.2 Binary Numeric Promotion
       //TODO unboxing of reference objects
       if (ltd.isBoolean()||rtd.isBoolean()) {
-       if (!(ltd.isBoolean()&&rtd.isBoolean()))
-         throw new Error();
-       righttype=lefttype=new TypeDescriptor(TypeDescriptor.BOOLEAN);
+        if (!(ltd.isBoolean()&&rtd.isBoolean()))
+          throw new Error();
+        righttype=lefttype=new TypeDescriptor(TypeDescriptor.BOOLEAN);
       } else if (ltd.isPtr()||rtd.isPtr()) {
-       if (!(ltd.isPtr()&&rtd.isPtr())) {
-      if(!rtd.isEnum()) {
-        throw new Error();
-      }
-    }
-       righttype=rtd;
-       lefttype=ltd;
+        if (!(ltd.isPtr()&&rtd.isPtr())) {
+          if(!rtd.isEnum()) {
+            throw new Error();
+          }
+        }
+        righttype=rtd;
+        lefttype=ltd;
       } else if (ltd.isDouble()||rtd.isDouble())
-       righttype=lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
+        righttype=lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
       else if (ltd.isFloat()||rtd.isFloat())
-       righttype=lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
+        righttype=lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
       else if (ltd.isLong()||rtd.isLong())
-       righttype=lefttype=new TypeDescriptor(TypeDescriptor.LONG);
+        righttype=lefttype=new TypeDescriptor(TypeDescriptor.LONG);
       else
-       righttype=lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        righttype=lefttype=new TypeDescriptor(TypeDescriptor.INT);
 
       on.setLeftType(lefttype);
       on.setRightType(righttype);
@@ -1389,20 +1553,20 @@ NextMethod:
       // 5.6.2 Binary Numeric Promotion
       //TODO unboxing of reference objects
       if (!ltd.isNumber()||!rtd.isNumber()) {
-       if (!ltd.isNumber())
-         throw new Error("Leftside is not number"+on.printNode(0)+"type="+ltd.toPrettyString());
-       if (!rtd.isNumber())
-         throw new Error("Rightside is not number"+on.printNode(0));
+        if (!ltd.isNumber())
+          throw new Error("Leftside is not number"+on.printNode(0)+"type="+ltd.toPrettyString());
+        if (!rtd.isNumber())
+          throw new Error("Rightside is not number"+on.printNode(0));
       }
 
       if (ltd.isDouble()||rtd.isDouble())
-       lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
+        lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
       else if (ltd.isFloat()||rtd.isFloat())
-       lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
+        lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
       else if (ltd.isLong()||rtd.isLong())
-       lefttype=new TypeDescriptor(TypeDescriptor.LONG);
+        lefttype=new TypeDescriptor(TypeDescriptor.LONG);
       else
-       lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        lefttype=new TypeDescriptor(TypeDescriptor.INT);
       righttype=lefttype;
       on.setLeftType(lefttype);
       on.setRightType(righttype);
@@ -1411,30 +1575,30 @@ NextMethod:
 
     case Operation.ADD:
       if (ltd.isString()||rtd.isString()) {
-       ClassDescriptor stringcl=getClass(TypeUtil.StringClass);
-       TypeDescriptor stringtd=new TypeDescriptor(stringcl);
-       NameDescriptor nd=new NameDescriptor("String");
-       NameDescriptor valuend=new NameDescriptor(nd, "valueOf");
-       if (!(ltd.isString()&&(on.getLeft() instanceof OpNode))) {
-         MethodInvokeNode leftmin=new MethodInvokeNode(valuend);
-         leftmin.setNumLine(on.getLeft().getNumLine());
-         leftmin.addArgument(on.getLeft());
-         on.left=leftmin;
-         checkExpressionNode(md, nametable, on.getLeft(), null);
-       }
+        ClassDescriptor stringcl=getClass(null, TypeUtil.StringClass);
+        TypeDescriptor stringtd=new TypeDescriptor(stringcl);
+        NameDescriptor nd=new NameDescriptor("String");
+        NameDescriptor valuend=new NameDescriptor(nd, "valueOf");
+        if (!(ltd.isString()&&(on.getLeft() instanceof OpNode))) {
+          MethodInvokeNode leftmin=new MethodInvokeNode(valuend);
+          leftmin.setNumLine(on.getLeft().getNumLine());
+          leftmin.addArgument(on.getLeft());
+          on.left=leftmin;
+          checkExpressionNode(md, nametable, on.getLeft(), null);
+        }
 
-       if (!(rtd.isString()&&(on.getRight() instanceof OpNode))) {
-         MethodInvokeNode rightmin=new MethodInvokeNode(valuend);
-         rightmin.setNumLine(on.getRight().getNumLine());
-         rightmin.addArgument(on.getRight());
-         on.right=rightmin;
-         checkExpressionNode(md, nametable, on.getRight(), null);
-       }
+        if (!(rtd.isString()&&(on.getRight() instanceof OpNode))) {
+          MethodInvokeNode rightmin=new MethodInvokeNode(valuend);
+          rightmin.setNumLine(on.getRight().getNumLine());
+          rightmin.addArgument(on.getRight());
+          on.right=rightmin;
+          checkExpressionNode(md, nametable, on.getRight(), null);
+        }
 
-       on.setLeftType(stringtd);
-       on.setRightType(stringtd);
-       on.setType(stringtd);
-       break;
+        on.setLeftType(stringtd);
+        on.setRightType(stringtd);
+        on.setType(stringtd);
+        break;
       }
 
     case Operation.SUB:
@@ -1444,16 +1608,16 @@ NextMethod:
       // 5.6.2 Binary Numeric Promotion
       //TODO unboxing of reference objects
       if (ltd.isArray()||rtd.isArray()||!ltd.isNumber()||!rtd.isNumber())
-       throw new Error("Error in "+on.printNode(0));
+        throw new Error("Error in "+on.printNode(0));
 
       if (ltd.isDouble()||rtd.isDouble())
-       lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
+        lefttype=new TypeDescriptor(TypeDescriptor.DOUBLE);
       else if (ltd.isFloat()||rtd.isFloat())
-       lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
+        lefttype=new TypeDescriptor(TypeDescriptor.FLOAT);
       else if (ltd.isLong()||rtd.isLong())
-       lefttype=new TypeDescriptor(TypeDescriptor.LONG);
+        lefttype=new TypeDescriptor(TypeDescriptor.LONG);
       else
-       lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        lefttype=new TypeDescriptor(TypeDescriptor.INT);
       righttype=lefttype;
       on.setLeftType(lefttype);
       on.setRightType(righttype);
@@ -1464,16 +1628,16 @@ NextMethod:
     case Operation.RIGHTSHIFT:
     case Operation.URIGHTSHIFT:
       if (!rtd.isIntegerType())
-       throw new Error();
+        throw new Error();
       //5.6.1 Unary Numeric Promotion
       if (rtd.isByte()||rtd.isShort()||rtd.isInt())
-       righttype=new TypeDescriptor(TypeDescriptor.INT);
+        righttype=new TypeDescriptor(TypeDescriptor.INT);
       else
-       righttype=rtd;
+        righttype=rtd;
 
       on.setRightType(righttype);
       if (!ltd.isIntegerType())
-       throw new Error();
+        throw new Error();
 
     case Operation.UNARYPLUS:
     case Operation.UNARYMINUS:
@@ -1482,12 +1646,12 @@ NextMethod:
           case Operation.PREINC:
           case Operation.PREDEC:*/
       if (!ltd.isNumber())
-       throw new Error();
+        throw new Error();
       //5.6.1 Unary Numeric Promotion
       if (ltd.isByte()||ltd.isShort()||ltd.isInt())
-       lefttype=new TypeDescriptor(TypeDescriptor.INT);
+        lefttype=new TypeDescriptor(TypeDescriptor.INT);
       else
-       lefttype=ltd;
+        lefttype=ltd;
       on.setLeftType(lefttype);
       on.setType(lefttype);
       break;
@@ -1498,9 +1662,8 @@ NextMethod:
 
     if (td!=null)
       if (!typeutil.isSuperorType(td, on.getType())) {
-       System.out.println(td);
-       System.out.println(on.getType());
-       throw new Error("Type of rside not compatible with type of lside"+on.printNode(0));
+        throw new Error("Type of rside not compatible with type of lside"+on.printNode(0));
       }
   }
+
 }