Re: Toggle EOL

["Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu>]

> > 
> > I think that the EOL information in the mode-line should be
> > better separated from the coding-system info.  The patch below
> > gives it its own help-echo and its own keymap (mouse-3 switches
> > between the three different possible EOL).
> > 
> > Any comment/objection ?
> 
> I like it.  But is the patch correct?  My patch program complained
> that the last hunk was corrupted.  And I think this line:
> +    (:eval (mode-line-eol-desc))))

Yes, there was a typo and I fiddle with the patch by hand :-(
This one should work better,


	Stefan

diff -u -u -b -r1.100 bindings.el
--- bindings.el	16 Apr 2002 14:55:13 -0000	1.100
+++ bindings.el	28 Jun 2002 20:41:28 -0000
@@ -135,6 +135,38 @@
   "Local keymap for the coding-system part of the mode line.")
 
 
+(defvar mode-line-eol-map
+  (let ((map (make-sparse-keymap)))
+    (define-key map [mode-line mouse-3]
+      (lambda ()
+	(interactive)
+	(let ((eol (coding-system-eol-type buffer-file-coding-system)))
+	  (set-buffer-file-coding-system
+	   (cond ((eq eol 0) 'dos) ((eq eol 1) 'mac) (t 'unix))))))
+    (purecopy map))
+  "Local keymap for the `coding-system-eol-type' part of the mode line.")
+
+(defvar mode-line-eol-desc-cache nil)
+
+(defun mode-line-eol-desc ()
+  (let* ((eol (coding-system-eol-type buffer-file-coding-system))
+	 (mnemonic (coding-system-eol-type-mnemonic buffer-file-coding-system))
+	 (desc (assq eol mode-line-eol-desc-cache)))
+    (if (and desc (eq (cadr desc) mnemonic))
+	(cddr desc)
+      (if desc (setq mode-line-eol-desc-cache nil)) ;Flush the cache if stale.
+      (setq desc
+	    (propertize
+	     mnemonic
+	     'help-echo (format "%s end-of-line; mouse-3 to cycle"
+				(if (eq eol 0) "Unix-style LF"
+				  (if (eq eol 1) "Dos-style CRLF"
+				    (if (eq eol 2) "Mac-style CR"
+				      "Undecided"))))
+	     'keymap mode-line-eol-map))
+      (push (cons eol (cons mnemonic desc)) mode-line-eol-desc-cache)
+      desc)))
+
 (defvar mode-line-mule-info
   `(""
     (current-input-method
@@ -145,7 +177,7 @@
 			     ".  mouse-2: disable, mouse-3: describe")
 		  local-map ,mode-line-input-method-map))
     ,(propertize
-      "%Z"
+      "%z"
       'help-echo
       #'(lambda (window object point)
 	  (with-current-buffer (window-buffer window)
@@ -157,7 +189,8 @@
 			  " buffer; mouse-3: describe coding system")
 		(concat "Unibyte " (symbol-name buffer-file-coding-system)
 			" buffer")))))
-      'local-map mode-line-coding-system-map))
+      'local-map mode-line-coding-system-map)
+    (:eval (mode-line-eol-desc)))
   "Mode-line control for displaying information of multilingual environment.
 Normally it displays current input method (if any activated) and
 mnemonics of the following coding systems: