(shell-command): Fix message spelling.
[bpt/emacs.git] / lisp / server.el
index 0de7dcd..888cda5 100644 (file)
@@ -1,6 +1,6 @@
 ;;; server.el --- Lisp code for GNU Emacs running as server process.
 
-;; Copyright (C) 1986, 1987, 1992, 1994 Free Software Foundation, Inc.
+;; Copyright (C) 1986, 1987, 1992, 1994, 1995 Free Software Foundation, Inc.
 
 ;; Author: William Sommerfeld <wesommer@athena.mit.edu>
 ;; Keywords: processes
@@ -122,8 +122,8 @@ by the programs that invoke the emacs server.")
       (save-excursion
        (set-buffer "*server*")
        (goto-char (point-max))
-       (insert string)
-       (or (bobp) (newline)))))
+       (insert (current-time-string) " " string)
+       (or (bolp) (newline)))))
 
 (defun server-sentinel (proc msg)
   (cond ((eq (process-status proc) 'exit)
@@ -147,9 +147,18 @@ Prefix arg means just kill any existing server communications subprocess."
        (set-process-sentinel server-process nil)
        (condition-case () (delete-process server-process) (error nil))))
   (condition-case () (delete-file "~/.emacs_server") (error nil))
-  (condition-case ()
-      (delete-file (format "/tmp/esrv%d-%s" (user-uid) (system-name)))
-    (error nil))
+  (let* ((sysname (system-name))
+        (dot-index (string-match "\\." sysname)))
+    (condition-case ()
+       (delete-file (format "/tmp/esrv%d-%s" (user-uid) sysname))
+      (error nil))
+    ;; In case the server file name was made with a domainless hostname,
+    ;; try deleting that name too.
+    (if dot-index
+       (condition-case ()
+           (delete-file (format "/tmp/esrv%d-%s" (user-uid)
+                                (substring sysname 0 dot-index)))
+         (error nil))))
   ;; If we already had a server, clear out associated status.
   (while server-clients
     (let ((buffer (nth 1 (car server-clients))))
@@ -171,34 +180,41 @@ Prefix arg means just kill any existing server communications subprocess."
 (defun server-process-filter (proc string)
   (server-log string)
   (setq string (concat server-previous-string string))
-  (if (not (and (eq ?\n (aref string (1- (length string))))
-               (eq 0 (string-match "Client: " string))))
-      ;; If input is not complete, save it for later.
-      (setq server-previous-string string)
-    ;; If it is complete, process it now, and discard what was saved.
-    (setq string (substring string (match-end 0)))
-    (setq server-previous-string "")
-    (let ((client (list (substring string 0 (string-match " " string))))
+  ;; If the input is multiple lines,
+  ;; process each line individually.
+  (while (string-match "\n" string)
+    (let ((request (substring string 0 (match-beginning 0)))
+         client
          (files nil)
          (lineno 1))
-      (setq string (substring string (match-end 0)))
-      (while (string-match "[^ ]+ " string)
-       (let ((arg
-              (substring string (match-beginning 0) (1- (match-end 0)))))
-         (setq string (substring string (match-end 0)))
-         (if (string-match "\\`\\+[0-9]+\\'" arg)
-             (setq lineno (read (substring arg 1)))
-           (setq files
-                 (cons (list arg lineno)
-                       files))
-           (setq lineno 1))))
-      (server-visit-files files client)
-      ;; CLIENT is now a list (CLIENTNUM BUFFERS...)
-      (setq server-clients (cons client server-clients))
-      (server-switch-buffer (nth 1 client))
-      (run-hooks 'server-switch-hook)
-      (message (substitute-command-keys
-               "When done with a buffer, type \\[server-edit].")))))
+      ;; Remove this line from STRING.
+      (setq string (substring string (match-end 0)))     
+      (if (string-match "^Error: " request)
+         (message (concat "Server error: " (substring request (match-end 0))))
+       (if (string-match "^Client: " request)
+           (progn
+             (setq request (substring request (match-end 0)))
+             (setq client (list (substring request 0 (string-match " " request))))
+             (setq request (substring request (match-end 0)))
+             (while (string-match "[^ ]+ " request)
+               (let ((arg
+                      (substring request (match-beginning 0) (1- (match-end 0)))))
+                 (setq request (substring request (match-end 0)))
+                 (if (string-match "\\`\\+[0-9]+\\'" arg)
+                     (setq lineno (read (substring arg 1)))
+                   (setq files
+                         (cons (list arg lineno)
+                               files))
+                   (setq lineno 1))))
+             (server-visit-files files client)
+             ;; CLIENT is now a list (CLIENTNUM BUFFERS...)
+             (setq server-clients (cons client server-clients))
+             (server-switch-buffer (nth 1 client))
+             (run-hooks 'server-switch-hook)
+             (message (substitute-command-keys
+                       "When done with a buffer, type \\[server-edit]")))))))
+  ;; Save for later any partial line that remains.
+  (setq server-previous-string string))
 
 (defun server-visit-files (files client)
   "Finds FILES and returns the list CLIENT with the buffers nconc'd.
@@ -236,36 +252,51 @@ FILES is an alist whose elements are (FILENAME LINENUMBER)."
 \f
 (defun server-buffer-done (buffer)
   "Mark BUFFER as \"done\" for its client(s).
-Buries the buffer, and returns another server buffer
-as a suggestion for what to select next."
+This buries the buffer, then returns a list of the form (NEXT-BUFFER KILLED).
+NEXT-BUFFER is another server buffer, as a suggestion for what to select next,
+or nil.  KILLED is t if we killed BUFFER (because it was a temp file)."
   (let ((running (eq (process-status server-process) 'run))
        (next-buffer nil)
+       (killed nil)
+       (first t)
        (old-clients server-clients))
     (while old-clients
       (let ((client (car old-clients)))
        (or next-buffer 
            (setq next-buffer (nth 1 (memq buffer client))))
        (delq buffer client)
+       ;; Delete all dead buffers from CLIENT.
+       (let ((tail client))
+         (while tail
+           (and (bufferp (car tail))
+                (null (buffer-name (car tail)))
+                (delq (car tail) client))
+           (setq tail (cdr tail))))
        ;; If client now has no pending buffers,
        ;; tell it that it is done, and forget it entirely.
        (if (cdr client) nil
          (if running
              (progn
+               ;; Don't send emacsserver two commands in close succession.
+               ;; It cannot handle that.
+               (or first (sit-for 1))
+               (setq first nil)
                (send-string server-process 
                             (format "Close: %s Done\n" (car client)))
                (server-log (format "Close: %s Done\n" (car client)))))
          (setq server-clients (delq client server-clients))))
       (setq old-clients (cdr old-clients)))
-    (if (buffer-name buffer)
+    (if (and (bufferp buffer) (buffer-name buffer))
        (progn
          (save-excursion
            (set-buffer buffer)
            (setq server-buffer-clients nil)
            (run-hooks 'server-done-hook))
          (if (server-temp-file-p buffer)
-             (kill-buffer buffer)
+             (progn (kill-buffer buffer)
+                    (setq killed t))
            (bury-buffer buffer))))
-    next-buffer))
+    (list next-buffer killed)))
 
 (defun server-temp-file-p (buffer)
   "Return non-nil if BUFFER contains a file considered temporary.
@@ -279,7 +310,9 @@ are considered temporary."
 
 (defun server-done ()
   "Offer to save current buffer, mark it as \"done\" for clients.
-Then bury it, and return a suggested buffer to select next."
+This buries the buffer, then returns a list of the form (NEXT-BUFFER KILLED).
+NEXT-BUFFER is another server buffer, as a suggestion for what to select next,
+or nil.  KILLED is t if we killed the BUFFER (because it was a temp file)."
   (let ((buffer (current-buffer)))
     (if server-buffer-clients
        (progn
@@ -337,11 +370,15 @@ starts server process and that is all.  Invoked by \\[server-edit]."
          (not server-process)
          (memq (process-status server-process) '(signal exit)))
       (server-start nil)
-    (server-switch-buffer (server-done))))
+    (apply 'server-switch-buffer (server-done))))
 
-(defun server-switch-buffer (next-buffer)
+(defun server-switch-buffer (&optional next-buffer killed-one)
   "Switch to another buffer, preferably one that has a client.
 Arg NEXT-BUFFER is a suggestion; if it is a live buffer, use it."
+  ;; KILLED-ONE is t in a recursive call
+  ;; if we have already killed one temp-file server buffer.
+  ;; This means we should avoid the final "switch to some other buffer"
+  ;; since we've already effectively done that.
   (cond ((and (windowp server-window)
              (window-live-p server-window))
         (select-window server-window))
@@ -364,11 +401,12 @@ Arg NEXT-BUFFER is a suggestion; if it is a live buffer, use it."
        ;; If NEXT-BUFFER is a dead buffer,
        ;; remove the server records for it
        ;; and try the next surviving server buffer.
-       (server-switch-buffer
-        (server-buffer-done next-buffer)))
+       (apply 'server-switch-buffer
+              (server-buffer-done next-buffer)))
     (if server-clients
-       (server-switch-buffer (nth 1 (car server-clients)))
-      (switch-to-buffer (other-buffer)))))
+       (server-switch-buffer (nth 1 (car server-clients)) killed-one)
+      (if (not killed-one)
+         (switch-to-buffer (other-buffer))))))
 
 (global-set-key "\C-x#" 'server-edit)
 \f