(checkdoc-output-mode-map, checkdoc-find-error-mouse, checkdoc-find-error):

Use posn-set-point to merge the mouse and non-mouse case.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index b3cada9ab065f17e9e1626f3837b0ff2fb3c6e1a..d7c2e3d31446b72b1b4a8beaa07592040bbce593 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,25 @@
+2004-04-14  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * pcvs-info.el (cvs-status-map): Update binding name.
+
+       * pcvs.el (cvs-mode-toggle-mark): Rename from cvs-mouse-toggle-mark.
+       Make it work for non-mouse events.
+
+       * subr.el (posn-set-point): New function.
+
+       * mouse.el (mouse-set-point): Use it.
+
+       * progmodes/compile.el (compile-goto-error): Use it.
+
+       * smerge-mode.el (smerge-popup-context-menu): Use it.
+
+       * emacs-lisp/checkdoc.el (checkdoc-output-mode-map)
+       (checkdoc-find-error-mouse, checkdoc-find-error): Use it to merge the
+       mouse and non-mouse case.
+
+       * diff-mode.el (diff-goto-source): Make it work for mouse bindings.
+       (diff-mouse-goto-source): Make it an alias of diff-goto-source.
+
 2004-04-14  Jesper Harder  <harder@ifa.au.dk>
 
        * info.el (info-apropos): Don't clobber Info-history-list.

diff --git a/lisp/emacs-lisp/checkdoc.el b/lisp/emacs-lisp/checkdoc.el
index 8e68cb428dc99ac6bb42e3403ebf7270cb8e5310..fddab94dfd49990f9377babbd6fa5e2be3e04f47 100644 (file)
--- a/lisp/emacs-lisp/checkdoc.el
+++ b/lisp/emacs-lisp/checkdoc.el
@@ -653,8 +653,7 @@ style."
                        (sit-for 2))
                    (setq err-list (cdr err-list))))
                (beginning-of-defun)
-               (let ((pe (car err-list))
-                     (ne (funcall findfunc nil)))
+               (let ((ne (funcall findfunc nil)))
                  (if ne
                      (setq err-list (cons ne err-list))
                    (cond ((not err-list)
@@ -900,7 +899,7 @@ Prefix argument TAKE-NOTES means to continue through the whole buffer and
 save warnings in a separate buffer.  Second optional argument START-POINT
 is the starting location.  If this is nil, `point-min' is used instead."
   (interactive "P")
-  (let ((wrong nil) (msg nil) (errors nil)
+  (let ((wrong nil) (msg nil)
        ;; Assign a flag to spellcheck flag
        (checkdoc-spellcheck-documentation-flag
         (car (memq checkdoc-spellcheck-documentation-flag
@@ -2598,14 +2597,13 @@ This function will not modify `match-data'."
   (setq checkdoc-output-mode-map (make-sparse-keymap))
   (if (not (string-match "XEmacs" emacs-version))
       (define-key checkdoc-output-mode-map [mouse-2]
-       'checkdoc-find-error-mouse))
+       'checkdoc-find-error))
   (define-key checkdoc-output-mode-map "\C-c\C-c" 'checkdoc-find-error)
   (define-key checkdoc-output-mode-map "\C-m" 'checkdoc-find-error))
 
 (defun checkdoc-output-mode ()
   "Create and setup the buffer used to maintain checkdoc warnings.
-\\<checkdoc-output-mode-map>\\[checkdoc-find-error]  - Go to this error location
-\\[checkdoc-find-error-mouse] - Goto the error clicked on."
+\\<checkdoc-output-mode-map>\\[checkdoc-find-error]  - Go to this error location."
   (if (get-buffer checkdoc-diagnostic-buffer)
       (get-buffer checkdoc-diagnostic-buffer)
     (save-excursion
@@ -2619,16 +2617,11 @@ This function will not modify `match-data'."
       (run-hooks 'checkdoc-output-mode-hook)
       (current-buffer))))
 
-(defun checkdoc-find-error-mouse (e)
-  ;; checkdoc-params: (e)
-  "Call `checkdoc-find-error' where the user clicks the mouse."
-  (interactive "e")
-  (mouse-set-point e)
-  (checkdoc-find-error))
-
-(defun checkdoc-find-error ()
+(defalias 'checkdoc-find-error-mouse 'checkdoc-find-error)
+(defun checkdoc-find-error (&optional event)
   "In a checkdoc diagnostic buffer, find the error under point."
-  (interactive)
+  (interactive (list last-input-event))
+  (if event (posn-set-point (event-end e)))
   (beginning-of-line)
   (if (looking-at "\\(\\(\\w+\\|\\s_\\)+\\.el\\):\\([0-9]+\\):")
       (let ((l (string-to-int (match-string 3)))