unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on
@ 2019-09-19  0:48 'Ihor Radchenko'
  2019-09-19 14:11 ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: 'Ihor Radchenko' @ 2019-09-19  0:48 UTC (permalink / raw)
  To: 37455

Expected behaviour:

According to the manual, adding insert-in-front-hooks or
insert-behind-hooks text properties should trigger the functions listed
in the hooks when the text is inserted in front/behind the text.

Observed behaviour:

The functions never run unless I type very fast (with my own config) or
when font-lock-mode is enabled.

Steps to reproduce (the font-lock part):
1. emacs -Q
2. Insert the following elisp into scratch (the code is from https://www.reddit.com/r/emacs/comments/d5waor/how_to_use_modification_hooks_in_text_properties/):

(defun test-property ()
  (put-text-property (point) (mark) 'insert-behind-hooks
		  (list (lambda (beg end)
			     (message "before %d %d" beg end))))
  (put-text-property (point) (mark) 'insert-in-front-hooks
		  (list (lambda (beg end)
			     (message "in front %d %d" beg end)))))

1. Select the region containing all the text in the scratch buffer
2. M-: (test-property) RET
3. Try to type anything
4. No message is shown
5. Disable font-lock-mode
6. Try to type anything
7. The message is shown

Regards,
Ihor

-- 
Ihor Radchenko,
PhD,
Center for Advancing Materials Performance from the Nanoscale (CAMP-nano)
State Key Laboratory for Mechanical Behavior of Materials, Xi'an Jiaotong University, Xi'an, China
Email: yantar92@gmail.com, ihor_radchenko@alumni.sutd.edu.sg






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

* bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on
  2019-09-19  0:48 bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on 'Ihor Radchenko'
@ 2019-09-19 14:11 ` Eli Zaretskii
  2019-09-19 16:46   ` Ihor Radchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2019-09-19 14:11 UTC (permalink / raw)
  To: 'Ihor Radchenko'; +Cc: 37455

> From: 'Ihor Radchenko' <yantar92@gmail.com>
> Date: Thu, 19 Sep 2019 08:48:57 +0800
> 
> Expected behaviour:
> 
> According to the manual, adding insert-in-front-hooks or
> insert-behind-hooks text properties should trigger the functions listed
> in the hooks when the text is inserted in front/behind the text.
> 
> Observed behaviour:
> 
> The functions never run unless I type very fast (with my own config) or
> when font-lock-mode is enabled.

We have too many hooks, and they fight each other.  Recursively.

Should be fixed now on the master branch.  Thanks.





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

* bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on
  2019-09-19 14:11 ` Eli Zaretskii
@ 2019-09-19 16:46   ` Ihor Radchenko
  2019-09-19 17:35     ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Ihor Radchenko @ 2019-09-19 16:46 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 37455

Thanks!
I can confirm that the hooks work as expected in master.

Also, a word about the property names (from reddit comment by @00-11
https://www.reddit.com/r/emacs/comments/d5waor/how_to_use_modification_hooks_in_text_properties/f0q54zn/):

"Too bad they have names that end in -hooks, which is misleading. Even
if they were actually hooks (which are variables), because their
functions take arguments they would be considered "abnormal
hooks", and abnormal hooks are conventionally named with the
suffix -functions (not -hooks). Following that convention would
also be less misleading, lessening the suggestion that they're
hooks."

Would it make sense to change the text and overlay property names to
something like insert-in-front-functions, insert-behind-functions, and
modification-functions?

Regards,
Ihor
      

Eli Zaretskii <eliz@gnu.org> writes:

>> From: 'Ihor Radchenko' <yantar92@gmail.com>
>> Date: Thu, 19 Sep 2019 08:48:57 +0800
>> 
>> Expected behaviour:
>> 
>> According to the manual, adding insert-in-front-hooks or
>> insert-behind-hooks text properties should trigger the functions listed
>> in the hooks when the text is inserted in front/behind the text.
>> 
>> Observed behaviour:
>> 
>> The functions never run unless I type very fast (with my own config) or
>> when font-lock-mode is enabled.
>
> We have too many hooks, and they fight each other.  Recursively.
>
> Should be fixed now on the master branch.  Thanks.

-- 
Ihor Radchenko,
PhD,
Center for Advancing Materials Performance from the Nanoscale (CAMP-nano)
State Key Laboratory for Mechanical Behavior of Materials, Xi'an Jiaotong University, Xi'an, China
Email: yantar92@gmail.com, ihor_radchenko@alumni.sutd.edu.sg





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

* bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on
  2019-09-19 16:46   ` Ihor Radchenko
@ 2019-09-19 17:35     ` Eli Zaretskii
  0 siblings, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2019-09-19 17:35 UTC (permalink / raw)
  To: Ihor Radchenko; +Cc: 37455-done

> From: Ihor Radchenko <yantar92@gmail.com>
> Cc: 37455@debbugs.gnu.org
> Date: Fri, 20 Sep 2019 00:46:52 +0800
> 
> Thanks!
> I can confirm that the hooks work as expected in master.

Thanks, I'm therefore closing this bug report.

> "Too bad they have names that end in -hooks, which is misleading. Even
> if they were actually hooks (which are variables), because their
> functions take arguments they would be considered "abnormal
> hooks", and abnormal hooks are conventionally named with the
> suffix -functions (not -hooks). Following that convention would
> also be less misleading, lessening the suggestion that they're
> hooks."
> 
> Would it make sense to change the text and overlay property names to
> something like insert-in-front-functions, insert-behind-functions, and
> modification-functions?

I don't see a serious enough reason to change names of these
properties, which were introduced in Emacs 19(!).  The person who
wrote that raises many similar arguments, so I'm not surprised to hear
this one.

IOW, this ship has sailed decades ago, and it's too late to rename it
now.

Thanks.





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

end of thread, other threads:[~2019-09-19 17:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-19  0:48 bug#37455: 27.0.50; insert-behind-hooks and insert-in-front-hooks in text properties do nothing when font-lock-mode is on 'Ihor Radchenko'
2019-09-19 14:11 ` Eli Zaretskii
2019-09-19 16:46   ` Ihor Radchenko
2019-09-19 17:35     ` Eli Zaretskii

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