Table extends Cache
[sbp.git] / src / edu / berkeley / sbp / Parser.java
index 6bf274d..3fad1dd 100644 (file)
@@ -13,24 +13,11 @@ public abstract class Parser<T extends Token, R> {
 
     private final Table pt;
 
-    private static void reachable(Element e, HashSet<Position> h) {
-        if (e instanceof Atom) return;
-        for(Sequence s : ((Union)e))
-            reachable(s.firstp(), h);
-    }
-    private static void reachable(Position p, HashSet<Position> h) {
-        if (h.contains(p)) return;
-        h.add(p);
-        if (p.element() != null) reachable(p.element(), h);
-    }
-
-    /**
-     *  create a parser to parse the grammar with start symbol <tt>u</tt>
-     */
+    /** create a parser to parse the grammar with start symbol <tt>u</tt> */
     protected Parser(Union u)  { this.pt = new Table(u, top()); }
     protected Parser(Table pt) { this.pt = pt; }
 
-    public abstract Forest<R> shiftedToken(T t);
+    public abstract Forest<R> shiftedToken(T t, Token.Location loc);
     public abstract Topology<T> top();
 
 
@@ -40,12 +27,14 @@ public abstract class Parser<T extends Token, R> {
     /** parse <tt>input</tt>, using the table <tt>pt</tt> to drive the parser */
     public Forest<R> parse(Token.Stream<T> input) throws IOException, Failed {
         GSS gss = new GSS();
-        GSS.Phase current = gss.new Phase(null, input.next());
+        Token.Location loc = input.getLocation();
+        GSS.Phase current = gss.new Phase(null, input.next(), loc);
         current.newNode(null, null, pt.start, true, null);
         for(;;) {
-            GSS.Phase next = gss.new Phase(current, input.next());
+            loc = input.getLocation();
+            GSS.Phase next = gss.new Phase(current, input.next(), loc);
             current.reduce();
-            Forest forest = current.token==null ? null : shiftedToken((T)current.token);
+            Forest forest = current.token==null ? null : shiftedToken((T)current.token, loc);
             current.shift(next, forest);
             if (current.isDone()) return (Forest<R>)current.finalResult;
             current.checkFailure();
@@ -62,7 +51,7 @@ public abstract class Parser<T extends Token, R> {
         public Failed() { this("", null); }
         public Failed(String message, Token.Location loc) { this.location = loc; this.message = message; }
         public Token.Location getLocation() { return location; }
-        public String toString() { return message + (location==null ? "" : (" at " + location + "\n" + location.getContext())); }
+        public String toString() { return message + (location==null ? "" : (" at " + location)); }
     }
 
     public static class Ambiguous extends RuntimeException {
@@ -80,23 +69,10 @@ public abstract class Parser<T extends Token, R> {
 
     // Table //////////////////////////////////////////////////////////////////////////////
 
-    static class Top extends Union { public Top() { super("0"); } }
-
     /** an SLR(1) parse table which may contain conflicts */
-    static class Table {
+    static class Table extends Walk.Cache {
 
-        private final Union start0 = new Top();
-        private final Sequence start0seq;
-        
-        public final Walk.Cache cache = new Walk.Cache();
-        
-        public HashSet<Position> closure()       {
-            HashSet<Position> hp = new HashSet<Position>();
-            reachable(start0, hp);
-            return hp;
-        }
-        public Position firstPosition()          { return start0seq.firstp(); }
-        public Position lastPosition()           { Position ret = start0seq.firstp(); while(!ret.isLast()) ret = ret.next(); return ret; }
+        public final Walk.Cache cache = this;
         
         private void walk(Element e, HashSet<Element> hs) {
             if (e==null) return;
@@ -109,21 +85,6 @@ public abstract class Parser<T extends Token, R> {
                     walk(p.element(), hs);
             }
         }
-        public HashSet<Element> walk() {
-            HashSet<Element> ret = new HashSet<Element>();
-            walk(start0, ret);
-            return ret;
-        }
-
-        /*
-        public String toString() {
-            StringBuffer sb = new StringBuffer();
-            for(Element e : walk())
-                if (e instanceof Union)
-                    ((Union)e).toString(sb);
-            return sb.toString();
-        }
-        */
 
         /** the start state */
         public final State   start;
@@ -134,23 +95,29 @@ public abstract class Parser<T extends Token, R> {
         /** construct a parse table for the given grammar */
         public Table(Topology top) { this("s", top); }
         public Table(String startSymbol, Topology top) { this(new Union(startSymbol), top); }
-        public Table(Union u, Topology top) {
-            start0seq = new Sequence.Singleton(u, null, null);
-            start0.add(start0seq);
+        public Table(Union ux, Topology top) {
+            Union start0 = new Union("0");
+            start0.add(new Sequence.Singleton(ux, null, null));
+
+            for(Sequence s : start0) cache.eof.put(s, true);
+            cache.eof.put(start0, true);
 
             // construct the set of states
             HashMap<HashSet<Position>,State>   all_states    = new HashMap<HashSet<Position>,State>();
-            HashSet<Element>                   all_elements  = walk();
+            HashSet<Element>                   all_elements  = new HashSet<Element>();
+            walk(start0, all_elements);
             for(Element e : all_elements)
                 cache.ys.put(e, new Walk.YieldSet(e, cache).walk());
-            this.start = new State(closure(), all_states, all_elements);
+            HashSet<Position> hp = new HashSet<Position>();
+            reachable(start0, hp);
+            this.start = new State(hp, all_states, all_elements);
 
             // for each state, fill in the corresponding "row" of the parse table
             for(State state : all_states.values())
                 for(Position p : state.hs) {
 
                     // the Grammar's designated "last position" is the only accepting state
-                    if (p==lastPosition())
+                    if (start0.contains(p.owner()) && p.next()==null)
                         state.accept = true;
 
                     // FIXME: how does right-nullability interact with follow restrictions?
@@ -236,7 +203,7 @@ public abstract class Parser<T extends Token, R> {
                     Atom a = (Atom)position.element();
                     HashSet<Position> hp = new HashSet<Position>();
                     reachable(position.next(), hp);
-                    bag0.addAll(a, /*clo.walk()*/hp);
+                    bag0.addAll(a, hp);
                 }
 
                 // Step 1b: for each _minimal, contiguous_ set of characters having an identical next-position
@@ -346,4 +313,19 @@ public abstract class Parser<T extends Token, R> {
     }
 
     private static final Forest[] emptyForestArray = new Forest[0];
+
+
+    // Helpers //////////////////////////////////////////////////////////////////////////////
+
+    private static void reachable(Element e, HashSet<Position> h) {
+        if (e instanceof Atom) return;
+        for(Sequence s : ((Union)e))
+            reachable(s.firstp(), h);
+    }
+    private static void reachable(Position p, HashSet<Position> h) {
+        if (h.contains(p)) return;
+        h.add(p);
+        if (p.element() != null) reachable(p.element(), h);
+    }
+
 }