all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Alan Mackenzie <acm@muc.de>
Cc: emacs-devel@gnu.org
Subject: Re: M-w, then C-y.  C-y inserts text properties that aren't on original.
Date: Sat, 20 Feb 2010 11:02:39 +0200	[thread overview]
Message-ID: <83r5ogwa8g.fsf@gnu.org> (raw)
In-Reply-To: <20100219143513.GC918@muc.de>

> Date: Fri, 19 Feb 2010 14:35:13 +0000
> Cc: emacs-devel@gnu.org
> From: Alan Mackenzie <acm@muc.de>
> 
> `insert-for-yank' carries on, and calls `remove-yank-excluded-properties'
> which deliberately changes 'category properties into other properties.
> 
> Presumably there was some special case in the distant past which required
> this transformation.  It is surely wrong in the general case - yanking
> into a buffer shouldn't add text properties which weren't in the
> original - surely?
> 
> I'm going to try and track down that special case.  It looks like I'll
> have to fix the inserted string in CC Mode by hand.

The reason for this behavior is this change:

    2002-04-27  Richard M. Stallman  <rms@gnu.org>

	    * subr.el (insert-for-yank): Replace `category' property
	    with whatever properties it stands for.

The discussion that led to this change starts here:

  http://lists.gnu.org/archive/html/emacs-devel/2002-04/msg00648.html

You will find there the reason for the change (buttons in *Help*
buffers, and the way we put the mouse-face property on them).

I wonder if we could make the code which replaces `category' with what
it stands for, with a slightly less general code that replaces only
those values of `category' that are used for buttons.  Or some similar
change which will make that feature more focused on the problem it
needs to solve.

I added a comment in `remove-yank-excluded-properties' explaining what
do we do that for and pointing to the above URL for details.




  reply	other threads:[~2010-02-20  9:02 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-18 20:29 M-w, then C-y. C-y inserts text properties that aren't on original Alan Mackenzie
2010-02-19  4:47 ` Stefan Monnier
2010-02-19  9:22 ` Eli Zaretskii
2010-02-19 12:24   ` Alan Mackenzie
2010-02-19 14:35     ` Alan Mackenzie
2010-02-20  9:02       ` Eli Zaretskii [this message]
2010-02-20  9:25         ` Eli Zaretskii
     [not found]           ` <20100221215951.GD4407@muc.de>
2010-02-22  4:12             ` Eli Zaretskii
2010-02-22 12:56               ` Alan Mackenzie
2010-02-22 22:11                 ` Daniel Colascione
2010-02-23  4:43           ` Stefan Monnier
  -- strict thread matches above, loose matches on Subject: below --
2010-02-22 20:30 MON KEY

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83r5ogwa8g.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=acm@muc.de \
    --cc=emacs-devel@gnu.org \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.