unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Lift {global,local}-key-binding to Lisp
@ 2021-01-13 19:26 Eli Zaretskii
  2021-01-14 19:24 ` Stefan Kangas
  2021-01-15 12:09 ` Dmitry Gutov
  0 siblings, 2 replies; 28+ messages in thread
From: Eli Zaretskii @ 2021-01-13 19:26 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

  commit a9658cd5b07e88a5d413cbb4dfd8f9d9d0c8bbf5
  Author:     Stefan Kangas <stefan@marxist.se>
  AuthorDate: Wed Jan 13 18:54:09 2021 +0100
  Commit:     Stefan Kangas <stefan@marxist.se>
  CommitDate: Wed Jan 13 18:54:09 2021 +0100

      Lift {global,local}-key-binding to Lisp

      * lisp/subr.el (local-key-binding, global-key-binding): New defuns.
      * src/keymap.c (Flocal_key_binding, Fglobal_key_binding): Remove DEFUNs.
      (syms_of_keymap): Remove defsubrs for above DEFUNs.
      * test/lisp/subr-tests.el (subr-test-local-key-binding)
      (subr-test-global-key-binding): New tests.

Stefan, why are we moving these and other functions to Lisp?  Are
there any advantages to moving them?  Is there any plan behind this
and other similar changes?

And why don't we discuss such changes before making them?

In general, unless we get some significant gains, I'd prefer not to
move around code just to move it.  If nothing else, it makes it harder
for people who, like me, are familiar with the original code, to find
stuff, because suddenly it isn't where it used to be.  The result is
that I cannot use my memory anymore, I need to consult the code each
time I need to answer some question or consider something related to
this code.  It's a needless churn, and I ask myself what do we gain in
return?



^ permalink raw reply	[flat|nested] 28+ messages in thread

end of thread, other threads:[~2021-01-21 19:59 UTC | newest]

Thread overview: 28+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-13 19:26 Lift {global,local}-key-binding to Lisp Eli Zaretskii
2021-01-14 19:24 ` Stefan Kangas
2021-01-14 20:10   ` Eli Zaretskii
2021-01-14 20:24     ` Eli Zaretskii
2021-01-15  1:58     ` Leo Liu
2021-01-15  4:16     ` Óscar Fuentes
2021-01-15  7:42       ` Eli Zaretskii
2021-01-21 16:03     ` Stefan Kangas
2021-01-21 16:59       ` [External] : " Drew Adams
2021-01-21 17:50         ` Dmitry Gutov
2021-01-21 18:16           ` Drew Adams
2021-01-21 18:58             ` Dmitry Gutov
2021-01-21 19:59       ` Eli Zaretskii
2021-01-14 21:03   ` Andrea Corallo via Emacs development discussions.
2021-01-15  7:45     ` Eli Zaretskii
2021-01-15 12:09 ` Dmitry Gutov
2021-01-15 12:18   ` Eli Zaretskii
2021-01-15 13:24     ` Dmitry Gutov
2021-01-15 13:45       ` Eli Zaretskii
2021-01-15 18:09         ` Dmitry Gutov
2021-01-17 14:27           ` Christopher Miles
2021-01-15 18:03       ` Drew Adams
2021-01-16  0:51   ` Leo Liu
2021-01-17 14:33     ` Christopher Miles
2021-01-17 15:08       ` Eli Zaretskii
2021-01-18  3:29         ` Christopher Miles
2021-01-18 16:43           ` Eli Zaretskii
2021-01-17 16:10       ` Basil L. Contovounesios

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).