all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: "N. Jackson" <nljlistbox2@gmail.com>, 23377@debbugs.gnu.org
Subject: bug#23377: 25.0.93; Completion is extremely slow for insert-char
Date: Tue, 26 Apr 2016 08:49:39 -0700	[thread overview]
Message-ID: <54bd647f-6a22-471e-620b-6ff376e546bb@cs.ucla.edu> (raw)
In-Reply-To: <jwvtwio37wm.fsf-monnier+emacsbugs@gnu.org>

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

On 04/26/2016 04:51 AM, Stefan Monnier wrote:
> It's not adding a feature.  It's disabling a feature with an easy to use
> switch to re-enable it.
> Don't make it a defcustom, just a defvar.

OK, thanks, that's easy. Proposed patch (to emacs-25) attached.

On my platform the performance is pretty bad without this, even without 
listing all the characters. I particularly notice it when I run Emacs 
over X from my office to home, something that's normally quite 
tolerable. I had already noticed this performance problem, but hadn't 
gotten around to looking into it until N. Jackson's bug report made it 
clear I wasn't alone.

In the long run there are some things we can do to improve things (e.g., 
not compute menu items until displayed, not display those ugly boxed and 
useless hexadecimal numbers, use a better menu that lets users choose 
characters by shape rather than just by name, etc.) that could let us 
turn annotation on by default, but that all needs to wait until after 
Emacs 25 comes out.


[-- Attachment #2: 0001-insert-char-annotates-names-only-on-request.patch --]
[-- Type: application/x-patch, Size: 1900 bytes --]

  reply	other threads:[~2016-04-26 15:49 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-26  0:25 bug#23377: 25.0.93; Completion is extremely slow for insert-char N. Jackson
2016-04-26  1:04 ` Paul Eggert
2016-04-26  2:17   ` Stefan Monnier
2016-04-26  2:51     ` Drew Adams
2016-04-26  4:10       ` Paul Eggert
2016-04-26  5:43         ` Drew Adams
2016-04-26  6:21         ` Eli Zaretskii
2016-04-26  6:34         ` Eli Zaretskii
2016-04-26 13:27           ` N. Jackson
2016-04-26 20:41             ` Paul Eggert
2016-04-27  1:31               ` N. Jackson
2016-04-26 11:51         ` Stefan Monnier
2016-04-26 15:49           ` Paul Eggert [this message]
2016-04-26 16:04             ` Drew Adams
2016-04-26 16:59               ` Paul Eggert
2016-04-26 17:15                 ` Eli Zaretskii
2016-04-26 18:52                 ` N. Jackson
2016-04-26 19:10                   ` Eli Zaretskii
2016-04-27  2:09                     ` N. Jackson
     [not found]                   ` <<83r3dsyynq.fsf@gnu.org>
2016-04-26 20:18                     ` Drew Adams
2016-04-26 16:27             ` Eli Zaretskii
2016-04-26 23:58               ` John Wiegley
2016-04-27  0:26                 ` Paul Eggert
2016-04-27  0:53                   ` Lars Magne Ingebrigtsen
2016-04-27  1:19                     ` John Wiegley
2016-04-27  7:24                       ` Eli Zaretskii
2016-04-27 12:27                         ` Stefan Monnier
2016-04-27 19:04                           ` John Wiegley
2016-04-27  0:10             ` Stefan Monnier
2016-04-26  6:26 ` Eli Zaretskii
2016-04-27  1:54   ` N. Jackson
2016-04-27  7:21     ` Eli Zaretskii
2016-04-27 15:16       ` N. Jackson
2017-09-30 16:40         ` Noam Postavsky
2017-09-30 20:27           ` N. Jackson
2019-06-25 13:53             ` Lars Ingebrigtsen

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54bd647f-6a22-471e-620b-6ff376e546bb@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=23377@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=nljlistbox2@gmail.com \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.