Re: [elpa] 01/03: [gnugo frolic int] Regularize keymap decl + init.

[Thien-Thi Nguyen <ttn@gnu.org>]

[-- Attachment #1: Type: text/plain, Size: 1270 bytes --]

() Stefan Monnier <monnier@IRO.UMontreal.CA>
() Wed, 09 Apr 2014 09:45:48 -0400

   > +(unless gnugo-frolic-mode-map
   > +  (setq gnugo-frolic-mode-map (make-sparse-keymap))
   > +  [...])

   Any reason not to use the standard

      (defvar gnugo-frolic-mode-map
        (let ((map (make-sparse-keymap)))
          [...]
          map))

   Or even using pcase-dolist:

      (defvar gnugo-frolic-mode-map
        (let ((map (make-sparse-keymap)))
          (suppress-keymap map)
          (pcase-dolist [...])
          map))

Well, the latter is still scary to this pre-‘dolist’-w/o-cl.el dinosaur,
although it certainly looks sveltely attractive (modulo backticks, but i
suppose they are unavoidable for ‘pcase’).

More disturbing is seeing a huge form early in the file, w/ many forward
references; i prefer to maintain top-level forms topo-sorted as much as
possible (surely a sign of C/Scheme braindamage -- call a doctor!).

But anyway, i'll stew on it for a while; thanks for the suggestion.

-- 
Thien-Thi Nguyen
   GPG key: 4C807502
   (if you're human and you know it)
      read my lisp: (responsep (questions 'technical)
                               (not (via 'mailing-list)))
                     => nil

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]