Revision: miles@gnu.org--gnu-2005/emacs--cvs-trunk--0--patch-417
[bpt/emacs.git] / lisp / apropos.el
index fa3be6f..b9d7e3f 100644 (file)
@@ -1,6 +1,6 @@
 ;;; apropos.el --- apropos commands for users and programmers
 
-;; Copyright (C) 1989, 1994, 1995, 2001 Free Software Foundation, Inc.
+;; Copyright (C) 1989,94,1995,2001,02,03,04,2005  Free Software Foundation, Inc.
 
 ;; Author: Joe Wells <jbw@bigbird.bu.edu>
 ;; Rewritten: Daniel Pfeiffer <occitan@esperanto.org>
@@ -58,6 +58,7 @@
 ;;; Code:
 
 (require 'button)
+(eval-when-compile (require 'cl))
 
 (defgroup apropos nil
   "Apropos commands for users and programmers"
@@ -95,13 +96,18 @@ turns off mouse highlighting."
   :group 'apropos
   :type 'face)
 
-(defcustom apropos-match-face 'secondary-selection
+(defcustom apropos-match-face 'match
   "*Face for matching text in Apropos documentation/value, or nil for none.
 This applies when you look for matches in the documentation or variable value
 for the regexp; the part that matches gets displayed in this font."
   :group 'apropos
   :type 'face)
 
+(defcustom apropos-sort-by-scores nil
+  "*Non-nil means sort matches by scores; best match is shown first.
+The computed score is shown for each match."
+  :group 'apropos
+  :type 'boolean)
 
 (defvar apropos-mode-map
   (let ((map (make-sparse-keymap)))
@@ -122,6 +128,12 @@ for the regexp; the part that matches gets displayed in this font."
 (defvar apropos-regexp nil
   "Regexp used in current apropos run.")
 
+(defvar apropos-orig-regexp nil
+  "Regexp as entered by user.")
+
+(defvar apropos-all-regexp nil
+  "Regexp matching apropos-all-words.")
+
 (defvar apropos-files-scanned ()
   "List of elc files already scanned in current run of `apropos-documentation'.")
 
@@ -131,12 +143,27 @@ for the regexp; the part that matches gets displayed in this font."
 (defvar apropos-item ()
   "Current item in or for `apropos-accumulator'.")
 
+(defvar apropos-synonyms '(
+  ("find" "open" "edit")
+  ("kill" "cut")
+  ("yank" "paste"))
+  "List of synonyms known by apropos.
+Each element is a list of words where the first word is the standard emacs
+term, and the rest of the words are alternative terms.")
+
+(defvar apropos-words ()
+  "Current list of words.")
+
+(defvar apropos-all-words ()
+  "Current list of words and synonyms.")
+
 \f
 ;;; Button types used by apropos
 
 (define-button-type 'apropos-symbol
   'face apropos-symbol-face
   'help-echo "mouse-2, RET: Display more help on this symbol"
+  'follow-link t
   'action #'apropos-symbol-button-display-help
   'skip t)
 
@@ -148,20 +175,25 @@ for the regexp; the part that matches gets displayed in this font."
 
 (define-button-type 'apropos-function
   'apropos-label "Function"
+  'help-echo "mouse-2, RET: Display more help on this function"
+  'follow-link t
   'action (lambda (button)
-           (describe-function (button-get button 'apropos-symbol)))
-  'help-echo "mouse-2, RET: Display more help on this function")
+           (describe-function (button-get button 'apropos-symbol))))
+
 (define-button-type 'apropos-macro
   'apropos-label "Macro"
+  'help-echo "mouse-2, RET: Display more help on this macro"
+  'follow-link t
   'action (lambda (button)
-           (describe-function (button-get button 'apropos-symbol)))
-  'help-echo "mouse-2, RET: Display more help on this macro")
+           (describe-function (button-get button 'apropos-symbol))))
+
 (define-button-type 'apropos-command
   'apropos-label "Command"
+  'help-echo "mouse-2, RET: Display more help on this command"
+  'follow-link t
   'action (lambda (button)
-           (describe-function (button-get button 'apropos-symbol)))
-  'help-echo "mouse-2, RET: Display more help on this command")
-  
+           (describe-function (button-get button 'apropos-symbol))))
+
 ;; We used to use `customize-variable-other-window' instead for a
 ;; customizable variable, but that is slow.  It is better to show an
 ;; ordinary help buffer and let the user click on the customization
@@ -170,31 +202,36 @@ for the regexp; the part that matches gets displayed in this font."
 (define-button-type 'apropos-variable
   'apropos-label "Variable"
   'help-echo "mouse-2, RET: Display more help on this variable"
+  'follow-link t
   'action (lambda (button)
            (describe-variable (button-get button 'apropos-symbol))))
 
 (define-button-type 'apropos-face
   'apropos-label "Face"
   'help-echo "mouse-2, RET: Display more help on this face"
+  'follow-link t
   'action (lambda (button)
            (describe-face (button-get button 'apropos-symbol))))
 
 (define-button-type 'apropos-group
   'apropos-label "Group"
   'help-echo "mouse-2, RET: Display more help on this group"
+  'follow-link t
   'action (lambda (button)
-           (customize-variable-other-window
+           (customize-group-other-window
             (button-get button 'apropos-symbol))))
 
 (define-button-type 'apropos-widget
   'apropos-label "Widget"
   'help-echo "mouse-2, RET: Display more help on this widget"
+  'follow-link t
   'action (lambda (button)
            (widget-browse-other-window (button-get button 'apropos-symbol))))
 
 (define-button-type 'apropos-plist
   'apropos-label "Plist"
   'help-echo "mouse-2, RET: Display more help on this plist"
+  'follow-link t
   'action (lambda (button)
            (apropos-describe-plist (button-get button 'apropos-symbol))))
 
@@ -219,7 +256,110 @@ before finding a label."
     (and label button)))
 
 \f
-;;;###autoload
+(defun apropos-words-to-regexp (words wild)
+  "Make regexp matching any two of the words in WORDS."
+  (concat "\\("
+         (mapconcat 'identity words "\\|")
+         "\\)"
+         (if (cdr words)
+             (concat wild
+                     "\\("
+                     (mapconcat 'identity words "\\|")
+                     "\\)")
+           "")))
+
+(defun apropos-rewrite-regexp (regexp)
+  "Rewrite a list of words to a regexp matching all permutations.
+If REGEXP is already a regexp, don't modify it."
+  (setq apropos-orig-regexp regexp)
+  (setq apropos-words () apropos-all-words ())
+  (if (string-equal (regexp-quote regexp) regexp)
+      ;; We don't actually make a regexp matching all permutations.
+      ;; Instead, for e.g. "a b c", we make a regexp matching
+      ;; any combination of two or more words like this:
+      ;; (a|b|c).*(a|b|c) which may give some false matches,
+      ;; but as long as it also gives the right ones, that's ok.
+      (let ((words (split-string regexp "[ \t]+")))
+       (dolist (word words)
+         (let ((syn apropos-synonyms) (s word) (a word))
+           (while syn
+             (if (member word (car syn))
+                 (progn
+                   (setq a (mapconcat 'identity (car syn) "\\|"))
+                   (if (member word (cdr (car syn)))
+                       (setq s a))
+                   (setq syn nil))
+               (setq syn (cdr syn))))
+           (setq apropos-words (cons s apropos-words)
+                 apropos-all-words (cons a apropos-all-words))))
+       (setq apropos-all-regexp (apropos-words-to-regexp apropos-all-words ".+"))
+       (apropos-words-to-regexp apropos-words ".*?"))
+    (setq apropos-all-regexp regexp)))
+
+(defun apropos-calc-scores (str words)
+  "Return apropos scores for string STR matching WORDS.
+Value is a list of offsets of the words into the string."
+  (let ((scores ())
+       i)
+    (if words
+       (dolist (word words scores)
+         (if (setq i (string-match word str))
+             (setq scores (cons i scores))))
+      ;; Return list of start and end position of regexp
+      (string-match apropos-regexp str)
+      (list (match-beginning 0) (match-end 0)))))
+
+(defun apropos-score-str (str)
+  "Return apropos score for string STR."
+  (if str
+      (let* (
+            (l (length str))
+            (score (- (/ l 10)))
+           i)
+       (dolist (s (apropos-calc-scores str apropos-all-words) score)
+         (setq score (+ score 1000 (/ (* (- l s) 1000) l)))))
+      0))
+
+(defun apropos-score-doc (doc)
+  "Return apropos score for documentation string DOC."
+  (let ((l (length doc)))
+    (if (> l 0)
+       (let ((score 0)
+             i)
+         (dolist (s (apropos-calc-scores doc apropos-all-words) score)
+           (setq score (+ score 50 (/ (* (- l s) 50) l)))))
+      0)))
+
+(defun apropos-score-symbol (symbol &optional weight)
+  "Return apropos score for SYMBOL."
+  (setq symbol (symbol-name symbol))
+  (let ((score 0)
+       (l (length symbol))
+       i)
+    (dolist (s (apropos-calc-scores symbol apropos-words) (* score (or weight 3)))
+      (setq score (+ score (- 60 l) (/ (* (- l s) 60) l))))))
+
+(defun apropos-true-hit (str words)
+  "Return t if STR is a genuine hit.
+This may fail if only one of the keywords is matched more than once.
+This requires that at least 2 keywords (unless only one was given)."
+  (or (not str)
+      (not words)
+      (not (cdr words))
+      (> (length (apropos-calc-scores str words)) 1)))
+
+(defun apropos-false-hit-symbol (symbol)
+  "Return t if SYMBOL is not really matched by the current keywords."
+  (not (apropos-true-hit (symbol-name symbol) apropos-words)))
+
+(defun apropos-false-hit-str (str)
+  "Return t if STR is not really matched by the current keywords."
+  (not (apropos-true-hit str apropos-words)))
+
+(defun apropos-true-hit-doc (doc)
+  "Return t if DOC is really matched by the current keywords."
+  (apropos-true-hit doc apropos-all-words))
+
 (define-derived-mode apropos-mode fundamental-mode "Apropos"
   "Major mode for following hyperlinks in output of apropos commands.
 
@@ -235,7 +375,7 @@ normal variables."
                               (if (or current-prefix-arg apropos-do-all)
                                  "variable"
                                "user option")
-                              " (regexp): "))
+                              " (regexp or words): "))
                      current-prefix-arg))
   (apropos-command regexp nil
                   (if (or do-all apropos-do-all)
@@ -246,7 +386,7 @@ normal variables."
 
 ;; For auld lang syne:
 ;;;###autoload
-(fset 'command-apropos 'apropos-command)
+(defalias 'command-apropos 'apropos-command)
 ;;;###autoload
 (defun apropos-command (apropos-regexp &optional do-all var-predicate)
   "Show commands (interactively callable functions) that match APROPOS-REGEXP.
@@ -260,8 +400,9 @@ satisfy the predicate VAR-PREDICATE."
                                   (if (or current-prefix-arg
                                           apropos-do-all)
                                       "or function ")
-                                  "(regexp): "))
+                                  "(regexp or words): "))
                     current-prefix-arg))
+  (setq apropos-regexp (apropos-rewrite-regexp apropos-regexp))
   (let ((message
         (let ((standard-output (get-buffer-create "*Apropos*")))
           (print-help-return-message 'identity))))
@@ -272,96 +413,109 @@ satisfy the predicate VAR-PREDICATE."
                                (if do-all 'functionp 'commandp))))
     (let ((tem apropos-accumulator))
       (while tem
-       (if (get (car tem) 'apropos-inhibit)
+       (if (or (get (car tem) 'apropos-inhibit)
+               (apropos-false-hit-symbol (car tem)))
            (setq apropos-accumulator (delq (car tem) apropos-accumulator)))
        (setq tem (cdr tem))))
     (let ((p apropos-accumulator)
-         doc symbol)
+         doc symbol score)
       (while p
        (setcar p (list
                   (setq symbol (car p))
+                  (setq score (apropos-score-symbol symbol))
                   (unless var-predicate
                     (if (functionp symbol)
                         (if (setq doc (documentation symbol t))
-                            (substring doc 0 (string-match "\n" doc))
+                            (progn
+                              (setq score (+ score (apropos-score-doc doc)))
+                              (substring doc 0 (string-match "\n" doc)))
                           "(not documented)")))
                   (and var-predicate
                        (funcall var-predicate symbol)
                        (if (setq doc (documentation-property
                                       symbol 'variable-documentation t))
-                           (substring doc 0
-                                      (string-match "\n" doc))))))
+                            (progn
+                              (setq score (+ score (apropos-score-doc doc)))
+                              (substring doc 0
+                                         (string-match "\n" doc)))))))
+       (setcar (cdr (car p)) score)
        (setq p (cdr p))))
     (and (apropos-print t nil)
         message
         (message message))))
 
 
+;;;###autoload
+(defun apropos-documentation-property (symbol property raw)
+  "Like (documentation-property SYMBOL PROPERTY RAW) but handle errors."
+  (condition-case ()
+      (let ((doc (documentation-property symbol property raw)))
+       (if doc (substring doc 0 (string-match "\n" doc))
+         "(not documented)"))
+    (error "(error retrieving documentation)")))
+
+
 ;;;###autoload
 (defun apropos (apropos-regexp &optional do-all)
   "Show all bound symbols whose names match APROPOS-REGEXP.
 With optional prefix DO-ALL or if `apropos-do-all' is non-nil, also
 show unbound symbols and key bindings, which is a little more
 time-consuming.  Returns list of symbols and documentation found."
-  (interactive "sApropos symbol (regexp): \nP")
-  (setq apropos-accumulator
-       (apropos-internal apropos-regexp
+  (interactive "sApropos symbol (regexp or words): \nP")
+  (setq apropos-regexp (apropos-rewrite-regexp apropos-regexp))
+  (apropos-symbols-internal
+   (apropos-internal apropos-regexp
                          (and (not do-all)
                               (not apropos-do-all)
                               (lambda (symbol)
                                 (or (fboundp symbol)
                                     (boundp symbol)
                                     (facep symbol)
-                                    (symbol-plist symbol))))))
-  (let ((tem apropos-accumulator))
-    (while tem
-      (if (get (car tem) 'apropos-inhibit)
-         (setq apropos-accumulator (delq (car tem) apropos-accumulator)))
-      (setq tem (cdr tem))))
-  (let ((p apropos-accumulator)
-       symbol doc properties)
-    (while p
-      (setcar p (list
-                (setq symbol (car p))
-                (when (fboundp symbol)
-                  (if (setq doc (condition-case nil
-                                    (documentation symbol t)
-                                  (void-function
-                                   "(alias for undefined function)")))
-                      (substring doc 0 (string-match "\n" doc))
-                    "(not documented)"))
-                (when (boundp symbol)
-                  (if (setq doc (documentation-property
-                                 symbol 'variable-documentation t))
-                      (substring doc 0 (string-match "\n" doc))
-                    "(not documented)"))
+                                    (symbol-plist symbol)))))
+   (or do-all apropos-do-all)))
+
+(defun apropos-symbols-internal (symbols keys &optional text)
+  ;; Filter out entries that are marked as apropos-inhibit.
+  (let ((all nil))
+    (dolist (symbol symbols)
+      (unless (get symbol 'apropos-inhibit)
+       (push symbol all)))
+    (setq symbols all))
+  (let ((apropos-accumulator
+        (mapcar
+         (lambda (symbol)
+           (let (doc properties)
+             (list
+              symbol
+              (apropos-score-symbol symbol)
+              (when (fboundp symbol)
+                (if (setq doc (condition-case nil
+                                  (documentation symbol t)
+                                (void-function
+                                 "(alias for undefined function)")
+                                (error
+                                 "(can't retrieve function documentation)")))
+                    (substring doc 0 (string-match "\n" doc))
+                  "(not documented)"))
+              (when (boundp symbol)
+                (apropos-documentation-property
+                   symbol 'variable-documentation t))
                 (when (setq properties (symbol-plist symbol))
                   (setq doc (list (car properties)))
                   (while (setq properties (cdr (cdr properties)))
                     (setq doc (cons (car properties) doc)))
                   (mapconcat #'symbol-name (nreverse doc) " "))
                 (when (get symbol 'widget-type)
-                  (if (setq doc (documentation-property
-                                 symbol 'widget-documentation t))
-                      (substring doc 0
-                                 (string-match "\n" doc))
-                    "(not documented)"))
-                (when (facep symbol)
-                  (if (setq doc (documentation-property
-                                 symbol 'face-documentation t))
-                      (substring doc 0
-                                 (string-match "\n" doc))
-                    "(not documented)"))
-                (when (get symbol 'custom-group)
-                  (if (setq doc (documentation-property
-                                 symbol 'group-documentation t))
-                      (substring doc 0
-                                 (string-match "\n" doc))
-                    "(not documented)"))))
-      (setq p (cdr p))))
-  (apropos-print
-   (or do-all apropos-do-all)
-   nil))
+                  (apropos-documentation-property
+                   symbol 'widget-documentation t))
+              (when (facep symbol)
+                (apropos-documentation-property
+                 symbol 'face-documentation t))
+              (when (get symbol 'custom-group)
+                  (apropos-documentation-property
+                   symbol 'group-documentation t)))))
+         symbols)))
+    (apropos-print keys nil text)))
 
 
 ;;;###autoload
@@ -370,23 +524,37 @@ time-consuming.  Returns list of symbols and documentation found."
 With optional prefix DO-ALL or if `apropos-do-all' is non-nil, also looks
 at the function and at the names and values of properties.
 Returns list of symbols and values found."
-  (interactive "sApropos value (regexp): \nP")
+  (interactive "sApropos value (regexp or words): \nP")
+  (setq apropos-regexp (apropos-rewrite-regexp apropos-regexp))
   (or do-all (setq do-all apropos-do-all))
   (setq apropos-accumulator ())
    (let (f v p)
      (mapatoms
       (lambda (symbol)
        (setq f nil v nil p nil)
-       (or (memq symbol '(apropos-regexp do-all apropos-accumulator
-                                         symbol f v p))
+       (or (memq symbol '(apropos-regexp
+                          apropos-orig-regexp apropos-all-regexp
+                          apropos-words apropos-all-words
+                          do-all apropos-accumulator
+                          symbol f v p))
            (setq v (apropos-value-internal 'boundp symbol 'symbol-value)))
        (if do-all
            (setq f (apropos-value-internal 'fboundp symbol 'symbol-function)
                  p (apropos-format-plist symbol "\n    " t)))
+       (if (apropos-false-hit-str v)
+           (setq v nil))
+       (if (apropos-false-hit-str f)
+           (setq f nil))
+       (if (apropos-false-hit-str p)
+           (setq p nil))
        (if (or f v p)
-           (setq apropos-accumulator (cons (list symbol f v p)
+           (setq apropos-accumulator (cons (list symbol
+                                                 (+ (apropos-score-str f)
+                                                    (apropos-score-str v)
+                                                    (apropos-score-str p))
+                                                 f v p)
                                            apropos-accumulator))))))
-  (apropos-print nil t))
+  (apropos-print nil "\n----------------\n"))
 
 
 ;;;###autoload
@@ -396,11 +564,12 @@ With optional prefix DO-ALL or if `apropos-do-all' is non-nil, also use
 documentation that is not stored in the documentation file and show key
 bindings.
 Returns list of symbols and documentation found."
-  (interactive "sApropos documentation (regexp): \nP")
+  (interactive "sApropos documentation (regexp or words): \nP")
+  (setq apropos-regexp (apropos-rewrite-regexp apropos-regexp))
   (or do-all (setq do-all apropos-do-all))
   (setq apropos-accumulator () apropos-files-scanned ())
   (let ((standard-input (get-buffer-create " apropos-temp"))
-       f v)
+       f v sf sv)
     (unwind-protect
        (save-excursion
          (set-buffer standard-input)
@@ -413,18 +582,26 @@ Returns list of symbols and documentation found."
                 (if (integerp v) (setq v))
                 (setq f (apropos-documentation-internal f)
                       v (apropos-documentation-internal v))
+                (setq sf (apropos-score-doc f)
+                      sv (apropos-score-doc v))
                 (if (or f v)
                     (if (setq apropos-item
                               (cdr (assq symbol apropos-accumulator)))
                         (progn
                           (if f
-                              (setcar apropos-item f))
+                              (progn
+                                (setcar (nthcdr 1 apropos-item) f)
+                                (setcar apropos-item (+ (car apropos-item) sf))))
                           (if v
-                              (setcar (cdr apropos-item) v)))
+                              (progn
+                                (setcar (nthcdr 2 apropos-item) v)
+                                (setcar apropos-item (+ (car apropos-item) sv)))))
                       (setq apropos-accumulator
-                            (cons (list symbol f v)
+                            (cons (list symbol
+                                        (+ (apropos-score-symbol symbol 2) sf sv)
+                                        f v)
                                   apropos-accumulator)))))))
-         (apropos-print nil t))
+         (apropos-print nil "\n----------------\n"))
       (kill-buffer standard-input))))
 
 \f
@@ -444,7 +621,8 @@ Returns list of symbols and documentation found."
   (if (consp doc)
       (apropos-documentation-check-elc-file (car doc))
     (and doc
-        (string-match apropos-regexp doc)
+        (string-match apropos-all-regexp doc)
+        (save-match-data (apropos-true-hit-doc doc))
         (progn
           (if apropos-match-face
               (put-text-property (match-beginning 0)
@@ -488,25 +666,31 @@ Returns list of symbols and documentation found."
       (beginning-of-line 2)
       (if (save-restriction
            (narrow-to-region (point) (1- sepb))
-           (re-search-forward apropos-regexp nil t))
+           (re-search-forward apropos-all-regexp nil t))
          (progn
            (setq beg (match-beginning 0)
                  end (point))
            (goto-char (1+ sepa))
-           (or (setq type (if (eq ?F (preceding-char))
-                              1        ; function documentation
-                            2)         ; variable documentation
-                     symbol (read)
-                     beg (- beg (point) 1)
-                     end (- end (point) 1)
-                     doc (buffer-substring (1+ (point)) (1- sepb))
-                     apropos-item (assq symbol apropos-accumulator))
-               (setq apropos-item (list symbol nil nil)
-                     apropos-accumulator (cons apropos-item
-                                               apropos-accumulator)))
-           (if apropos-match-face
-               (put-text-property beg end 'face apropos-match-face doc))
-           (setcar (nthcdr type apropos-item) doc)))
+           (setq type (if (eq ?F (preceding-char))
+                          2    ; function documentation
+                        3)             ; variable documentation
+                 symbol (read)
+                 beg (- beg (point) 1)
+                 end (- end (point) 1)
+                 doc (buffer-substring (1+ (point)) (1- sepb)))
+           (when (apropos-true-hit-doc doc)
+             (or (and (setq apropos-item (assq symbol apropos-accumulator))
+                      (setcar (cdr apropos-item)
+                              (+ (cadr apropos-item) (apropos-score-doc doc))))
+                 (setq apropos-item (list symbol
+                                          (+ (apropos-score-symbol symbol 2)
+                                             (apropos-score-doc doc))
+                                          nil nil)
+                       apropos-accumulator (cons apropos-item
+                                                 apropos-accumulator)))
+             (if apropos-match-face
+                 (put-text-property beg end 'face apropos-match-face doc))
+             (setcar (nthcdr type apropos-item) doc))))
       (setq sepa (goto-char sepb)))))
 
 (defun apropos-documentation-check-elc-file (file)
@@ -525,34 +709,40 @@ Returns list of symbols and documentation found."
        (if (save-restriction
              ;; match ^ and $ relative to doc string
              (narrow-to-region beg end)
-             (re-search-forward apropos-regexp nil t))
+             (re-search-forward apropos-all-regexp nil t))
            (progn
              (goto-char (+ end 2))
              (setq doc (buffer-substring beg end)
                    end (- (match-end 0) beg)
-                   beg (- (match-beginning 0) beg)
-                   this-is-a-variable (looking-at "(def\\(var\\|const\\) ")
-                   symbol (progn
-                            (skip-chars-forward "(a-z")
-                            (forward-char)
-                            (read))
-                   symbol (if (consp symbol)
-                              (nth 1 symbol)
-                            symbol))
-             (if (if this-is-a-variable
-                     (get symbol 'variable-documentation)
-                   (and (fboundp symbol) (apropos-safe-documentation symbol)))
-                 (progn
-                   (or (setq apropos-item (assq symbol apropos-accumulator))
-                       (setq apropos-item (list symbol nil nil)
-                             apropos-accumulator (cons apropos-item
-                                                       apropos-accumulator)))
-                   (if apropos-match-face
-                       (put-text-property beg end 'face apropos-match-face
-                                          doc))
-                   (setcar (nthcdr (if this-is-a-variable 2 1)
-                                   apropos-item)
-                           doc)))))))))
+                   beg (- (match-beginning 0) beg))
+             (when (apropos-true-hit-doc doc)
+               (setq this-is-a-variable (looking-at "(def\\(var\\|const\\) ")
+                     symbol (progn
+                              (skip-chars-forward "(a-z")
+                              (forward-char)
+                              (read))
+                     symbol (if (consp symbol)
+                                (nth 1 symbol)
+                              symbol))
+               (if (if this-is-a-variable
+                       (get symbol 'variable-documentation)
+                     (and (fboundp symbol) (apropos-safe-documentation symbol)))
+                   (progn
+                     (or (and (setq apropos-item (assq symbol apropos-accumulator))
+                              (setcar (cdr apropos-item)
+                                      (+ (cadr apropos-item) (apropos-score-doc doc))))
+                         (setq apropos-item (list symbol
+                                                  (+ (apropos-score-symbol symbol 2)
+                                                     (apropos-score-doc doc))
+                                                  nil nil)
+                               apropos-accumulator (cons apropos-item
+                                                         apropos-accumulator)))
+                     (if apropos-match-face
+                         (put-text-property beg end 'face apropos-match-face
+                                            doc))
+                     (setcar (nthcdr (if this-is-a-variable 3 2)
+                                     apropos-item)
+                             doc))))))))))
 
 
 
@@ -579,18 +769,29 @@ Will return nil instead."
     function))
 
 
-(defun apropos-print (do-keys spacing)
+(defun apropos-print (do-keys spacing &optional text)
   "Output result of apropos searching into buffer `*Apropos*'.
 The value of `apropos-accumulator' is the list of items to output.
-Each element should have the format (SYMBOL FN-DOC VAR-DOC [PLIST-DOC]).
+Each element should have the format
+ (SYMBOL SCORE FN-DOC VAR-DOC [PLIST-DOC WIDGET-DOC FACE-DOC GROUP-DOC]).
 The return value is the list that was in `apropos-accumulator', sorted
 alphabetically by symbol name; but this function also sets
-`apropos-accumulator' to nil before returning."
+`apropos-accumulator' to nil before returning.
+
+If SPACING is non-nil, it should be a string; separate items with that string.
+If non-nil TEXT is a string that will be printed as a heading."
   (if (null apropos-accumulator)
-      (message "No apropos matches for `%s'" apropos-regexp)
+      (message "No apropos matches for `%s'" apropos-orig-regexp)
     (setq apropos-accumulator
-         (sort apropos-accumulator (lambda (a b)
-                                     (string-lessp (car a) (car b)))))
+         (sort apropos-accumulator
+               (lambda (a b)
+                 ;; Don't sort by score if user can't see the score.
+                 ;; It would be confusing.  -- rms.
+                 (if apropos-sort-by-scores
+                     (or (> (cadr a) (cadr b))
+                         (and (= (cadr a) (cadr b))
+                              (string-lessp (car a) (car b))))
+                   (string-lessp (car a) (car b))))))
     (with-output-to-temp-buffer "*Apropos*"
       (let ((p apropos-accumulator)
            (old-buffer (current-buffer))
@@ -598,15 +799,19 @@ alphabetically by symbol name; but this function also sets
        (set-buffer standard-output)
        (apropos-mode)
        (if (display-mouse-p)
-           (insert "If moving the mouse over text changes the text's color,\n"
-                   (substitute-command-keys
-                    "you can click \\[push-button] on that text to get more information.\n")))
+           (insert
+            "If moving the mouse over text changes the text's color, "
+            "you can click\n"
+            "mouse-2 (second button from right) on that text to "
+            "get more information.\n"))
        (insert "In this buffer, go to the name of the command, or function,"
                " or variable,\n"
                (substitute-command-keys
                 "and type \\[apropos-follow] to get full documentation.\n\n"))
+       (if text (insert text "\n\n"))
        (while (consp p)
-         (or (not spacing) (bobp) (terpri))
+         (when (and spacing (not (bobp)))
+           (princ spacing))
          (setq apropos-item (car p)
                symbol (car apropos-item)
                p (cdr p))
@@ -616,6 +821,8 @@ alphabetically by symbol name; but this function also sets
                              ;; changed the variable!
                              ;; Just say `no' to variables containing faces!
                              'face apropos-symbol-face)
+         (if apropos-sort-by-scores
+             (insert " (" (number-to-string (cadr apropos-item)) ") "))
          ;; Calculate key-bindings if we want them.
          (and do-keys
               (commandp symbol)
@@ -653,33 +860,32 @@ alphabetically by symbol name; but this function also sets
                                              key))
                       key)
                     item ", "))
-                (insert "M-x")
-                (put-text-property (- (point) 3) (point)
-                                   'face apropos-keybinding-face)
-                (insert " " (symbol-name symbol) " ")
-                (insert "RET")
-                (put-text-property (- (point) 3) (point)
-                                   'face apropos-keybinding-face)))
+                (insert "M-x ... RET")
+                (when apropos-keybinding-face
+                  (put-text-property (- (point) 11) (- (point) 8)
+                                     'face apropos-keybinding-face)
+                  (put-text-property (- (point) 3) (point)
+                                     'face apropos-keybinding-face))))
          (terpri)
-         (apropos-print-doc 1
+         (apropos-print-doc 2
                             (if (commandp symbol)
                                 'apropos-command
                               (if (apropos-macrop symbol)
                                   'apropos-macro
                                 'apropos-function))
                             t)
-         (apropos-print-doc 2 'apropos-variable t)
-         (apropos-print-doc 6 'apropos-group t)
-         (apropos-print-doc 5 'apropos-face t)
-         (apropos-print-doc 4 'apropos-widget t)
-         (apropos-print-doc 3 'apropos-plist nil))
+         (apropos-print-doc 3 'apropos-variable t)
+         (apropos-print-doc 7 'apropos-group t)
+         (apropos-print-doc 6 'apropos-face t)
+         (apropos-print-doc 5 'apropos-widget t)
+         (apropos-print-doc 4 'apropos-plist nil))
        (setq buffer-read-only t))))
   (prog1 apropos-accumulator
     (setq apropos-accumulator ())))    ; permit gc
 
 
 (defun apropos-macrop (symbol)
-  "T if SYMBOL is a Lisp macro."
+  "Return t if SYMBOL is a Lisp macro."
   (and (fboundp symbol)
        (consp (setq symbol
                    (symbol-function symbol)))
@@ -715,13 +921,15 @@ alphabetically by symbol name; but this function also sets
 
 (defun apropos-describe-plist (symbol)
   "Display a pretty listing of SYMBOL's plist."
-  (with-output-to-temp-buffer "*Help*"
+  (help-setup-xref (list 'apropos-describe-plist symbol) (interactive-p))
+  (with-output-to-temp-buffer (help-buffer)
     (set-buffer standard-output)
     (princ "Symbol ")
     (prin1 symbol)
     (princ "'s plist is\n (")
     (if apropos-symbol-face
-       (put-text-property 8 (- (point) 14) 'face apropos-symbol-face))
+       (put-text-property (+ (point-min) 7) (- (point) 14)
+                          'face apropos-symbol-face))
     (insert (apropos-format-plist symbol "\n  "))
     (princ ")")
     (print-help-return-message)))
@@ -729,4 +937,5 @@ alphabetically by symbol name; but this function also sets
 
 (provide 'apropos)
 
+;;; arch-tag: d56fa2ac-e56b-4ce3-84ff-852f9c0dc66e
 ;;; apropos.el ends here