Charj: Fixing bad code, catching a NPE and printing the error and moving on.
authorJonathan Lifflander <jliffl2@illinois.edu>
Wed, 30 Jun 2010 23:30:30 +0000 (18:30 -0500)
committerJonathan Lifflander <jliffl2@illinois.edu>
Wed, 30 Jun 2010 23:30:30 +0000 (18:30 -0500)
Still needs more fixing...commented location where the same thing is happening,
but the NPE is actually being acted on.

src/langs/charj/src/charj/translator/CharjAST.java

index 48b59c229f5aaff334345bc6a07d2af1ed3f2634..5f6f03d7f2c71aa9b95cf3308cd597123635d1ec 100644 (file)
@@ -85,16 +85,14 @@ public class CharjAST extends CommonTree
 
     public CharjAST getChildOfType(int type)
     {
-        try
-        {
-            for(CharjAST c : getChildren())
+        List<CharjAST> children = getChildren();
+        
+        if (children != null) {
+            for(CharjAST c : children)
                 if(c.getType() == type)
                     return c;
         }
-        catch(NullPointerException npe)
-        {
-            npe.printStackTrace();
-        }
+        
         return null;
     }
 
@@ -151,6 +149,7 @@ public class CharjAST extends CommonTree
             for(CharjAST c : children)
                addChild(c);
         }
+        //TODO: fix this bad code, do not catch an NPE and act on it
         catch(NullPointerException npe)
         {
             //npe.printStackTrace();