Add logging package. Handle flush prompts.
[eow] / empire.lisp
index 2a0c137a79684a8419573dff9a0e48cd3a6edafd..358d0010f5d8c5c1d1d715a60ebf89afd2c46ecb 100644 (file)
     :initarg :stream
     :accessor network-stream
     :documentation "Stream used to talk to the empire server.")
-   (logging-stream
-    :initarg :logging-stream
-    :accessor logging-stream
-    :initform t
-    :documentation "Messages coming back from the server are sent to
-this stream.")
    (connection-handler
     :accessor connection-handler
     :initform nil
@@ -58,7 +52,6 @@ this stream.")
                        (server-port *default-empire-server-port*)
                        (socket nil)
                        (network-stream nil)
-                       (logging-stream nil)
                        session)
   (make-instance 'connection
                 :user user
@@ -67,7 +60,6 @@ this stream.")
                 :server-port server-port
                 :socket socket
                 :stream network-stream
-                :logging-stream logging-stream
                 :session session))
 
 ;;; interface
@@ -75,7 +67,6 @@ this stream.")
                 (password nil)
                 (server-name *default-empire-server*)
                 (server-port *default-empire-server-port*)
-                (logging-stream *standard-output*)
                 session)
   "Connect to server and return a connection object."
   (let* ((socket (usocket:socket-connect server-name server-port))
@@ -84,13 +75,13 @@ this stream.")
                                      :password password
                                      :socket socket
                                       :network-stream stream
-                                      :logging-stream logging-stream
                                       :server-name server-name
                                      :server-port server-port
                                      :session session))
         (handler (spawn-with-name (format nil "empire-handler-~a" user)
                                   #'handle-connection connection)))
     (setf (connection-handler connection) handler)
+    (empire-log:info "empire:connect: ~a" connection)
     connection))
 
 (defgeneric quit (connection))
@@ -107,7 +98,8 @@ this stream.")
                                       #'handle-connection c)))
        (setf socket sock
              stream s
-             connection-handler handler))))
+             connection-handler handler)))
+  (empire-log:info "empire:reconnect: ~a" c))
 
 (defparameter +C_CMDOK+ "0")
 (defparameter +C_DATA+ "1")
@@ -145,6 +137,7 @@ this stream.")
 (defgeneric handle-init (base-mode message))
 (defgeneric handle-exit (base-mode message))
 (defgeneric handle-prompt (base-mode message))
+(defgeneric handle-flush (base-mode message))
 (defgeneric handle-simple-message (base-mode message))
 
 (defmethod handle-init ((m init-mode) message)
@@ -165,31 +158,30 @@ this stream.")
   (quit (connection m)))
 
 (defmethod handle-data ((m play-mode) message)
-  (let* ((c (connection m))
-        (log (logging-stream c)))
-    (format log "handle-data ~a~%" message)
+  (let* ((c (connection m)))
     (empire-web:data (session c) message)))
 
 (defmethod handle-simple-message ((m play-mode) message)
-  (let* ((c (connection m))
-        (log (logging-stream c)))
-    (format log "handle-simple-message ~a~%" message)
+  (let* ((c (connection m)))
     (empire-web:data (session c) message)))
 
+(defmethod handle-flush ((m play-mode) message)
+  (let* ((c (connection m)))
+    (empire-web:prompt (session c) message)))
+
 (defmethod handle-prompt ((m play-mode) message)
-  (let* ((c (connection m))
-        (log (logging-stream c)))
+  (let* ((c (connection m)))
     (with-input-from-string (s message)
       (let* ((minutes (read s)) ;;FIXME DANGEROUS
             (btus (read s)))
-       (format log "[~a:~a]: ~%" minutes btus)
-       (empire-web:prompt (session c) minutes btus)))))
+       (empire-web:prompt (session c)
+                          (format nil "[~a,~a]: " minutes btus))))))
 
 (defparameter *line-type-dispatch*
   (list `(,+C_DATA+ . handle-data)
        `(,+C_INIT+ . handle-init)
        `(,+C_EXIT+ . handle-exit)
-       `(,+C_FLUSH+ . handle-simple-message)
+       `(,+C_FLUSH+ . handle-flush)
        `(,+C_BADCMD+ . handle-simple-message)
        `(,+C_FLASH+ . handle-simple-message)
        `(,+C_PROMPT+ . handle-prompt)))
@@ -222,9 +214,8 @@ this stream.")
 (defmethod read-message ((c connection))
   (handler-case
       (let* ((s (network-stream c))
-            (log (logging-stream c))
             (line (read-line s)))
-       (format log "< ~a~%" line)
+       (empire-log:info "< ~a~%" line)
        (multiple-value-bind (message type) (parse-server-line line)
          (let ((handler (lookup-handler type))
                (mode (connection-mode c)))
@@ -241,9 +232,8 @@ this stream.")
     (force-output s))
   
 (defmethod send-message ((c connection) message)
-  (let ((s (network-stream c))
-       (log (logging-stream c)))
-    (format log "< ~a~%" message)
+  (let ((s (network-stream c)))
+    (empire-log:info "> ~a~%" message)
     (let ((sent-p nil)
          (tries 3))
       (loop
@@ -256,6 +246,7 @@ this stream.")
              (sb-int:closed-stream-error ()
                (progn
                  (decf tries)
+                 (empire-log:info "Connection close - retrying (~a tries left)" tries)
                  (reconnect c))))))
     message))