unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Alexander Pohoyda <alexander.pohoyda@gmx.net>
Cc: emacs-devel@gnu.org
Subject: Re: (font-lock-mode 1) does not always force font-lock
Date: 26 Oct 2003 12:32:51 +0100	[thread overview]
Message-ID: <87ismcb5ek.fsf@oak.pohoyda.family> (raw)
In-Reply-To: <m3oew5d4q4.fsf@hniksic.iskon.hr>

Hrvoje Niksic <hniksic@xemacs.org> writes:

> > Hrvoje Niksic <hniksic@xemacs.org> writes:
> >
> >> > Yes, both cases work. If I comment out (font-lock-mode 1), things
> >> > still work.
> >> 
> >> Please repeat the tests with `emacs -q' then.
> >
> > OK. There is no difference whether the buffer is named "*foo*" or just
> > "foo". Both cases work.
> 
> I'm afraid you tested the wrong thing.  You need to try "*foo*" and
> " *foo*", i.e. with and without the leading space in the buffer name.

Yes, indeed, I misread your original posting. Sorry for that.
Now I was able to reproduce the problem you described.

Please open the lisp/font-core.el file, function font-lock-mode, 
right at the beginning of the function, after very long comment, code
goes like this:

  ;; Don't turn on Font Lock mode if we don't have a display (we're running a
  ;; batch job) or if the buffer is invisible (the name starts with a space).
  (when (or noninteractive (eq (aref (buffer-name) 0) ?\ ))
    (setq font-lock-mode nil))
  (funcall font-lock-function font-lock-mode)

That's what happens: font-lock-default-function is called with nil
argument. So, the buffer is considered to be invisible.

I don't know whether this is a bug or not, but I'm wondering why
there's no such function:

(defun buffer-invisible-p (&optional buffer)
  "Return t if BUFFER is invisible.
No argument or nil as argument means use current buffer as BUFFER."
  (eq (aref (buffer-name buffer) 0) ?\ ))

do to the job.

-- 
Alexander Pohoyda <alexander.pohoyda@gmx.net>
PGP Key fingerprint: 7F C9 CC 5A 75 CD 89 72  15 54 5F 62 20 23 C6 44

  reply	other threads:[~2003-10-26 11:32 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <m3smlkygno.fsf@hniksic.iskon.hr>
2003-10-23 18:38 ` (font-lock-mode 1) does not always force font-lock Richard Stallman
2003-10-23 22:30   ` Hrvoje Niksic
2003-10-24  5:22     ` Alexander Pohoyda
2003-10-24  9:49       ` Hrvoje Niksic
2003-10-24 19:12         ` Alexander Pohoyda
2003-10-24 20:15           ` Hrvoje Niksic
2003-10-25  7:35             ` Alexander Pohoyda
2003-10-25  9:52               ` Hrvoje Niksic
2003-10-26 11:32                 ` Alexander Pohoyda [this message]
2003-10-26 16:56                   ` Hrvoje Niksic
2003-10-26 19:07                     ` Stefan Monnier
2003-10-27 23:44                       ` Richard Stallman
2003-10-28 22:09                         ` Alexander Pohoyda
2003-11-18 21:13                         ` Alexander Pohoyda

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=87ismcb5ek.fsf@oak.pohoyda.family \
    --to=alexander.pohoyda@gmx.net \
    --cc=emacs-devel@gnu.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).