GNU bug report logs - #34080
27.0.50, 26.1; undocumented key bindings for *vc-diff*

Previous Next

Package: emacs;

Reported by: Van L <van <at> scratch.space>

Date: Tue, 15 Jan 2019 05:31:01 UTC

Severity: wishlist

Found in version 27.0.50

Done: Sean Whitton <spwhitton <at> spwhitton.name>

Full log


Message #53 received at 34080 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Sean Whitton <spwhitton <at> spwhitton.name>
Cc: Van L <van <at> scratch.space>, 34080 <at> debbugs.gnu.org,
 Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#34080: 27.0.50, 26.1; undocumented key bindings for *vc-diff*
Date: Tue, 08 Jul 2025 14:08:15 -0400
> The reason for this is, I think, simple: C-h m describes the major
> mode's own map, diff-mode-map, but the TAB binding does not appear in
> that map.  It's only in diff-mode-shared-map, the parent of
> diff-mode-map.

`diff-mode-shared-map` is not a parent of `diff-mode-map`.
It's a parent of the ESC submap of `diff-mode-map`, and its bindings
appear just fine, e.g.:

    C-M-i		diff-hunk-next

`C-M-i` is another name for `M-TAB`.

> We probably don't want C-h m to include bindings from the parents of
> major mode maps.

We do, and AFAIK, it does.

> But there is something about diff-mode-shared-map that I can't figure
> out.  How do its bindings get overridden/deactivated in the case that
> the buffer is not read-only?

`diff-mode-shared-map` is used as a minor-mode map, (de)activated
according to ``diff-mode-read-only`:

    (or (assq 'diff-mode-read-only minor-mode-map-alist)
        (nconc minor-mode-map-alist
               (list (cons 'diff-mode-read-only
                           (make-composed-keymap diff-mode-shared-map
                                                 special-mode-map)))))


- Stefan





This bug report was last modified 21 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.