bug#49466: 28.0.50; C-<mouse-2> is undefined

[Stefan Monnier]

> Stefan, is there some way to make this auto-loaded?  That is, whenever
> the user presses C-mouse-2, automatically load facemenu and show the
> drop-down menu in facemenu-menu?  I tried autoloading the
> facemenu-menu keymap like ispell and pcvs do, but that doesn't seem to
> work, in the sense that pressing C-mouse-2 complains that the binding
> is not a command.

The patch below works for me, should I push it?


        Stefan "who hasn't followed this discussion"


diff --git a/lisp/facemenu.el b/lisp/facemenu.el
index 9d75302a97..8631be917a 100644
--- a/lisp/facemenu.el
+++ b/lisp/facemenu.el
@@ -244,6 +244,8 @@ facemenu-menu
   (define-key map [fc] (cons "Face" 'facemenu-face-menu)))
 (defalias 'facemenu-menu facemenu-menu)
 
+;;;###autoload (autoload 'facemenu-menu "facemenu" nil nil 'keymap)
+;;;###autoload
 (define-key global-map [C-down-mouse-2] 'facemenu-menu)
 
 (easy-menu-add-item