unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Gregory Heytings <gregory@heytings.org>
To: "Paul W. Rankin" <pwr@bydasein.com>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: 'M-o' ('facemap-keymap') has now been removed until March 10th 2021
Date: Wed, 24 Mar 2021 12:12:03 +0000	[thread overview]
Message-ID: <8786a8e8fa911a5c9ef3@heytings.org> (raw)
In-Reply-To: <286451D4-3958-4EA4-B87A-A78DCB684E5C@bydasein.com>


>> Does it?  AFAICS, font-lock-unfontify-region does not "remove _all_ 
>> text properties", it uses remove-list-of-text-properties, which 
>> "removes _some_ properties from text", namely those of font-lock-mode.
>
> Sure, you're right that it does not remove *all* text properties per se. 
> It removes, let's say, "all that it can" which is more than "all that is 
> intended".
>

Please provide a recipe to demonstrate when this actually matters.

>> It allows you to toggle font-lock-mode correctly, which M-x 
>> font-lock-mode can't do.  Try the following: emacs -Q, unless, M-., 
>> M-h, M-w, C-x b RET, M-x text-mode, C-y, M-x font-lock-mode.
>
> This is again correct behaviour.  When you yank the defun from an 
> emacs-lisp-mode buffer into a text-mode buffer, you're inserting text 
> with (most of) its text properties, except for the property `(fontified 
> . t)' because within the context of *this* buffer, the text properties 
> have not been applied with font-lock-mode and so this would be false and 
> prevent font-lock from doing its work.
>

This is not the problem here.  The problem is that M-x font-lock-mode 
(last part of the recipe) is supposed to turn font-lock-mode off, yet the 
font-lock-mode fontification remains.

>
> I hope this clears things up for you. Unless there's any outside 
> objection I'm going to fix up the command in master to reflect as pasted 
> above.
>

I strongly object, there's no need to "fix" a working command to make it 
worse.



  reply	other threads:[~2021-03-24 12:12 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-23 21:38 'M-o' ('facemap-keymap') has now been removed until March 10th 2021 Paul W. Rankin via Emacs development discussions.
2021-03-23 22:07 ` Gregory Heytings
2021-03-23 23:08   ` Paul W. Rankin via Emacs development discussions.
2021-03-23 23:13     ` Gregory Heytings
2021-03-24  5:40       ` Paul W. Rankin via Emacs development discussions.
2021-03-24  8:01         ` Gregory Heytings
2021-03-24  9:09           ` Paul W. Rankin via Emacs development discussions.
2021-03-24  9:20             ` Gregory Heytings
2021-03-24  9:33               ` Paul W. Rankin via Emacs development discussions.
2021-03-24  9:44                 ` Gregory Heytings
2021-03-24 12:00                   ` Paul W. Rankin via Emacs development discussions.
2021-03-24 12:12                     ` Gregory Heytings [this message]
2021-03-24 12:35                       ` Paul W. Rankin via Emacs development discussions.
2021-03-24 13:01                         ` Gregory Heytings
2021-03-24 13:09                 ` Stefan Monnier
2021-03-24 13:30                   ` Paul W. Rankin via Emacs development discussions.
2021-03-24 15:14                     ` Stefan Monnier
2021-03-24 15:38                       ` Paul W. Rankin via Emacs development discussions.
2021-03-24 15:40                         ` Stefan Monnier
2021-03-24 15:53                           ` Paul W. Rankin via Emacs development discussions.
2021-03-24 21:47                             ` Stefan Monnier
2021-03-24 17:09             ` Eli Zaretskii
2021-03-24 21:58               ` Stefan Monnier
2021-03-25  6:15                 ` Eli Zaretskii
2021-03-25 14:03                   ` Stefan Monnier
2021-03-24 13:07   ` Stefan Monnier
2021-03-24 13:41     ` Paul W. Rankin via Emacs development discussions.
2021-03-24 13:53       ` Gregory Heytings
  -- strict thread matches above, loose matches on Subject: below --
2021-02-10 18:40 Lars Ingebrigtsen
2021-02-10 19:14 ` Alan Mackenzie
2021-02-10 19:19   ` Lars Ingebrigtsen
2021-02-10 19:38     ` Lars Ingebrigtsen
2021-02-10 19:47       ` Alan Mackenzie
2021-02-11 13:34 ` Richard Stallman
2021-03-11 16:27 ` Lars Ingebrigtsen
2021-03-11 16:53   ` Eli Zaretskii
2021-03-11 17:02     ` Gregory Heytings
2021-03-11 17:29       ` Eli Zaretskii
2021-03-12 12:09       ` Filipp Gunbin
2021-03-12 12:46         ` Gregory Heytings
2021-03-11 17:37   ` Stefan Kangas
2021-03-11 18:25   ` Alfred M. Szmidt
2021-03-17 16:32   ` Sean Whitton
2021-03-18  3:43     ` Lars Ingebrigtsen
2021-03-18  4:35       ` Sean Whitton
2021-03-18  4:40         ` Lars Ingebrigtsen
2021-03-18  4:16   ` Lars Ingebrigtsen
2021-03-18  9:00     ` Eli Zaretskii
2021-03-18 10:27       ` Eli Zaretskii
2021-03-19  7:46         ` Lars Ingebrigtsen
2021-03-19  8:06           ` Eli Zaretskii
2021-03-19  9:35             ` Gregory Heytings
2021-03-19 12:01               ` Eli Zaretskii
2021-03-20  7:58             ` Lars Ingebrigtsen
2021-03-20  8:39               ` Andreas Schwab
2021-03-20  8:45                 ` Lars Ingebrigtsen
2021-03-20  8:58                   ` Gregory Heytings
2021-03-20  9:10               ` Eli Zaretskii
2021-03-18 13:28       ` Jean Louis
2021-03-18 14:45         ` Eli Zaretskii
2021-03-18  9:45     ` Alfred M. Szmidt
2021-03-18 13:25     ` Jean Louis
2021-03-18 23:03     ` Sean Whitton
2021-03-19 13:14     ` Gregory Heytings
2021-03-20  7:54       ` Lars Ingebrigtsen

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=8786a8e8fa911a5c9ef3@heytings.org \
    --to=gregory@heytings.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=pwr@bydasein.com \
    /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).