[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and,
From: |
João Távora |
Subject: |
Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations |
Date: |
Wed, 2 Jun 2021 00:29:40 +0100 |
On Wed, Jun 2, 2021 at 12:22 AM João Távora <joaotavora@gmail.com> wrote:
>
> On Wed, Jun 2, 2021 at 12:04 AM Dmitry Gutov <dgutov@yandex.ru> wrote:
> >
> > On 01.06.2021 21:47, João Távora wrote:
> > > In what way? Really, all other things being equal, in what way is
> > > threading arguments through lots of functions, bloating the arglist,
> > > implementations and the docstrings of said functions, calling group-fun
> > > multiple times and in multiple places in minibuffer.el better than_not_
> > > doing those things?
> >
> > Why do you think it forces you to "thread" them?
>
> The current implementation in minibuffer.el threads the function down
> to the point where it is called twice again (once with the same
> second argument -- a possible inefficiency -- and then with a non-nil
> second argument). If the results of a single call were stored in
> the candidate string, this wouldn't be necessary.
By the way, another classic way to avoid threading in Lisp
is binding a dynamic variable. We do that loads (but I
still thing storing a property is cleaner).
João
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, (continued)
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Stefan Monnier, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Stefan Monnier, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/02
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Stefan Monnier, 2021/06/02
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Dmitry Gutov, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations,
João Távora <=
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Dmitry Gutov, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/01
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Dmitry Gutov, 2021/06/02
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/02
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, Dmitry Gutov, 2021/06/02
- Re: [PATCH] (icomplete-vertical-mode): Add support for affixations and, annotations, João Távora, 2021/06/02
- complexity in minibuffer (was: (icomplete-vertical-mode): Add support for affixations and, annotations), Stefan Monnier, 2021/06/02
- Re: complexity in minibuffer (was: (icomplete-vertical-mode): Add support for affixations and, annotations), João Távora, 2021/06/02
- Re: complexity in minibuffer, Stefan Monnier, 2021/06/02
- Re: complexity in minibuffer, João Távora, 2021/06/02