all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Joseph Garvin <joe.garvin@radix-trading.com>, 28704@debbugs.gnu.org
Subject: bug#28704: 27.0.50; Sorting packages list slow when using display-line-numbers-type visual
Date: Thu, 05 Oct 2017 10:09:25 +0200	[thread overview]
Message-ID: <59D5E8B5.5030701@gmx.at> (raw)
In-Reply-To: <s9dlgkqlwjj.fsf@radix-trading.com>

 > Then run: M-x list-packages
 >
 > Then click on one of the column headers to trigger sorting by that
 > column, e.g. Status. Emacs will freeze for a couple of minutes.
 >
 > Profiler reveals:
 >
 > - command-execute                                               25144  98%
 >   - call-interactively                                           25144  98%
 >    - funcall-interactively                                       25144  98%
 >     - tabulated-list-col-sort                                    24960  97%
 >      - tabulated-list--sort-by-column-name                       24960  97%
 >       - tabulated-list-print                                     24956  97%
 >        - apply                                                   24916  97%
 >         - tabulated-list-print-entry                             24903  97%
 >          - tabulated-list-line-number-width                      24736  96%
 >             #<compiled 0x2a38871>                                    2   0%
 >
 > So it looks like all of the time is spent inside tabulated-list-line-number-width.

‘tabulated-list-print’ should calculate ‘line-number-display-width’ once
and ‘tabulated-list-line-number-width’ should use that value.

martin






  reply	other threads:[~2017-10-05  8:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-04 16:14 bug#28704: 27.0.50; Sorting packages list slow when using display-line-numbers-type visual Joseph Garvin
2017-10-05  8:09 ` martin rudalics [this message]
2017-10-05  9:43   ` Eli Zaretskii

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=59D5E8B5.5030701@gmx.at \
    --to=rudalics@gmx.at \
    --cc=28704@debbugs.gnu.org \
    --cc=joe.garvin@radix-trading.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.