From: storm@cua.dk (Kim F. Storm)
Cc: martin rudalics <rudalics@gmx.at>,
simon.marshall@misys.com, rms@gnu.org, emacs-devel@gnu.org
Subject: Re: jit lock sit-for provokes redisplay provokes imenu
Date: Mon, 21 Aug 2006 16:20:31 +0200 [thread overview]
Message-ID: <m33bbqqifk.fsf@kfs-l.imdomain.dk> (raw)
In-Reply-To: <87bqqfqy5e.fsf@furball.mit.edu> (Chong Yidong's message of "Sun, 20 Aug 2006 10:28:45 -0400")
Chong Yidong <cyd@stupidchicken.com> writes:
> martin rudalics <rudalics@gmx.at> writes:
>
>> Attached find ChangeLog entries and patches for this.
>
> I noticed this hasn't been committed. FWIW, I've looked through the
> changes and reread the emacs-devel thread, and the patch looks OK to
> me, with one nitpick:
>
>> + values returned by two individual calls of buffer-chars-modified-tick,
>
> This should be in quotes: `buffer-chars-modified-tick'
>
> This assumes that the patch does indeed solve the original problem
> (too many imenu updates during redisplay). I haven't been able to
> reproduce this problem, but presumably someone can.
Is this change still needed for fixing the original problem if we
completely eliminate sit-for in jit-lock?
OTOH, adding buffer-chars-modified-tick in itself (and the change to
imenu)still seems like a good thing, and I also think the change is
safe. But we need updates for lispref and NEWS.
--
Kim F. Storm <storm@cua.dk> http://www.cua.dk
next prev parent reply other threads:[~2006-08-21 14:20 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <81CCA6588E60BB42BE68BD029ED4826008838221@wimex2.wim.midas-kapiti.com>
2006-07-20 18:16 ` jit lock sit-for provokes redisplay provokes imenu Richard Stallman
2006-07-20 19:47 ` Drew Adams
2006-07-20 22:41 ` Richard Stallman
2006-07-21 9:13 ` martin rudalics
2006-07-21 19:37 ` Richard Stallman
2006-07-22 9:01 ` martin rudalics
2006-07-24 14:42 ` Richard Stallman
2006-07-26 9:32 ` martin rudalics
2006-07-21 14:53 ` martin rudalics
2006-07-22 4:38 ` Richard Stallman
2006-07-22 9:21 ` martin rudalics
2006-07-23 7:55 ` martin rudalics
2006-07-24 14:42 ` Richard Stallman
2006-07-26 11:24 ` martin rudalics
2006-07-26 15:10 ` Stefan Monnier
2006-07-26 16:16 ` martin rudalics
2006-07-27 14:35 ` Stefan Monnier
2006-07-26 22:34 ` Richard Stallman
2006-07-26 22:48 ` David Kastrup
2006-07-27 16:04 ` Richard Stallman
2006-07-27 16:27 ` David Kastrup
2006-07-27 23:45 ` Richard Stallman
2006-08-03 9:35 ` martin rudalics
2006-08-20 14:28 ` Chong Yidong
2006-08-21 14:20 ` Kim F. Storm [this message]
2006-08-21 15:20 ` Chong Yidong
2006-08-23 15:04 ` Kim F. Storm
2006-07-26 14:51 Marshall, Simon
2006-07-26 15:48 ` martin rudalics
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=m33bbqqifk.fsf@kfs-l.imdomain.dk \
--to=storm@cua.dk \
--cc=emacs-devel@gnu.org \
--cc=rms@gnu.org \
--cc=rudalics@gmx.at \
--cc=simon.marshall@misys.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.