* bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command
@ 2012-09-17 16:01 Drew Adams
2012-09-17 17:39 ` Drew Adams
2012-09-22 15:47 ` Chong Yidong
0 siblings, 2 replies; 5+ messages in thread
From: Drew Adams @ 2012-09-17 16:01 UTC (permalink / raw
To: 12462
C-h k C-x C-q
C-x C-q runs the command toggle-read-only, which is an interactive
compiled Lisp function in `files.el'.
It is bound to C-x C-q.
(toggle-read-only &optional ARG INTERACTIVE)
This function is obsolete since 24.3;
use `read-only-mode' instead.
Not documented.
If the command is obsolete, why is the key bound to it and not to its
replacement?
Why does the help say "Not documented"? Well, because there is no
longer any doc string. That's a second bug: users now have no doc for
the key. They can follow the link to the doc for `read-only-mode', but
that is not the command that is invoked.
If `C-x C-q' still invokes `toggle-read-only', then `C-h k C-x C-q'
should still provide the doc for `toggle-read-only'. You removed the
doc string, which explains what `toggle-read-only' does.
In GNU Emacs 24.2.50.1 (i386-mingw-nt5.1.2600)
of 2012-09-17 on MARVIN
Bzr revision: 110062 cyd@gnu.org-20120917054104-r93rtwkrtva73ewe
Windowing system distributor `Microsoft Corp.', version 5.1.2600
Configured using:
`configure --with-gcc (4.7) --no-opt --enable-checking --cflags
-ID:/devel/emacs/libs/libXpm-3.5.8/include
-ID:/devel/emacs/libs/libXpm-3.5.8/src
-ID:/devel/emacs/libs/libpng-dev_1.4.3-1/include
-ID:/devel/emacs/libs/zlib-dev_1.2.5-2/include
-ID:/devel/emacs/libs/giflib-4.1.4-1/include
-ID:/devel/emacs/libs/jpeg-6b-4/include
-ID:/devel/emacs/libs/tiff-3.8.2-1/include
-ID:/devel/emacs/libs/gnutls-3.0.9/include
-ID:/devel/emacs/libs/libiconv-1.13.1-1-dev/include
-ID:/devel/emacs/libs/libxml2-2.7.8/include/libxml2'
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command
2012-09-17 16:01 bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command Drew Adams
@ 2012-09-17 17:39 ` Drew Adams
2012-09-22 15:47 ` Chong Yidong
1 sibling, 0 replies; 5+ messages in thread
From: Drew Adams @ 2012-09-17 17:39 UTC (permalink / raw
To: 12462
Let me add that the doc string for `read-only-mode' is also not up-to-date, in
the sense that it, in turn, refers back to `toggle-read-only'.
If the latter is really deprecated (let alone obsolete), then the doc should be
updated to reflect this: `read-only-mode' doc should stand alone etc.
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command
2012-09-17 16:01 bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command Drew Adams
2012-09-17 17:39 ` Drew Adams
@ 2012-09-22 15:47 ` Chong Yidong
2012-09-22 22:58 ` Juri Linkov
1 sibling, 1 reply; 5+ messages in thread
From: Chong Yidong @ 2012-09-22 15:47 UTC (permalink / raw
To: Drew Adams; +Cc: 12462
"Drew Adams" <drew.adams@oracle.com> writes:
> If the command is obsolete, why is the key bound to it and not to its
> replacement?
Fixed in trunk.
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command
2012-09-22 15:47 ` Chong Yidong
@ 2012-09-22 22:58 ` Juri Linkov
2012-09-23 10:22 ` Chong Yidong
0 siblings, 1 reply; 5+ messages in thread
From: Juri Linkov @ 2012-09-22 22:58 UTC (permalink / raw
To: Chong Yidong; +Cc: 12462
>> If the command is obsolete, why is the key bound to it and not to its
>> replacement?
>
> Fixed in trunk.
I added [remap read-only-mode] to dired.el because this change
broke wdired. An old [remap toggle-read-only] could still remain for
backward-compatibility.
Also I'm not sure whether `toggle-read-only' should be replaced with
`read-only-mode' in the function `dired-toggle-read-only' and maybe the
name of the function `dired-toggle-read-only' should be changed to
`dired-read-only-mode' in a similar way?
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command
2012-09-22 22:58 ` Juri Linkov
@ 2012-09-23 10:22 ` Chong Yidong
0 siblings, 0 replies; 5+ messages in thread
From: Chong Yidong @ 2012-09-23 10:22 UTC (permalink / raw
To: Juri Linkov; +Cc: 12462
Juri Linkov <juri@jurta.org> writes:
> Also I'm not sure whether `toggle-read-only' should be replaced with
> `read-only-mode' in the function `dired-toggle-read-only'
Yes, this should use (read-only-mode 'toggle). I changed that and
several other similar use-cases in the trunk. Thanks.
> and maybe the name of the function `dired-toggle-read-only' should be
> changed to `dired-read-only-mode' in a similar way?
I don't think such a name change would be apropriate, since
dired-read-only-mode is not a separate minor mode.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-09-23 10:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-17 16:01 bug#12462: 24.2.50; `C-x C-q' is bound to an obsolete command Drew Adams
2012-09-17 17:39 ` Drew Adams
2012-09-22 15:47 ` Chong Yidong
2012-09-22 22:58 ` Juri Linkov
2012-09-23 10:22 ` Chong Yidong
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.