From: Richard Stallman <rms@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: transient-mark-mode in 22.0
Date: Wed, 08 Jun 2005 08:02:42 -0400 [thread overview]
Message-ID: <E1DfzGc-0006zB-Jp@fencepost.gnu.org> (raw)
In-Reply-To: <pan.2005.06.06.17.24.10.771015@as.arizona.edu> (message from JD Smith on Mon, 06 Jun 2005 10:24:11 -0700)
I noticed that setting the transient-mark-mode variable no longer does
anything
As far as I can see, it still does what it always did.
Did you actually observe that it fails to work?
But I see that the doc string of the variable says that setting it
does nothing. That doc string is erroneous.
define-minor-mode automatically generates doc strings that say you
can't set the variable directly, and often this is true, but sometimes
it is false. So I propose the patch below.
Your problem with mouse-up is probably unrelated to this. I am not
sure whether it is a bug, and I don't have time to look at the issue
now.
*** easy-mmode.el 05 Jun 2005 05:51:44 -0400 1.64
--- easy-mmode.el 08 Jun 2005 07:00:46 -0400
***************
*** 183,195 ****
(let ((curfile (or (and (boundp 'byte-compile-current-file)
byte-compile-current-file)
! load-file-name)))
! `(defcustom ,mode ,init-value
! ,(format "Non-nil if %s is enabled.
See the command `%s' for a description of this minor-mode.
Setting this variable directly does not take effect;
! use either \\[customize] or the function `%s'."
! pretty-name mode mode)
:set 'custom-set-minor-mode
:initialize 'custom-initialize-default
,@group
--- 183,201 ----
(let ((curfile (or (and (boundp 'byte-compile-current-file)
byte-compile-current-file)
! load-file-name))
! base-doc-string)
! (setq base-doc-string "Non-nil if %s is enabled.
See the command `%s' for a description of this minor-mode.
Setting this variable directly does not take effect;
! use either \\[customize] or the function `%s'.")
! (if (null body)
! (setq base-doc-string "Non-nil if %s is enabled.
! See the command `%s' for a description of this minor-mode."))
!
! `(defcustom ,mode ,init-value
! (let
! ,(format base-doc-string pretty-name mode mode)
:set 'custom-set-minor-mode
:initialize 'custom-initialize-default
,@group
next prev parent reply other threads:[~2005-06-08 12:02 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-06-06 17:24 transient-mark-mode in 22.0 JD Smith
2005-06-08 12:02 ` Richard Stallman [this message]
2005-06-08 21:17 ` JD Smith
2005-06-09 14:41 ` Richard Stallman
2005-06-09 17:54 ` JD Smith
2005-06-10 22:37 ` Richard Stallman
2005-06-10 22:49 ` JD Smith
2005-06-11 23:16 ` Richard Stallman
2005-06-12 16:29 ` Stefan Monnier
2005-06-12 16:09 ` Stefan Monnier
2005-06-13 15:02 ` Richard Stallman
2005-06-09 21:39 ` Stefan Monnier
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=E1DfzGc-0006zB-Jp@fencepost.gnu.org \
--to=rms@gnu.org \
--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 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).