unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@IRO.UMontreal.CA>
To: Michael Mauger <mmaug@yahoo.com>
Cc: Vitalie Spinu <spinuvit@gmail.com>, Emacs Devel <emacs-devel@gnu.org>
Subject: Re: Deffering redisplay in COMINT
Date: Wed, 16 Jan 2013 09:39:33 -0500	[thread overview]
Message-ID: <jwvfw21mctw.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <1358064889.40573.YahooMailNeo@web160906.mail.bf1.yahoo.com> (Michael Mauger's message of "Sun, 13 Jan 2013 00:14:49 -0800 (PST)")

> I ran several different configurations trying to identify important
> criteria related to the display of this large block of text.

>        Display |               GUI/X |              Text/NW|
> Truncate-lines |      nil |        t |      nil |         t|
>   ---Filter--- |    ----- |    ----- |    ----- |     -----|
>           BASE |   3:43   |   5:51   |   0:16   |    4:20  |
>             SS |   3:42   |   3:06   |   0:12   |    2:04  |
>            HSC |    N/A   |   0:07   |    N/A   |    0:04  |

From your tests, it seems clear that the problem is always redisplay
time, which is why the extra work of HSC pays off.

I think I can understand the difference between truncate and
non-truncate on a tty (the amount of buffer text redisplay has to
consider (i.e. (- (window-end) (window-start))) when refreshing the
display is larger (proportional to the line lengths) than with
wrap-around).  But I don't understand why we don't see the same impact
in the GUI case.  IOW I really don't understand the "3:42" of SS+GUI
with truncate-lines==nil.  I'd expect it to be *much* lower (like 0:20
or so).

Maybe we could speed it up the 2:04 case with a redisplay optimization that
skips more quickly over the truncated part of the lines (special casing
the "easy common case where there's no before/after-string, no `display'
property, ...").

You could try and tweak the SS code further to reduce the number of
redisplays.  E.g. not only you "wait for a complete line" but you also
"wait for the next second" so the buffer is only redisplayed once
a second.

Or you could try to do something halfway between SS and HSC: instead of
keeping the non-displayed part of lines in a separate buffer (as in
HSC), you do insert them (as in SS) but with an `invisible'
text-property set to t so that the redisplay can (hopefully) skip over
it very quickly.


        Stefan



      parent reply	other threads:[~2013-01-16 14:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-07  1:08 Deffering redisplay in COMINT Michael Mauger
2012-12-25 11:09 ` Vitalie Spinu
2013-01-06  8:09   ` Michael Mauger
2013-01-08 18:19     ` Stefan Monnier
2013-01-11 14:40       ` Michael Mauger
2013-01-11 15:55         ` Stefan Monnier
2013-01-13  8:14           ` Michael Mauger
2013-01-13 16:46             ` Eli Zaretskii
2013-01-16 14:39             ` Stefan Monnier [this message]

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=jwvfw21mctw.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=mmaug@yahoo.com \
    --cc=spinuvit@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 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).