Fix tabbing.... Please fix your editors so they do tabbing correctly!!! (Spaces...
[IRC.git] / Robust / src / IR / Tree / SemanticCheck.java
index 4feeb962ab0306c394fb6442e69220f2526965a0..a90adc06f55e8afcff8088ab6b44790eb841e2c8 100644 (file)
@@ -9,102 +9,134 @@ public class SemanticCheck {
   TypeUtil typeutil;
   Stack loopstack;
   HashSet toanalyze;
-  HashSet completed;
+  HashMap<ClassDescriptor, Integer> completed;
 
+  //This is the class mappings for a particular file based
+  //on the import names. Maps class to canonical class name.
+  static Hashtable singleImportMap;
+  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) {
+  public ClassDescriptor getClass(ClassDescriptor context, String classname) {
+    return getClass(context, classname, INIT);
+  }
+  public ClassDescriptor getClass(ClassDescriptor context, String classname, int fullcheck) {
+    if (context!=null) {
+//      System.out.println(context.getSymbol() + " is looking for " + classname);
+      Hashtable remaptable=context.getSingleImportMappings();
+      classname=remaptable.containsKey(classname)?((String)remaptable.get(classname)):classname;
+    }
     ClassDescriptor cd=typeutil.getClass(classname, toanalyze);
-    checkClass(cd);
+    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()));
-       // 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());
-    if(state.MGC) {
-      // TODO add version for normal Java later
-      // 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));
-        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);
-      }
-      
-      boolean hasConstructor = false;
-      for(Iterator method_it=cd.getMethods(); method_it.hasNext();) {
-       MethodDescriptor md=(MethodDescriptor)method_it.next();
-       checkMethod(cd,md);
-    hasConstructor |= md.isConstructor();
+  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) {
+         cd.setSuper(getClass(cd, cd.getSuper(), fullcheck));
+         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
+         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.getFieldTable().addParentIF(superif.getFieldTable());
+           cd.getMethodTable().addParentIF(superif.getMethodTable());
+         }
+       }
       }
