unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#74156: 29.4; Incorrect face with outline-minor-mode-highlight in lisp-mode
@ 2024-11-01 13:11 Andreas Matthias
  0 siblings, 0 replies; only message in thread
From: Andreas Matthias @ 2024-11-01 13:11 UTC (permalink / raw)
  To: 74156

[-- Attachment #1: Type: text/plain, Size: 955 bytes --]

Take this example Elisp file:


;;; xxx -- OK: face is outline-1
;;;; xxx -- OK: face is outline-2
(defun test ()) ; WRONG: face is outline-8

;; Local Variables:
;; outline-minor-mode-highlight: override
;; eval: (outline-minor-mode 1)
;; End:


After loading this file into Emacs you will see that the function definition
is displayed incorrectly in face outline-8.

I guess this is due to the definition of outline-regexp in lisp-mode.el:

";;;;* [^
\t\n]\\|(\\|\\(^;;;###\\(\\([-[:alnum:]]+?\\)-\\)?\\(autoload\\)\\)"

Notice the opening parenthesis in the regular expression that causes
the function definition to be recognized as an outline heading.

I removed the opening parenthesis and now the face of the function
definition
is correct. This is fine for me. But note that you loose the ability to move
to top-level expressions, like this function definition, with
(outline-next-visible-heading) and (outline-previous-visible-heading).


Andreas

[-- Attachment #2: Type: text/html, Size: 1164 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-11-01 13:11 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-01 13:11 bug#74156: 29.4; Incorrect face with outline-minor-mode-highlight in lisp-mode Andreas Matthias

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).