X-Git-Url: https://git.distorted.org.uk/~mdw/sod/blobdiff_plain/055797d428ee796327bd81e0406bc765cac9ef7f..122cd9509e433c57e037d2cdc73ac7b295d96636:/src/parser/parser-expr-impl.lisp

diff --git a/src/parser/parser-expr-impl.lisp b/src/parser/parser-expr-impl.lisp
index 16e0c53..d4bc3a0 100644
--- a/src/parser/parser-expr-impl.lisp
+++ b/src/parser/parser-expr-impl.lisp
@@ -26,7 +26,7 @@
 (cl:in-package #:sod-parser)
 
 ;;;--------------------------------------------------------------------------
-;;; Basic protocol implementation.
+;;; Basic protocol.
 
 (defclass expression-parse-state ()
   ((opstack :initform nil :type list)
@@ -35,9 +35,18 @@
   (:documentation
    "State for the expression parser.  Largely passive."))
 
-(defmethod push-value (value (state expression-parse-state))
-  (with-slots (valstack) state
-    (push value valstack)))
+(defgeneric push-operator (operator state)
+  (:documentation
+   "Push an OPERATOR onto the STATE's operator stack.
+
+   This should apply existing stacked operators as necessary to obey the
+   language's precedence rules."))
+
+(defgeneric apply-operator (operator state)
+  (:documentation
+   "Apply the OPERATOR to arguments on the STATE's value stack.
+
+   This should pop any necessary arguments, and push the result."))
 
 (defmethod push-operator (operator (state expression-parse-state))
   (with-slots (opstack) state
@@ -204,7 +213,7 @@
 		       (push-operator value state)))
 	       (multiple-value-bind (value winp) (parse p-operand)
 		 (unless winp (fail value))
-		 (push-value value state))
+		 (push value (slot-value state 'valstack)))
 	       (loop (multiple-value-bind (value winp) (parse p-postop)
 		       (unless winp (return))
 		       (push-operator value state)))))