-      if(!hasConstructor) {
-        // add a default constructor for this class
-        MethodDescriptor md = new MethodDescriptor(new Modifiers(Modifiers.PUBLIC),
-            cd.getSymbol(), false);
-        BlockNode bn=new BlockNode();
-        state.addTreeCode(md,bn);
-        cd.addMethod(md);
-        checkMethod(cd,md);
+      if (oldstatus<INIT&&fullcheck>=INIT) {
+       /* 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);
+       }
+       for(Iterator method_it=cd.getMethods(); method_it.hasNext(); ) {
+         MethodDescriptor md=(MethodDescriptor)method_it.next();
+         checkMethod(cd,md);
+       }
       }
     }
   }
 
   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;
+       TaskDescriptor td = (TaskDescriptor) obj;
        try {
          checkTask(td);
-       } catch( Error e ) {
-           System.out.println( "Error in "+td );
-           throw e;
+       } catch (Error e) {
+         System.out.println("Error in " + td);
+         throw e;
        }
       } else {
-       ClassDescriptor cd=(ClassDescriptor)obj;
+       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();
+       //set the class mappings based on imports.
+       singleImportMap = cd.getSingleImportMappings();
+
+       // 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 );
+           checkMethodBody(cd, md);
+         } catch (Error e) {
+           System.out.println("Error in " + md);
            throw e;
          }
        }
@@ -112,12 +144,13 @@ public class SemanticCheck {
     }
   }
 
-  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();
-      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);
       td.setClassDescriptor(field_cd);
@@ -129,7 +162,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) {
@@ -192,7 +225,7 @@ 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));
@@ -237,23 +270,21 @@ public class SemanticCheck {
   }
 
   public void checkMethod(ClassDescriptor cd, MethodDescriptor md) {
-    if(state.MGC) {
-      // TODO add version for normal Java later
-      /* 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());
-        }
+    /* 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());
       }
     }
-    /* Check return type */
-    if (!md.isConstructor())
-      if (!md.getReturnType().isVoid())
-       checkTypeDescriptor(md.getReturnType());
 
+    /* Check return type */
+    if (!md.isConstructor() && !md.isStaticBlock())
+      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 */
@@ -263,13 +294,21 @@ 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();) {
+      for(Iterator methodit=possiblematches.iterator(); methodit.hasNext(); ) {
        MethodDescriptor matchmd=(MethodDescriptor)methodit.next();
        if (md.matches(matchmd)) {
          if (matchmd.getModifiers().isFinal()) {
@@ -308,7 +347,7 @@ public class SemanticCheck {
     case Kind.IfStatementNode:
       checkIfStatementNode(md, nametable, (IfStatementNode)bsn);
       return;
-      
+
     case Kind.SwitchStatementNode:
       checkSwitchStatementNode(md, nametable, (SwitchStatementNode)bsn);
       return;
@@ -338,8 +377,8 @@ 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:
@@ -356,7 +395,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)) {
@@ -392,10 +431,13 @@ public class SemanticCheck {
   }
 
   void checkContinueBreakNode(Descriptor md, SymbolTable nametable, ContinueBreakNode cbn) {
-      if (loopstack.empty())
-         throw new Error("continue/break outside of loop");
-      LoopNode ln=(LoopNode)loopstack.peek();
+    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) {
@@ -427,48 +469,58 @@ 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++) {
       boolean containdefault = checkSwitchBlockNode(md, nametable, (SwitchBlockNode)sbn.get(i));
       if(hasdefault && containdefault) {
-        throw new Error("Error: duplicate default branch in switch-case statement in Method: " + md.getSymbol());
+       throw new Error("Error: duplicate default branch in switch-case statement in Method: " + md.getSymbol());
       }
       hasdefault = containdefault;
     }
   }
-  
+
   boolean checkSwitchBlockNode(Descriptor md, SymbolTable nametable, SwitchBlockNode sbn) {
     Vector<SwitchLabelNode> slnv = sbn.getSwitchConditions();
     int defaultb = 0;
     for(int i = 0; i < slnv.size(); i++) {
       if(slnv.elementAt(i).isdefault) {
-        defaultb++;
+       defaultb++;
       } else {
-        checkConstantExpressionNode(md, nametable, slnv.elementAt(i).getCondition(), new TypeDescriptor(TypeDescriptor.INT));
+       checkConstantExpressionNode(md, nametable, slnv.elementAt(i).getCondition(), new TypeDescriptor(TypeDescriptor.INT));
       }
     }
     if(defaultb > 1) {
       throw new Error("Error: duplicate default branch in switch-case statement in Method: " + md.getSymbol());
     } else {
+      loopstack.push(sbn);
       checkBlockNode(md, nametable, sbn.getSwitchBlockStatement());
+      loopstack.pop();
       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;
     }
     throw new Error();
   }
@@ -518,7 +570,7 @@ public class SemanticCheck {
     case Kind.TertiaryNode:
       checkTertiaryNode(md, nametable, (TertiaryNode)en, td);
       return;
-      
+
     case Kind.InstanceOfNode:
       checkInstanceOfNode(md, nametable, (InstanceOfNode) en, td);
       return;
@@ -526,22 +578,29 @@ public class SemanticCheck {
     case Kind.ArrayInitializerNode:
       checkArrayInitializerNode(md, nametable, (ArrayInitializerNode) en, td);
       return;
+
+    case Kind.ClassTypeNode:
+      checkClassTypeNode(md, nametable, (ClassTypeNode) en, td);
+      return;
     }
     throw new Error();
   }
 
+  void checkClassTypeNode(Descriptor md, SymbolTable nametable, ClassTypeNode tn, TypeDescriptor td) {
+    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) {
@@ -576,29 +635,36 @@ public class SemanticCheck {
       fd=FieldDescriptor.arrayLength;
     else
       fd=(FieldDescriptor) ltd.getClassDesc().getFieldTable().get(fieldname);
-    if(state.MGC) {
-      // TODO add version for normal Java later
-    if(ltd.isStatic()) {
+
+    if(ltd.isClassNameRef()) {
+      // the field access is using a class name directly
       if(ltd.getClassDesc().isEnum()) {
-        int value = ltd.getClassDesc().getEnumConstant(fieldname);
-        if(-1 == value) {
-          // check if this field is an enum constant
-          throw new Error(fieldname + " is not an enum constant in "+fan.printNode(0)+" in "+md);
-        }
-        fd = new FieldDescriptor(new Modifiers(Modifiers.PUBLIC|Modifiers.FINAL), new TypeDescriptor(TypeDescriptor.INT), fieldname, null, false);
-        fd.setAsEnum();
-        fd.setEnumValue(value);
-      } else if(!fd.isStatic()) {
-        // check if this field is a static field
-        throw new Error("Dereference of the non-static field "+ fieldname + " in "+fan.printNode(0)+" in "+md);
+       int value = ltd.getClassDesc().getEnumConstant(fieldname);
+       if(-1 == value) {
+         // check if this field is an enum constant
+         throw new Error(fieldname + " is not an enum constant in "+fan.printNode(0)+" in "+md);
+       }
+       fd = new FieldDescriptor(new Modifiers(Modifiers.PUBLIC|Modifiers.FINAL), new TypeDescriptor(TypeDescriptor.INT), fieldname, null, false);
+       fd.setAsEnum();
+       fd.setEnumValue(value);
+      } else if(fd == null) {
+       throw new Error("Could not find field "+ fieldname + " in "+fan.printNode(0)+" in "+md + " (Line: "+fan.getNumLine()+")");
+      } else if(fd.isStatic()) {
+       // check if this field is a static field
+       if(fd.getExpressionNode() != null) {
+         checkExpressionNode(md,nametable,fd.getExpressionNode(),null);
+       }
+      } else {
+       throw new Error("Dereference of the non-static 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);
+      fan2.setNumLine(left.getNumLine());
       fan2.setField(fd);
       fan.left=fan2;
       fan.fieldname="value";
@@ -609,7 +675,7 @@ 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);
     }
@@ -652,12 +718,29 @@ 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()))
-       throw new Error("Field node returns "+ln.getType()+", but need "+td+" in "+md);
+      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())) {
+         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)))) {
+           throw new Error("Field node returns "+ln.getType()+", but need "+td+" in "+md);
+         }
+       } else {
+         throw new Error("Field node returns "+ln.getType()+", but need "+td+" in "+md);
+       }
+      }
   }
 
   void checkNameNode(Descriptor md, SymbolTable nametable, NameNode nn, TypeDescriptor td) {
@@ -665,43 +748,68 @@ public class SemanticCheck {
     if (nd.getBase()!=null) {
       /* Big hack */
       /* Rewrite NameNode */
-      ExpressionNode en=translateNameDescriptorintoExpression(nd);
+      ExpressionNode en=translateNameDescriptorintoExpression(nd,nn.getNumLine());
       nn.setExpression(en);
       checkExpressionNode(md,nametable,en,td);
     } else {
       String varname=nd.toString();
+      if(varname.equals("this")) {
+       // "this"
+       nn.setVar((VarDescriptor)nametable.get("this"));
+       return;
+      }
       Descriptor d=(Descriptor)nametable.get(varname);
       if (d==null) {
-        if(state.MGC) {
-          // TODO add version for normal Java later
-        ClassDescriptor cd = null;
-        if(((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())){
-            // no such field in the class or it is not a static field
-            throw new Error("Name "+varname+" should not be used in static block: "+md);
-          } else {
-            // this is a static field
-            nn.setField(fd);
-            nn.setClassDesc(cd);
-            return;
-          }
-        } else {
-          cd=getClass(varname);
-          if(cd != null) {
-            // this is a class name
-            nn.setClassDesc(cd);
-            return;
-          } else {
-            throw new Error("Name "+varname+" undefined in: "+md);
-          }
-        }
-        } else {
-          throw new Error("Name "+varname+" undefined in: "+md);
-        }
+       ClassDescriptor cd = null;
+       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())) {
+           // 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(((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+" should not be used in static block: "+md);
+           }
+         } else {
+           // this is a static field
+           nn.setField(fd);
+           nn.setClassDesc(cd);
+           return;
+         }
+       } else {
+         // check if the var is a static field of the class
+         if(md instanceof MethodDescriptor) {
+           cd = ((MethodDescriptor)md).getClassDesc();
+           FieldDescriptor fd = (FieldDescriptor)cd.getFieldTable().get(varname);
+           if((fd != null) && (fd.isStatic())) {
+             nn.setField(fd);
+             nn.setClassDesc(cd);
+             if (td!=null)
+               if (!typeutil.isSuperorType(td,nn.getType()))
+                 throw new Error("Field node returns "+nn.getType()+", but need "+td);
+             return;
+           } else if(fd != null) {
+             throw new Error("Name "+varname+" should not be used in " + md);
+           }
+         }
+         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);
+         }
+       }
       }
       if (d instanceof VarDescriptor) {
        nn.setVar(d);
@@ -711,9 +819,11 @@ public class SemanticCheck {
          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);
@@ -732,8 +842,8 @@ public class SemanticCheck {
 
   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")) {
@@ -760,22 +870,63 @@ 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.dereference()); 
+      checkExpressionNode(md, nametable, ain.getVarInitializer(i), td==null?td: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);
   }
 
   void checkAssignmentNode(Descriptor md, SymbolTable nametable, AssignmentNode an, TypeDescriptor td) {
@@ -784,7 +935,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)||
@@ -806,13 +957,14 @@ public class SemanticCheck {
 
     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);
