Test for lazy-lock-mode before swapping hooks.
[bpt/emacs.git] / lisp / facemenu.el
index 5bf9e24..4e0d293 100644 (file)
@@ -1,7 +1,8 @@
-;;; facemenu.el -- Create a face menu for interactively adding fonts to text
-;; Copyright (c) 1994, 1995 Free Software Foundation, Inc.
+;;; facemenu.el --- create a face menu for interactively adding fonts to text
 
-;; Author: Boris Goldowsky <boris@cs.rochester.edu>
+;; Copyright (c) 1994, 1995, 1996 Free Software Foundation, Inc.
+
+;; Author: Boris Goldowsky <boris@gnu.ai.mit.edu>
 ;; Keywords: faces
 
 ;; This file is part of GNU Emacs.
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
+
 ;; This file defines a menu of faces (bold, italic, etc) which allows you to
 ;; set the face used for a region of the buffer.  Some faces also have
 ;; keybindings, which are shown in the menu.  Faces with names beginning with
@@ -124,13 +127,12 @@ This should be nil to put them at the top of the menu, or t to put them
 just before \"Other\" at the end.")
 
 (defvar facemenu-unlisted-faces
-  '(modeline region secondary-selection highlight scratch-face
-    font-lock-comment-face font-lock-string-face font-lock-keyword-face
-    font-lock-function-name-face font-lock-variable-name-face
-    font-lock-type-face font-lock-reference-face)
+  '(modeline region secondary-selection highlight scratch-face)
   "List of faces not to include in the Face menu.
-Set this before loading facemenu.el, or call `facemenu-update' after
-changing it.
+You can set this list before loading facemenu.el, or add a face to it before
+creating that face if you do not want it to be listed.  If you change the
+variable so as to eliminate faces that have already been added to the menu,
+call `facemenu-update' to recalculate the menu contents.
 
 If this variable is t, no faces will be added to the menu.  This is useful for
 temporarily turning off the feature that automatically adds faces to the menu
