GNU bug report logs - #48545
28.0.50; `icomplete-vertical-mode` does not support the `group-function`

Previous Next

Package: emacs;

Reported by: Daniel Mendler <mail <at> daniel-mendler.de>

Date: Thu, 20 May 2021 18:57:02 UTC

Severity: normal

Found in version 28.0.50

Done: João Távora <joaotavora <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: João Távora <joaotavora <at> gmail.com>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: Daniel Mendler <mail <at> daniel-mendler.de>, 48545 <at> debbugs.gnu.org
Subject: bug#48545: 28.0.50; `icomplete-vertical-mode` does not support the `group-function`
Date: Sun, 22 Aug 2021 16:44:09 +0100
Dmitry Gutov <dgutov <at> yandex.ru> writes:

> I don't think I'm obligated to support every trivial suggestion with a
> patch and a benchmark.

Of course you're not obliged to, but the fact that you won't code up
your own "trivial suggestion" is eloquent.

> Or explain, from various POVs, why removing the sorting step "because
> it's expensive" is faulty reasoning.

Could be, if it _were_ the reasoning.  But it's not, as is quite easy to
read here.

> Or why the grouping approach in icomplete mode should match what the
> default UI does and what other completion UIs (from which the grouping
> feature was extracted) do as well.

The sorting in the default completion UI is already discrepant to
completions-all-sorted-completions, regardless of grouping.  It's also
quite slow, as reported.  I doubt we want Icomplete to "match" that (not
to mention the fact that Icomplete is a different type of UI).

> Whatever, do what you like. I'm out of this thread.

I've pushed ba852512f23fdab674086e35d4207e3970dd0912 to fix the
described bugs and wrap this up.  Left a TODO in minibuffer.el for
anyone to experiment with "trivial suggestions" (or complex ones at
that).

João




This bug report was last modified 3 years and 326 days ago.

Previous Next


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