@@ -820,12 +972,45 @@ public class SemanticCheck {
     }
 
     if (!postinc&&!typeutil.isSuperorType(an.getDest().getType(),an.getSrc().getType())) {
-      throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
+      TypeDescriptor dt = an.getDest().getType();
+      TypeDescriptor st = an.getSrc().getType();
+      if(an.getSrc().kind() == Kind.ArrayInitializerNode) {
+       if(dt.getArrayCount() != st.getArrayCount()) {
+         throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
+       } else {
+         do {
+           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())) {
+           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));
+         }
+       }
+      } 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())) {
+         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)))) {
+           throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
+         }
+       } else {
+         throw new Error("Type of rside ("+an.getSrc().getType().toPrettyString()+") not compatible with type of lside ("+an.getDest().getType().toPrettyString()+")"+an.printNode(0));
+       }
+      }
     }
   }
 
   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());
@@ -847,92 +1032,98 @@ public class SemanticCheck {
   }
 
 
-  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 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 flag effects */
-    if (con.getFlagEffects()!=null) {
-      FlagEffects fe=con.getFlagEffects();
-      ClassDescriptor cd=typetolookin.getClassDesc();
+    /* Check Array Initializers */
+    if ((con.getArrayInitializer() != null)) {
+      checkArrayInitializerNode(md, nametable, con.getArrayInitializer(), td);
+    }
 
-      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());
+    /* 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);
+         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();
+      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");
+       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);
+         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();
+      // Array's don't need constructor calls
+      ClassDescriptor classtolookin = typetolookin.getClassDesc();
+      checkClass(classtolookin, INIT);
+
+      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())
+       if (con.numArgs() != currmd.numParameters())
          continue;
-       for(int i=0; i<con.numArgs(); i++) {
-         if (!typeutil.isSuperorType(currmd.getParamType(i),tdarray[i]))
+       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())
+       if (!con.isGlobal() && currmd.isGlobal())
          continue;
 
        /* Method okay so far */
-       if (bestmd==null)
-         bestmd=currmd;
+       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())
+         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");
+           throw new Error("No method is most specific:" + bestmd + " and " + currmd);
          }
 
          /* 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);
     }
   }
@@ -959,13 +1150,18 @@ NextMethod:
 
 
 
-  ExpressionNode translateNameDescriptorintoExpression(NameDescriptor nd) {
+  ExpressionNode translateNameDescriptorintoExpression(NameDescriptor nd, int numLine) {
     String id=nd.getIdentifier();
     NameDescriptor base=nd.getBase();
-    if (base==null)
-      return new NameNode(nd);
-    else
-      return new FieldAccessNode(translateNameDescriptorintoExpression(base),id);
+    if (base==null) {
+      NameNode nn=new NameNode(nd);
+      nn.setNumLine(numLine);
+      return nn;
+    } else {
+      FieldAccessNode fan=new FieldAccessNode(translateNameDescriptorintoExpression(base,numLine),id);
+      fan.setNumLine(numLine);
+      return fan;
+    }
   }
 
 
@@ -973,13 +1169,17 @@ NextMethod:
     /*Typecheck subexpressions
        and get types for expressions*/
 
+    boolean isstatic = false;
+    if((md instanceof MethodDescriptor) && ((MethodDescriptor)md).isStatic()) {
+      isstatic = true;
+    }
     TypeDescriptor[] tdarray=new TypeDescriptor[min.numArgs()];
     for(int i=0; i<min.numArgs(); i++) {
       ExpressionNode en=min.getArg(i);
       checkExpressionNode(md,nametable,en,null);
       tdarray[i]=en.getType();
-      if(state.MGC && en.getType().isClass() && en.getType().getClassDesc().isEnum()) {
-        tdarray[i] = new TypeDescriptor(TypeDescriptor.INT);
+      if(en.getType().isClass() && en.getType().getClassDesc().isEnum()) {
+       tdarray[i] = new TypeDescriptor(TypeDescriptor.INT);
       }
     }
     TypeDescriptor typetolookin=null;
@@ -994,21 +1194,40 @@ NextMethod:
       if (rootname.equals("super")) {
        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());
+       }
+       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.setExpression(translateNameDescriptorintoExpression(min.getBaseName(),min.getNumLine()));
        checkExpressionNode(md, nametable, min.getExpression(), null);
        typetolookin=min.getExpression().getType();
       } else {
-       //we have a type
-       ClassDescriptor cd;
-       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")) {
+         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(null, "System");
+         /*else {
+            cd=getClass(min.getBaseName().getSymbol());
+            }*/
+         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();
@@ -1023,19 +1242,25 @@ NextMethod:
     if (!typetolookin.isClass())
       throw new Error("Error with method call to "+min.getMethodName());
     ClassDescriptor classtolookin=typetolookin.getClassDesc();
+    checkClass(classtolookin, INIT);
     //System.out.println("Method name="+min.getMethodName());
 
     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;
       for(int i=0; i<min.numArgs(); i++) {
        if (!typeutil.isSuperorType(currmd.getParamType(i),tdarray[i]))
-         continue NextMethod;
+         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)
@@ -1044,7 +1269,7 @@ NextMethod:
        if (typeutil.isMoreSpecific(currmd,bestmd)) {
          bestmd=currmd;
        } else if (!typeutil.isMoreSpecific(bestmd, currmd))
-         throw new Error("No method is most specific");
+         throw new Error("No method is most specific:"+bestmd+" and "+currmd);
 
        /* Is this more specific than bestmd */
       }
@@ -1056,13 +1281,37 @@ 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 (!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);
       }
     }
+
+    /* 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);
+      }
+    }
   }
 
 
@@ -1071,7 +1320,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();
@@ -1148,8 +1397,11 @@ NextMethod:
          throw new Error();
        righttype=lefttype=new TypeDescriptor(TypeDescriptor.BOOLEAN);
       } else if (ltd.isPtr()||rtd.isPtr()) {
-       if (!(ltd.isPtr()&&rtd.isPtr()))
-         throw new Error();
+       if (!(ltd.isPtr()&&rtd.isPtr())) {
+         if(!rtd.isEnum()) {
+           throw new Error();
+         }
+       }
        righttype=rtd;
        lefttype=ltd;
       } else if (ltd.isDouble()||rtd.isDouble())
@@ -1197,12 +1449,13 @@ NextMethod:
 
     case Operation.ADD:
       if (ltd.isString()||rtd.isString()) {
-       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 (!(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);
@@ -1210,6 +1463,7 @@ NextMethod:
 
        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);
@@ -1287,4 +1541,5 @@ NextMethod:
        throw new Error("Type of rside not compatible with type of lside"+on.printNode(0));
       }
   }
+
 }