@@ -223,7 +225,7 @@ when they are created.")
   (define-key map [in] (cons "Indentation" 'facemenu-indentation-menu))
   (define-key map [ju] (cons "Justification" 'facemenu-justification-menu))
   (define-key map [s2] (list "-----------------"))
-  (define-key map [sp] (cons "Special Props" 'facemenu-special-menu))
+  (define-key map [sp] (cons "Special Properties" 'facemenu-special-menu))
   (define-key map [bg] (cons "Background Color" 'facemenu-background-menu))
   (define-key map [fg] (cons "Foreground Color" 'facemenu-foreground-menu))
   (define-key map [fc] (cons "Face" 'facemenu-face-menu)))
@@ -239,6 +241,22 @@ when they are created.")
 requested in `facemenu-keybindings'.")
 (defalias 'facemenu-keymap facemenu-keymap)
 
+
+(defvar facemenu-add-face-function nil
+  "Function called at beginning of text to change or `nil'.
+This function is passed the FACE to set and END of text to change, and must
+return a string which is inserted.  It may set `facemenu-end-add-face'.")
+
+(defvar facemenu-end-add-face nil
+  "String to insert or function called at end of text to change or `nil'.
+This function is passed the FACE to set, and must return a string which is
+inserted.")
+
+(defvar facemenu-remove-face-function nil
+  "When non-`nil' function called to remove faces.
+This function is passed the START and END of text to change.
+May also be `t' meaning to use `facemenu-add-face-function'.")
+
 ;;; Internal Variables
 
 (defvar facemenu-color-alist nil
@@ -281,7 +299,7 @@ typing a character to insert cancels the specification."
       (let ((start (or start (region-beginning)))
            (end (or end (region-end))))
        (facemenu-add-face face start end))
-    (facemenu-self-insert-face face)))
+    (facemenu-add-face face)))
 
 ;;;###autoload
 (defun facemenu-set-foreground (color &optional start end)
@@ -334,15 +352,7 @@ typing a character to insert cancels the specification."
   (facemenu-get-face face)
   (if start 
       (facemenu-add-face face start end)
-    (facemenu-self-insert-face face)))
-
-(defun facemenu-self-insert-face (face)
-  (setq self-insert-face (if (eq last-command self-insert-face-command)
-                            (cons face (if (listp self-insert-face)
-                                           self-insert-face
-                                         (list self-insert-face)))
-                          face)
-       self-insert-face-command this-command))
+    (facemenu-add-face face)))
 
 ;;;###autoload
 (defun facemenu-set-invisible (start end)
@@ -397,22 +407,42 @@ These special properties include `invisible', `intangible' and `read-only'."
 (defun list-text-properties-at (p)
   "Pop up a buffer listing text-properties at LOCATION."
   (interactive "d")
-  (let ((props (text-properties-at p)))
+  (let ((props (text-properties-at p))
+       category
+       str)
     (if (null props)
        (message "None")
-      (with-output-to-temp-buffer "*Text Properties*"
-       (princ (format "Text properties at %d:\n\n" p))
-       (while props
-         (princ (format "%-20s %S\n"
-                        (car props) (car (cdr props))))
-         (setq props (cdr (cdr props))))))))
+      (if (and (not (cdr (cdr props)))
+              (not (eq (car props) 'category))
+              (< (length (setq str (format "Text property at %d:  %s  %S"
+                                           p (car props) (car (cdr props)))))
+                 (frame-width)))
+         (message "%s" str)
+       (with-output-to-temp-buffer "*Text Properties*"
+         (princ (format "Text properties at %d:\n\n" p))
+         (while props
+           (if (eq (car props) 'category)
+               (setq category (car (cdr props))))
+           (princ (format "%-20s %S\n"
+                          (car props) (car (cdr props))))
+           (setq props (cdr (cdr props))))
+         (if category
+             (progn
+               (setq props (symbol-plist category))
+               (princ (format "\nCategory %s:\n\n" category))
+               (while props
+                 (princ (format "%-20s %S\n"
+                                (car props) (car (cdr props))))
+                 (if (eq (car props) 'category)
+                     (setq category (car (cdr props))))
+                 (setq props (cdr (cdr props)))))))))))
 
 ;;;###autoload
 (defun facemenu-read-color (&optional prompt)
   "Read a color using the minibuffer."
   (let ((col (completing-read (or prompt "Color: ") 
                              (or facemenu-color-alist
-                                 (if (eq 'x window-system)
+                                 (if window-system
                                      (mapcar 'list (x-defined-colors))))
                              nil t)))
     (if (equal "" col)
@@ -426,7 +456,7 @@ If the optional argument LIST is non-nil, it should be a list of
 colors to display.  Otherwise, this command computes a list
 of colors that the current display can handle."
   (interactive)
-  (if (and (null list) (eq 'x window-system))
+  (if (and (null list) window-system)
       (progn
        (setq list (x-defined-colors))
        ;; Delete duplicate colors.
@@ -461,55 +491,86 @@ This function queries the window-system server to find out what the
 color names mean.  It returns nil if the colors differ or if it can't
 determine the correct answer."
   (cond ((equal a b) t)
-       ((and (eq 'x window-system)
-             (equal (x-color-values a) (x-color-values b))))))
+       ((and (or (eq window-system 'x) (eq window-system 'win32))
+             (equal (x-color-values a) (x-color-values b))))
+       ((eq window-system 'pc)
+        (and (x-color-defined-p a) (x-color-defined-p b)
+             (eq (msdos-color-translate a) (msdos-color-translate b))))))
 
-(defun facemenu-add-face (face start end)
+(defun facemenu-add-face (face &optional start end)
   "Add FACE to text between START and END.
-For each section of that region that has a different face property, FACE will
-be consed onto it, and other faces that are completely hidden by that will be
-removed from the list.
+If START is `nil' or START to END is empty, add FACE to next typed character
+instead.  For each section of that region that has a different face property,
+FACE will be consed onto it, and other faces that are completely hidden by
+that will be removed from the list.
+If `facemenu-add-face-function' and maybe `facemenu-end-add-face' are non-`nil'
+they are used to set the face information.
 
 As a special case, if FACE is `default', then the region is left with NO face
 text property.  Otherwise, selecting the default face would not have any
-effect."
-  (interactive "*xFace:\nr")
-  (if (eq face 'default)
-      (remove-text-properties start end '(face default))
-    (let ((part-start start) part-end)
-      (while (not (= part-start end))
-       (setq part-end (next-single-property-change part-start 'face nil end))
-       (let ((prev (get-text-property part-start 'face)))
-         (put-text-property part-start part-end 'face
-                            (if (null prev)
-                                face
-                              (facemenu-discard-redundant-faces
-                               (cons face
-                                     (if (listp prev) prev (list prev)))))))
-       (setq part-start part-end)))))
-
-(defun facemenu-discard-redundant-faces (face-list &optional mask)
-  "Remove from FACE-LIST any faces that won't show at all.
-This means they have no non-nil elements that aren't also non-nil in an
-earlier face."
-  (let ((useful nil))
-    (cond ((null face-list) nil)
-         ((null mask)
-          (cons (car face-list)
-                (facemenu-discard-redundant-faces
-                 (cdr face-list) 
-                 (copy-sequence (internal-get-face (car face-list))))))
-         ((let ((i (length mask))
-                (face (internal-get-face (car face-list))))
-            (while (>= (setq i (1- i)) 0)
-              (if (and (aref face i)
-                       (not (aref mask i)))
-                  (progn (setq useful t)
-                         (aset mask i t))))
-            useful)
-          (cons (car face-list)
-                (facemenu-discard-redundant-faces (cdr face-list) mask)))
-         (t (facemenu-discard-redundant-faces (cdr face-list) mask)))))
+effect.  See `facemenu-remove-face-function'."
+  (interactive "*xFace: \nr")
+  (if (and (eq face 'default)
+          (not (eq facemenu-remove-face-function t)))
+      (if facemenu-remove-face-function
+         (funcall facemenu-remove-face-function start end)
+       (if (and start (< start end))
+           (remove-text-properties start end '(face default))
+         (setq self-insert-face 'default
+               self-insert-face-command this-command)))
+    (if facemenu-add-face-function
+       (save-excursion
+         (if end (goto-char end))
+         (save-excursion
+           (if start (goto-char start))
+           (insert-before-markers
+            (funcall facemenu-add-face-function face end)))
+         (if facemenu-end-add-face
+             (insert (if (stringp facemenu-end-add-face)
+                         facemenu-end-add-face
+                       (funcall facemenu-end-add-face face)))))
+      (if (and start (< start end))
+         (let ((part-start start) part-end)
+           (while (not (= part-start end))
+             (setq part-end (next-single-property-change part-start 'face
+                                                         nil end))
+             (let ((prev (get-text-property part-start 'face)))
+               (put-text-property part-start part-end 'face
+                                  (if (null prev)
+                                      face
+                                    (facemenu-active-faces
+                                     (cons face
+                                           (if (listp prev)
+                                               prev
+                                             (list prev)))))))
+             (setq part-start part-end)))
+       (setq self-insert-face (if (eq last-command self-insert-face-command)
+                                  (cons face (if (listp self-insert-face)
+                                                 self-insert-face
+                                               (list self-insert-face)))
+                                face)
+             self-insert-face-command this-command)))))
+
+(defun facemenu-active-faces (face-list &optional frame)
+  "Return from FACE-LIST those faces that would be used for display.
+This means each face attribute is not specified in a face earlier in FACE-LIST
+and such a face is therefore active when used to display text.
+If the optional argument FRAME is given, use the faces in that frame; otherwise
+use the selected frame.  If t, then the global, non-frame faces are used."
+  (let* ((mask-atts (copy-sequence (internal-get-face (car face-list) frame)))
+        (active-list (list (car face-list)))
+        (face-list (cdr face-list))
+        (mask-len (length mask-atts)))
+    (while face-list
+      (if (let ((face-atts (internal-get-face (car face-list) frame))
+               (i mask-len) (useful nil))
+           (while (> (setq i (1- i)) 1)
+             (and (aref face-atts i) (not (aref mask-atts i))
+                  (aset mask-atts i (setq useful t))))
+           useful)
+         (setq active-list (cons (car face-list) active-list)))
+      (setq face-list (cdr face-list)))
+    (nreverse active-list)))
 
 (defun facemenu-get-face (symbol)
   "Make sure FACE exists.
@@ -523,10 +584,12 @@ or nil if given a bad color."
                 (color (substring name 3)))
            (cond ((string-match "^fg:" name)
                   (set-face-foreground face color)
-                  (and (eq 'x window-system) (x-color-defined-p color)))
+                  (and window-system
+                       (x-color-defined-p color)))
                  ((string-match "^bg:" name)
                   (set-face-background face color)
-                  (and (eq 'x window-system) (x-color-defined-p color)))
+                  (and window-system
+                       (x-color-defined-p color)))
                  (t))))
       symbol))
 
@@ -569,7 +632,7 @@ Automatically called when a new face is created."
   nil) ; Return nil for facemenu-iterate
 
 (defun facemenu-complete-face-list (&optional oldlist)
-  "Return list of all faces that are look different.
+  "Return list of all faces that look different.
 Starts with given ALIST of faces, and adds elements only if they display 
 differently from any face already on the list.
 The faces on ALIST will end up at the end of the returned list, in reverse