Disable #. in prompt reader

[eow] / empire.lisp

diff --git a/empire.lisp b/empire.lisp
index 8ce62fb2fb88d38e21f91b1326d9d4a160ddb206..9f28084bbf82bc8dad8939a1adb2cf911069ba96 100644 (file)
--- a/empire.lisp
+++ b/empire.lisp
@@ -91,7 +91,7 @@
 (defgeneric quit (connection))
 (defgeneric handle-connection (connection))
 (defgeneric read-message (connection))
-(defgeneric send-message (connection message))
+(defgeneric send-message (connection message &key next-mode))
 (defgeneric connected-p (connection))
 (defgeneric reconnect (conncetion))
 
@@ -127,8 +127,8 @@
 (defclass base-mode ()
   ((connection :initarg :connection :accessor connection)))
 (defclass init-mode (base-mode)
-  ((play-sent-p :initform nil
-               :documentation "Have we already sent the play command?")))
+  ((phase :initform :initial
+         :documentation "Initialization phase")))
 (defclass play-mode (base-mode) nil)
 
 (defun make-mode (connection mode)
@@ -141,6 +141,7 @@
     (empire-log:info "~a: set-new-mode ~a -> ~a" c (connection-mode c) new-mode)
     (setf (connection-mode c) mode)))
 
+(defgeneric handle-cmdok (base-mode message))
 (defgeneric handle-data (base-mode message))
 (defgeneric handle-init (base-mode message))
 (defgeneric handle-exit (base-mode message))
@@ -151,14 +152,35 @@
 
 (defmethod handle-init ((m init-mode) message)
   (declare (ignorable message))
-  (with-slots (play-sent-p) m
-    (if (not play-sent-p)
-       (let ((c (connection m)))
-         (login-and-play c)
-         (setf play-sent-p t))
-       (set-new-mode m 'play-mode))))
+  (with-slots (phase connection) m
+    (ecase phase
+      (:initial (handle-cmdok m message))
+      (:play-sent (set-new-mode m 'play-mode)))))
+
+(defmethod handle-cmdok ((m init-mode) message)
+  (declare (ignorable message))
+  (with-slots (phase connection) m
+    (flet ((init-phase (mode send-args next-phase)
+            (with-slots (phase connection) mode
+              (send-message-one connection (apply #'format nil send-args))
+              (setf phase next-phase))))
+      (with-slots (user password) (connection m)
+       (ecase phase
+         (:initial (init-phase m `("client eow ~a" ,*version*) :client-sent))
+         (:client-sent (init-phase m `("coun ~a" ,user) :coun-sent))
+         (:coun-sent (init-phase m `("pass ~a" ,password) :pass-sent))
+         (:pass-sent (init-phase m '("play") :play-sent)))))))
 
 (defmethod handle-exit ((m init-mode) message)
+  (declare (ignorable message))
+  (with-slots (phase connection) m
+    (ecase phase
+      (:play-sent (send-message-one connection "kill")
+                 (setf phase :kill-sent))
+      (:kill-sent (send-message-one connection "play")
+                 (setf phase :play-sent)))))
+
+(defmethod handle-data ((m init-mode) message)
   (declare (ignorable message))
   t)
 
@@ -176,16 +198,27 @@
 
 (defmethod handle-flush ((m play-mode) message)
   (let* ((c (connection m)))
-    (empire-web:prompt (session c) message)))
+    (empire-web:prompt (session c) message)
+    (send-next-line c)))
 
 (defmethod send-next-line ((c connection))
-  (send-message-one c (locked-queue:dequeue (send-queue c))))
+  (let ((next-event (locked-queue:dequeue (send-queue c))))
+    (etypecase next-event
+      (string (send-message-one c next-event))
+      (cons (let ((message (car next-event))
+                 (mode (cdr next-event)))
+             (send-message-one c message)
+             (set-new-mode c mode))))))
+
+(defun read-no-eval (stream)
+  (let ((*read-eval* nil))
+    (read stream)))
 
 (defmethod handle-prompt ((m play-mode) message)
   (let* ((c (connection m)))
     (with-input-from-string (s message)
-      (let* ((minutes (read s)) ;;FIXME DANGEROUS
-            (btus (read s)))
+      (let* ((minutes (read-no-eval s))
+            (btus (read-no-eval s)))
        (empire-web:prompt (session c)
                           (format nil "[~a,~a]: " minutes btus))))
     (send-next-line c)))
@@ -201,7 +234,7 @@
        `(,+C_FLUSH+ . handle-flush)
        `(,+C_BADCMD+ . handle-simple-message)
        `(,+C_FLASH+ . handle-simple-message)
-       `(,+C_CMDOK+ . handle-ignore)
+       `(,+C_CMDOK+ . handle-cmdok)
        `(,+C_PROMPT+ . handle-prompt)))
 
 (defun parse-server-line (line)
@@ -242,6 +275,7 @@
                (error 'no-handler :mode mode :type type))))
        line)
     (sb-int:closed-stream-error () nil)
+    (sb-int:simple-stream-error () nil)
     (end-of-file () nil)))
 
 (defun raw-send-message (s message)
@@ -255,15 +289,18 @@
   (let ((s (network-stream c)))
     (raw-send-message s message)))
 
-(defmethod send-message ((c connection) message)
+(defmethod send-message ((c connection) message &key next-mode)
   (if (not (connected-p c))
       (reconnect c))
-  (locked-queue:enqueue (send-queue c) message))
+  (if next-mode
+      (locked-queue:enqueue (send-queue c) (cons message next-mode))
+      (locked-queue:enqueue (send-queue c) message)))
 
 (defparameter +special-command-char+ #\;)
 
 (defun special-command-p (line)
-  (char= +special-command-char+ (aref line 0)))
+  (and (> (length line) 0)
+       (char= +special-command-char+ (aref line 0))))
 
 (defmethod special-xup ((c connection))
   t)
@@ -277,20 +314,5 @@
   (cond ((special-command-p line) (special-command c (subseq line 1)))
        (t (send-message c line))))
 
-(defgeneric login-and-play (connection))
-(defmethod login-and-play ((c connection))
-  (send-message-one c (format nil "client eow ~a" *version*))
-  (send-message-one c "user FIXME")
-  (with-slots (user password) c
-    (send-message-one c (format nil "coun ~a" user))
-    (send-message-one c (format nil "pass ~a" password)))
-  (send-message-one c "kill")
-  (send-message-one c "play"))
-
 (defmethod quit ((c connection))
   (usocket:socket-close (socket c)))
-
-
-;;; Tests
-(defvar *l1* "2 Empire server ready")
-;; (parse-server-line *l1*)