From: Stefan Monnier <monnier@IRO.UMontreal.CA>
To: Alan Mackenzie <acm@muc.de>
Cc: bug-cc-mode@gnu.org, Lars Ingebrigtsen <larsi@gnus.org>,
emacs-devel@gnu.org
Subject: Re: Emacs revision #107149
Date: Sun, 12 Feb 2012 22:15:18 -0500 [thread overview]
Message-ID: <jwv1upz1li7.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <20120212222238.GA2810@acm.acm> (Alan Mackenzie's message of "Sun, 12 Feb 2012 22:22:38 +0000")
> Try this instead: It solves the actual problem stated in the comment,
> rather than papering over it:
> *** mm-view.el~ 2012-02-08 14:25:18.000000000 +0000
> --- mm-view.el 2012-02-12 21:50:34.000000000 +0000
> ***************
> *** 601,610 ****
> (require 'font-lock)
> ;; I find font-lock a bit too verbose.
> (let ((font-lock-verbose nil)
> ! (font-lock-support-mode nil))
> ;; Disable support modes, e.g., jit-lock, lazy-lock, etc.
> ;; Note: XEmacs people use `font-lock-mode-hook' to run those modes.
> ! (set (make-local-variable 'font-lock-mode-hook) nil)
> (setq buffer-file-name (mm-handle-filename handle))
> (set (make-local-variable 'enable-local-variables) nil)
> (with-demoted-errors
> --- 601,612 ----
> (require 'font-lock)
> ;; I find font-lock a bit too verbose.
> (let ((font-lock-verbose nil)
> ! (font-lock-support-mode nil)
> ! (font-lock-mode-hook font-lock-mode-hook))
> ;; Disable support modes, e.g., jit-lock, lazy-lock, etc.
> ;; Note: XEmacs people use `font-lock-mode-hook' to run those modes.
> ! (remove-hook 'font-lock-mode-hook 'turn-on-fast-lock)
> ! (remove-hook 'font-lock-mode-hook 'turn-on-lazy-lock)
> (setq buffer-file-name (mm-handle-filename handle))
> (set (make-local-variable 'enable-local-variables) nil)
> (with-demoted-errors
Actually, a real fix to this problem involves adding new a font-lock
function, which I like to call font-lock-ensure-fontification, which
mm-view would then call to force immediate fontification of the
specified region. This function would know how to handle jit-lock,
lazy-lock, and whichever other accelerator we come up with.
Such a functionality is also needed in other packages, such as
in htmlfontify. We have a bug report for this, but I can't find it
any more.
Stefan
next prev parent reply other threads:[~2012-02-13 3:15 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-10 23:20 Emacs revision #107149 Alan Mackenzie
2012-02-11 15:56 ` Lars Ingebrigtsen
2012-02-11 18:07 ` Wolfgang Jenkner
2012-02-11 19:59 ` Alan Mackenzie
2012-02-12 20:42 ` Lars Ingebrigtsen
2012-02-12 22:22 ` Alan Mackenzie
2012-02-13 1:00 ` Wolfgang Jenkner
2012-02-13 21:35 ` Alan Mackenzie
2012-02-13 3:15 ` Stefan Monnier [this message]
2012-02-13 21:42 ` Alan Mackenzie
2012-02-13 22:18 ` Stefan Monnier
2012-02-13 21:03 ` Lars Ingebrigtsen
2012-02-13 22:07 ` Alan Mackenzie
2012-02-20 9:51 ` Lars Ingebrigtsen
2012-02-21 15:16 ` Alan Mackenzie
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=jwv1upz1li7.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=acm@muc.de \
--cc=bug-cc-mode@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=larsi@gnus.org \
/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).