X-Git-Url: http://git.megacz.com/?p=sbp.git;a=blobdiff_plain;f=src%2Fedu%2Fberkeley%2Fsbp%2FTree.java;h=cce7bfe6a405dadb0b7aa3e2c8f19d803ea73820;hp=558d23459bcee59296d9111ec7aa784c0bc8c698;hb=e086905f34b249ca1ae83c000859f2c603919a6b;hpb=3ee451bce342d4bb61ad6235ba57bdf817bfdd1a

diff --git a/src/edu/berkeley/sbp/Tree.java b/src/edu/berkeley/sbp/Tree.java
index 558d234..cce7bfe 100644
--- a/src/edu/berkeley/sbp/Tree.java
+++ b/src/edu/berkeley/sbp/Tree.java
@@ -1,9 +1,8 @@
-// Copyright 2006 all rights reserved; see LICENSE file for BSD-style license
+// Copyright 2006-2007 all rights reserved; see LICENSE file for BSD-style license
 
 package edu.berkeley.sbp;
 import edu.berkeley.sbp.*;
 import edu.berkeley.sbp.util.*;
-import edu.berkeley.sbp.bind.*;
 import java.io.*;
 import java.util.*;
 import java.lang.reflect.*;
@@ -13,53 +12,71 @@ public class Tree<NodeType>
     extends PrintableTree<Tree<NodeType>>
     implements Iterable<Tree<NodeType>> {
 
-    private final Input.Region location;
-    private final NodeType     head;
-    private final Tree<NodeType>[]    children;
-    private final boolean      lift;
+    private static final Tree[] emptyTree = new Tree[0];
 
-    /** the element at the head of the tree */
-    public NodeType                 head()        { return head; }
-
-    private Tree<NodeType> lifted() { return children[children.length-1]; }
+    private final Input.Region     location;
+    private final NodeType         ihead;
+    private final Tree<NodeType>[] children;
 
     /** the number of children the tree has */
-    public int               size() {
-        return lift
-            ? (children.length-1)+lifted().size()
-            : children.length;
-    }
+    public int               size() { return children.length; }
+
+    /** the element at the head of the tree */
+    public NodeType                 head()        { return ihead; }
+    public NodeType              getHead()        { return ihead; }
 
     /** the tree's children */
     public Iterable<Tree<NodeType>> children()    { return this; }
 
     /** the tree's children */
-    public Iterator<Tree<NodeType>> iterator()    {
-        return lift
-            ? new ConcatenateIterator(new ArrayIterator(children, 0, children.length-1),
-                                      children[children.length-1].iterator())
-            : new ArrayIterator(children);
-    }
+    public Iterator<Tree<NodeType>> iterator()    { return new ArrayIterator(children); }
 
     /** get the <tt>i</t>th child */
-    public Tree<NodeType> child(int i)  {
-        return lift && i >= children.length-1
-            ? children[children.length-1].child(i-(children.length-1))
-            : children[i];
-    }
+    public Tree<NodeType> child(int i)            { return children[i]; }
 
     /** get the input region that this tree was parsed from */
     public Input.Region    getRegion() { return location; }
 
-    public Tree(Input.Region loc, NodeType head)                     { this(loc, head, null); }
-    public Tree(Input.Region loc, NodeType head, Tree<NodeType>[] children) { this(loc, head, children, false); }
+    public Tree(Input.Region loc, NodeType head) {
+        location = loc;
+        ihead = head;
+        children = emptyTree;
+    }
+    public Tree(Input.Region loc, NodeType head, List<Tree<NodeType>> kids){
+        location = loc;
+        ihead = head;
+        if (kids.size() == 0)
+            children = emptyTree;
+        else {
+            children = new Tree[kids.size()];
+            kids.toArray(children);
+        }
+    }
+    public Tree(Input.Region loc, NodeType head, Tree<NodeType>[] kids) {
+        location = loc;
+        ihead = head;
+        children = (kids == null ? emptyTree : kids.clone());
+    }
 
+    // FIXME: fairly inefficient because we keep copying arrays
     /** package-private constructor, allows setting the "lift" bit */
-    Tree(Input.Region loc, NodeType head, Tree<NodeType>[] children, boolean lift) {
+    Tree(Input.Region loc, NodeType head, Tree<NodeType>[] children, boolean[] lifts) {
         this.location = loc;
-        this.head = head;
-        this.lift = lift && children != null && children.length > 0;
-        this.children = ArrayUtil.clone(children, Tree.class);
+        this.ihead = head;
+
+        int count = 0;
+        for(int i=0; i<children.length; i++)
+            count += lifts[i] ? children[i].size() : 1;
+
+        this.children = new Tree[count];
+        int j = 0;
+        for(int i=0; i<children.length; i++) {
+            if (!lifts[i])
+                this.children[j++] = children[i];
+            else
+                for(int k=0; k<children[i].size(); k++)
+                    this.children[j++] = children[i].child(k);
+        }
     }
 
 
@@ -71,7 +88,9 @@ public class Tree<NodeType>
         if (head()==null) return null;
         if (head() instanceof ToJava) {
             StringBuffer sb = new StringBuffer();
-            ((ToJava)head()).toJava(sb);
+            try {
+                ((ToJava)head()).toJava(sb);
+            } catch (IOException e) { throw new RuntimeException(e); }
             return sb.toString();
         }
         return (head()==null?"null":("\""+StringUtil.toJavaString(head().toString())+"\""));