From: Boruch Baum <boruch_baum@gmx.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: 43709@debbugs.gnu.org
Subject: bug#43709: minibuffer keybinding cheatsheet and launcher [CODE SUBMISSION]
Date: Thu, 1 Oct 2020 14:25:04 -0400 [thread overview]
Message-ID: <20201001182504.mrdldh3rdf3uuvbm@E15-2016.optimum.net> (raw)
In-Reply-To: <jwveemijcub.fsf-monnier+emacs@gnu.org>
On 2020-10-01 09:51, Stefan Monnier wrote:
> > ;; If you've ever used packages such as `ivy' or `magit', you've
> For those who haven't, it would be good to describe what your package does.
In progress...
> > (require 'cl-seq) ;; cl-member, cl-position
> Please require `cl-lib`grather than `cl-seq`
Done.
> > (let ((not-found t))
> > (mapc (lambda (x)
> > (when (string-match x (format "%s" cmd))
> > (setq not-found nil)))
> > key-assist-exclude-regexps)
> > not-found)))
>
> You should move the `(format "%s" cmd)` out of the loop (and use
> `dolist` instead of `mapc`).
Done, but I went with symbol-name instead of format.
> > ((keymapp spec)
> > (let (cmd)
> > (dolist (elem spec)
>
> Please use `map-keymap` rather than `dolist` so as to handle all the
> different keymap formats.
I had tried that originally, but map-keymap brings in data from parent
keymaps, which is undesirable in this case. There is a function
map-keymap-internal which might be a usable alternative, but it returns
a single unusable cruft element (the parent keymap) when it exists, so
the result doesn't seem much better than the current code, no?
> > (funcall (car (nth choice commands))))))
>
> This should probably be `command-execute` rather than `funcall` to fix
> the wrong-number-of-arguments error I bumped into.
Great catch. It solves the bug you found. Done.
As for posting updates to the code submission, I can do any combination of
the following, or something else, but want guidance before proceeding:
1) post to this list;
2) post to the pre-existing thread on emacs-bugs;
3) create a github repo, and post there (and gain potential wider
audience, more features, but fragment communication over two platforms);
4) create a repo on some imagined savannah / gnu server;
Finally: Tomorrow approaching sunset I go offline for about nine
days[1], so what I can squeeze in before then, great; afterwards I
welcome any continued feedback but please be patient and don't expect
any response for ~10-12 days.
But really finally this time: I've twice already made passing reference
to another package about ready to submit, that adds a lot of new
functionality to dired. I can post it now in its current state, and
inconsiderately disappear offline for 9+ days, or I could just as
inconsiderately not post it and deny you for 9+ days. I'm undecided, so
unless I hear otherwise, I'll wait.
--
[1] https://en.wikipedia.org/wiki/Sukkot
--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1 7286 0036 9E45 1595 8BC0
next prev parent reply other threads:[~2020-10-01 18:25 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-29 16:59 bug#43709: minibuffer keybinding cheatsheet and launcher [CODE SUBMISSION] Boruch Baum
2020-09-29 17:41 ` Drew Adams
2020-09-29 18:22 ` Boruch Baum
2020-09-29 19:05 ` Drew Adams
2020-09-29 18:26 ` Boruch Baum
2020-10-01 13:51 ` Stefan Monnier
2020-10-01 15:31 ` Boruch Baum
2020-10-01 16:04 ` Stefan Monnier
2020-10-01 20:46 ` Boruch Baum
2020-10-01 18:25 ` Boruch Baum [this message]
2020-10-02 7:14 ` bug#43709: minibuffer keybinding cheatsheet and launcher [V2 CODE SUBMISSION] Boruch Baum
2020-10-02 18:24 ` Drew Adams
2020-10-15 18:53 ` bug#43709: minibuffer keybinding cheatsheet and launcher [V3 " Boruch Baum
2020-10-15 18:57 ` Drew Adams
2021-07-21 12:14 ` bug#43709: minibuffer keybinding cheatsheet and launcher [CODE SUBMISSION] Lars Ingebrigtsen
2021-07-21 13:48 ` Boruch Baum
2021-07-22 6:49 ` Phil Sainty
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201001182504.mrdldh3rdf3uuvbm@E15-2016.optimum.net \
--to=boruch_baum@gmx.com \
--cc=43709@debbugs.gnu.org \
--cc=monnier@iro.umontreal.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).