GNU bug report logs - #33263
27.0.50; Tidying up Gnus modes

Previous Next

Full log

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

From: Noam Postavsky <npostavs <at> gmail.com>
To: Eric Abrahamsen <eric <at> ericabrahamsen.net>
Cc: 33263 <at> debbugs.gnu.org
Subject: Re: bug#33263: 27.0.50; Tidying up Gnus modes
Date: Tue, 06 Nov 2018 08:27:56 -0500

Eric Abrahamsen <eric <at> ericabrahamsen.net> writes:

> Noam Postavsky <npostavs <at> gmail.com> writes:
>
>> Eric Abrahamsen <eric <at> ericabrahamsen.net> writes:
>>
>>> +(defvar gnus-browse-mode-map (make-keymap))
>>>
>>>  (unless gnus-browse-mode-map
>>> -  (setq gnus-browse-mode-map (make-keymap))
>>> +  (defvar gnus-browse-mode-map (make-keymap))
>>
>> This doesn't look right.
>
> Hmm, you're right, I think I got confused there. TBH I don't really know
> why these things are wrapped in `unless', but I think what should have
> happened is that `gnus-browse-mode-map' is first defvar'ed to nil, and
> later setq'ed to (make-keymap).

The usual idiom for this is

    (defvar gnus-browse-mode-map
      (let ((map (make-sparse-keymap)))
        (define-key map ...)
        ...
        map))

It seems that (the expansion of) gnus-define-keys relies on dynamic
binding of the map variable (i.e., it must be called after the defvar,
not inside), so that's the reason to split in two parts.

Previous Next

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