From: Drew Adams <drew.adams@oracle.com>
To: Eli Zaretskii <eliz@gnu.org>, Drew Adams <drew.adams@oracle.com>
Cc: 22296@debbugs.gnu.org
Subject: bug#22296: 25.1.50; Document changes to `delete-selection-mode'
Date: Sun, 3 Jan 2016 08:24:35 -0800 (PST) [thread overview]
Message-ID: <940d59ef-c3d6-42df-ba0c-602cc154d431@default> (raw)
In-Reply-To: <<83mvsmadhv.fsf@gnu.org>>
> I looked at delsel.el, and the only user-visible change in behavior
> since v24.3 is the one that was already announced in NEWS.24:
>
> ** Delete Selection mode can now be used without Transient Mark mode.
Somehow I didn't notice that mention.
> If you know about other changes (that are not bugfixes), please name
> them.
Frankly, I do not understand all of the changes I see in the code
since 24.3. There are a boatload of them. `diff' the Emacs 25
version against the 24.3 version.
1. It "can now be used without transient mark mode" is misleading.
You could always turn off t-m-mode (after turning d-s-mode on)
and still use d-s-mode. (But you might have been surprised to see
stuff disappear because you didn't notice what was in the region.)
The changes for this seem to be: (1) d-s-mode does not automatically
turn on t-m-mode now, and (2) d-s-mode does NOT have any effect unless
you turn on t-m-mode temporarily. IOW, d-s-mode works now only with
temporary t-m-mode; it no longer works with a non-active region.
(And yes, this is a good change.)
2. d-s-mode now supports rectangular regions (?). Not clear
to me what this is about, from a quick glance at the code.
Maybe you will characterize this as a bug fix, but the behavior
seems to be changed, in any case.
3. Command `delete-selection-repeat-replace-region' was added.
It seems to be bound, so far, only in CUA mode (to `M-v').
Not real clear to me what this is about/for.
4. Apparently you can no longer specify `kill' as the d-s behavior:
(put 'some-command 'delete-selection 'kill). Well, it's still
supported for now, but `kill' has been removed from the file-header
Commentary, and the code comments say that it is deprecated.
Why is this? Where is this deprecation documented? Why
shouldn't a user or a library still be able to override a
`delete-selection' setting for a given command, to specify
it as `kill'? Not clear to me what this is about.
5. There is an additional, undocumented (why?) possible value
for `delete-selection', which you can apparently put on a
command's symbol: `delete-selection-uses-region-p'. Please
consider documenting this (e.g., not just mentioning it in NEWS).
6. The default delete-selection behavior for multiple commands
has apparently changed. For example, `open-line' now has the
default behavior and not the `kill' behavior. Likewise, unloading
(`delsel-unload-function') deals with a different command list.
Those are a start - things I see that I wonder about. You may
others, or you may understand these things better or differently.
next parent reply other threads:[~2016-01-03 16:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <<660d1a46-fd5a-460f-8f26-50ba9a52cc60@default>
[not found] ` <<83mvsmadhv.fsf@gnu.org>
2016-01-03 16:24 ` Drew Adams [this message]
2016-01-03 17:02 ` bug#22296: 25.1.50; Document changes to `delete-selection-mode' Eli Zaretskii
[not found] <<<<<<660d1a46-fd5a-460f-8f26-50ba9a52cc60@default>
[not found] ` <<<<<<83mvsmadhv.fsf@gnu.org>
[not found] ` <<<<<940d59ef-c3d6-42df-ba0c-602cc154d431@default>
[not found] ` <<<<<834meua953.fsf@gnu.org>
[not found] ` <<<<68435980-44a2-4d33-8f64-8fbbdedb4a4e@default>
[not found] ` <<<<83vb7a8pxi.fsf@gnu.org>
[not found] ` <<<8b4cae3f-6f4c-4784-85fc-36a659fe15fd@default>
[not found] ` <<<83r3hy8owz.fsf@gnu.org>
[not found] ` <<02f413b1-6aa6-4095-a9a9-a60c50faa5ca@default>
[not found] ` <<83io34p8yn.fsf@gnu.org>
2016-01-08 15:03 ` Drew Adams
[not found] <<<<<660d1a46-fd5a-460f-8f26-50ba9a52cc60@default>
[not found] ` <<<<<83mvsmadhv.fsf@gnu.org>
[not found] ` <<<<940d59ef-c3d6-42df-ba0c-602cc154d431@default>
[not found] ` <<<<834meua953.fsf@gnu.org>
[not found] ` <<<68435980-44a2-4d33-8f64-8fbbdedb4a4e@default>
[not found] ` <<<83vb7a8pxi.fsf@gnu.org>
[not found] ` <<8b4cae3f-6f4c-4784-85fc-36a659fe15fd@default>
[not found] ` <<83r3hy8owz.fsf@gnu.org>
2016-01-03 19:54 ` Drew Adams
2016-01-08 12:11 ` Eli Zaretskii
[not found] <<<<660d1a46-fd5a-460f-8f26-50ba9a52cc60@default>
[not found] ` <<<<83mvsmadhv.fsf@gnu.org>
[not found] ` <<<940d59ef-c3d6-42df-ba0c-602cc154d431@default>
[not found] ` <<<834meua953.fsf@gnu.org>
[not found] ` <<68435980-44a2-4d33-8f64-8fbbdedb4a4e@default>
[not found] ` <<83vb7a8pxi.fsf@gnu.org>
2016-01-03 18:52 ` Drew Adams
2016-01-03 19:05 ` Eli Zaretskii
[not found] <<<660d1a46-fd5a-460f-8f26-50ba9a52cc60@default>
[not found] ` <<<83mvsmadhv.fsf@gnu.org>
[not found] ` <<940d59ef-c3d6-42df-ba0c-602cc154d431@default>
[not found] ` <<834meua953.fsf@gnu.org>
2016-01-03 18:00 ` Drew Adams
2016-01-03 18:43 ` Eli Zaretskii
2016-01-03 4:12 Drew Adams
2016-01-03 15:28 ` Eli Zaretskii
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=940d59ef-c3d6-42df-ba0c-602cc154d431@default \
--to=drew.adams@oracle.com \
--cc=22296@debbugs.gnu.org \
--cc=eliz@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.