bug#37945: closed (Saving file erases minor mode font locking.)

[GNU bug Tracking System]

Your message dated Sun, 14 Jun 2020 14:20:47 +0200
with message-id <87blll2600.fsf@gnu.org>
and subject line Re: bug#37945: [Auctex] Revert font locking to defaults
has caused the debbugs.gnu.org bug report #37945,
regarding Saving file erases minor mode font locking.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
37945: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=37945
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: Saving file erases minor mode font locking. Date: Sun, 27 Oct 2019 10:45:09 -0700

When saving files, the lisp files for some style files (e.g. expl3, minted) call font-latex-update-font-lock, which has the following lines to revert the font locking to defaults (around line 1287-88 of font-latex.el):

(setq font-lock-set-defaults nil)

(font-lock-set-defaults)

This removes any font locking done by minor modes (e.g. hl-todo-mode, outshine-mode). There should be another way of applying these styles that does not override previously set font properties, or reapplies the font lock applied by minor modes.

Emacs  : GNU Emacs 26.3 (build 1, x86_64-apple-darwin18.7.0, Carbon Version 158 AppKit 1671.6)

 of 2019-10-19

Package: 12.1.2

Also covered as a bug in hl-todo-mode.

- Prashant

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#37945: [Auctex] Revert font locking to defaults
>
>
>
>
> Date: 
>
> Sun, 14 Jun 2020 14:20:47 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)
>
>
>
>
> Tassilo Horn <tsdh@gnu.org> writes:
>
> >> thanks a lot for working on this.  I tested the mentioned branch and
> >> indeed, it works flawlessly.  As far as I can see now, no other
> >> regressions were introduced,
> >
> > That's good news.  I give it some days of testing by others and then
> > I'll merge it into master.  So hopefully it'll be fixed in GNU AUCTeX
> > 12.2.4.
>
> Ok, the branch is now merged into master and we'll be doing a new ELPA
> release in some days.
>
> I'm closing this report now.  Thanks a lot for testing!
>
> Bye,
> Tassilo
>
>
> --- End Message ---