unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#31915: 26.0.50; highlight-regexp does not highlight regexp
@ 2018-06-20 16:26 Markus Triska
  2018-06-20 17:07 ` Eli Zaretskii
  2018-06-20 17:32 ` Michael Heerdegen
  0 siblings, 2 replies; 5+ messages in thread
From: Markus Triska @ 2018-06-20 16:26 UTC (permalink / raw)
  To: 31915


Please invoke Emacs via:

    $ emacs -Q --eval "(let ((x \"x\"))                 \
                         (find-file x)                  \
                         (insert x)                     \
                         (highlight-regexp x 'hi-green) \
                         (when font-lock-mode (insert x)))"

The docstring of highlight-regexp states:

    Use Font lock mode, if enabled, to highlight REGEXP.  Otherwise,
    use overlays for highlighting.  If overlays are used, the
    highlighting will not update as you type.

Since font-lock-mode is clearly enabled when the second "x" is inserted,
I expect it to be highlighted. However, only the first "x" is highlighted.

Could you please adapt the docstring or, alternatively, highlight-regexp
to match what is documented with what is observed? Personally, I would
prefer that also the second "x" be highlighted in this case. Thank you!


In GNU Emacs 26.0.50.1 (x86_64-unknown-linux-gnu, X toolkit, Xaw scroll bars)
 of 2016-10-21 built on my computer
Repository revision: 0929e061fb544c0f4c10ff0412a41cf7e8148270
System Description:	Debian GNU/Linux 9.3 (stretch)


Configured using:
 'configure --with-gif=no --with-tiff=no --with-gnutls=no'

Configured features:
XPM JPEG PNG SOUND NOTIFY FREETYPE XFT ZLIB TOOLKIT_SCROLL_BARS LUCID
X11






^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#31915: 26.0.50; highlight-regexp does not highlight regexp
  2018-06-20 16:26 bug#31915: 26.0.50; highlight-regexp does not highlight regexp Markus Triska
@ 2018-06-20 17:07 ` Eli Zaretskii
  2018-06-20 21:22   ` Markus Triska
  2018-06-20 17:32 ` Michael Heerdegen
  1 sibling, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2018-06-20 17:07 UTC (permalink / raw)
  To: Markus Triska; +Cc: 31915

> From: Markus Triska <triska@metalevel.at>
> Date: Wed, 20 Jun 2018 18:26:13 +0200
> 
> Please invoke Emacs via:
> 
>     $ emacs -Q --eval "(let ((x \"x\"))                 \
>                          (find-file x)                  \
>                          (insert x)                     \
>                          (highlight-regexp x 'hi-green) \
>                          (when font-lock-mode (insert x)))"
> 
> The docstring of highlight-regexp states:
> 
>     Use Font lock mode, if enabled, to highlight REGEXP.  Otherwise,
>     use overlays for highlighting.  If overlays are used, the
>     highlighting will not update as you type.
> 
> Since font-lock-mode is clearly enabled when the second "x" is inserted,
> I expect it to be highlighted.

Actually, that buffer is in Fundamental mode, where there's no
font-lock.  Somehow, I suspect you already know that.

> Could you please adapt the docstring or, alternatively, highlight-regexp
> to match what is documented with what is observed?

Would it suffice to say "Note that in Fundamental mode, there's no
font lock, so overlays are always used"?  Or did you mean something
else?

Thanks.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#31915: 26.0.50; highlight-regexp does not highlight regexp
  2018-06-20 16:26 bug#31915: 26.0.50; highlight-regexp does not highlight regexp Markus Triska
  2018-06-20 17:07 ` Eli Zaretskii
@ 2018-06-20 17:32 ` Michael Heerdegen
  2018-06-20 21:26   ` Markus Triska
  1 sibling, 1 reply; 5+ messages in thread
From: Michael Heerdegen @ 2018-06-20 17:32 UTC (permalink / raw)
  To: Markus Triska; +Cc: 31915

Markus Triska <triska@metalevel.at> writes:

> Since font-lock-mode is clearly enabled when the second "x" is inserted,
> I expect it to be highlighted. However, only the first "x" is highlighted.

"If overlays are used, the highlighting will not update as you type."
Do you want to clarify that this also applies to text that is inserted
without typing it?


Michael.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#31915: 26.0.50; highlight-regexp does not highlight regexp
  2018-06-20 17:07 ` Eli Zaretskii
@ 2018-06-20 21:22   ` Markus Triska
  0 siblings, 0 replies; 5+ messages in thread
From: Markus Triska @ 2018-06-20 21:22 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 31915

Eli Zaretskii <eliz@gnu.org> writes:

> Actually, that buffer is in Fundamental mode, where there's no
> font-lock.  Somehow, I suspect you already know that.

In the snippet I posted, I used the variable font-lock-mode to test
whether font-lock is enabled. Its documentation states:

    Non-nil if Font-Lock mode is enabled.

Is this documentation valid? If not, could you please adapt it, or -
alternatively - make the implementation match the description?

> Would it suffice to say "Note that in Fundamental mode, there's no
> font lock, so overlays are always used"?  Or did you mean something
> else?

Personally, I suspect that adding this would not cover all cases. I get
this impression from looking at hi-lock-set-pattern:

      (if (and font-lock-mode (font-lock-specified-p major-mode))

This condition seems to be different from what is documented.

Thank you and all the best,
Markus





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#31915: 26.0.50; highlight-regexp does not highlight regexp
  2018-06-20 17:32 ` Michael Heerdegen
@ 2018-06-20 21:26   ` Markus Triska
  0 siblings, 0 replies; 5+ messages in thread
From: Markus Triska @ 2018-06-20 21:26 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: 31915

Michael Heerdegen <michael_heerdegen@web.de> writes:

> "If overlays are used, the highlighting will not update as you type."
> Do you want to clarify that this also applies to text that is inserted
> without typing it?

This would be a welcome clarification, since it seems to be the case!

All the best,
Markus





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-06-20 21:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-20 16:26 bug#31915: 26.0.50; highlight-regexp does not highlight regexp Markus Triska
2018-06-20 17:07 ` Eli Zaretskii
2018-06-20 21:22   ` Markus Triska
2018-06-20 17:32 ` Michael Heerdegen
2018-06-20 21:26   ` Markus Triska

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