unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: E Sabof <esabof@gmail.com>
Cc: 14170@debbugs.gnu.org
Subject: bug#14170: 24.3; linum won't create all overlays after a folding
Date: Sat, 13 Apr 2013 23:16:11 -0400	[thread overview]
Message-ID: <jwvobdhst6t.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <CAEp6DyZ0qG3ie2P+rxRh9hWABbXcPG=DMJfVDnkHeY8uXJ526A@mail.gmail.com> (E. Sabof's message of "Sun, 14 Apr 2013 02:40:06 +0100")

>> > It crossed my mind, but I didn't like the idea, as it also removes
>> > "good" fortification.  I suppose this could be of little consequence.
>> I don't know what you mean by "removes good fontification".
> Actually you've said it in a comment:
> (remove-overlays start end 'nlinum t)
> ;; Warn jit-lock that this part of the buffer is not done any
> ;; more.  This has the downside that font-lock will be re-applied
> ;; as well.  But jit-lock doesn't know how to (and doesn't want
> ;; to) keep track of the status of its various
> ;; clients independently.
> (put-text-property start end 'fontified nil)

Oh, that's what you meant by "remove".  Right, it flushes fontification
which did not need to be flushed, but that can be recomputed.
Of course, the overall impact on performance could be a concern,
although it's unclear how important that would be.  Based on the
experience that having many overlays isn't nearly as bad as expected,
the nlinum--ol-count limit could be pushed much higher than 100, thus
reducing the potential performance impact of "removing good
fontification" correspondingly.


        Stefan





  reply	other threads:[~2013-04-14  3:16 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-09 21:33 bug#14170: 24.3; linum won't create all overlays after a folding E Sabof
2013-04-10  0:38 ` Stefan Monnier
2013-04-10  1:02   ` E Sabof
2013-04-10  1:51     ` Glenn Morris
2013-04-12 12:52       ` Eli Zaretskii
2013-04-14  1:21         ` Stefan Monnier
2013-04-14  6:15           ` Eli Zaretskii
2013-04-15 23:00             ` Stefan Monnier
2013-04-16  6:15               ` Eli Zaretskii
2013-04-11  1:37     ` E Sabof
2013-04-11  1:44       ` E Sabof
2013-04-11  2:50       ` Eli Zaretskii
2013-04-11  3:23         ` E Sabof
2013-04-11 16:14           ` Eli Zaretskii
2013-04-11 13:58       ` Stefan Monnier
     [not found]         ` <CAEp6DyZQx81yUkxKfqk2O8VJz68xYVs0+_Cii50MG4Wxc6q=rA@mail.gmail.com>
     [not found]           ` <jwvmwt1ucww.fsf-monnier+emacs@gnu.org>
2013-04-14  1:40             ` E Sabof
2013-04-14  3:16               ` Stefan Monnier [this message]
2016-06-09  3:03 ` Noam Postavsky

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=jwvobdhst6t.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=14170@debbugs.gnu.org \
    --cc=esabof@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).