all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Eli Zaretskii'" <eliz@gnu.org>
Cc: 11361@debbugs.gnu.org
Subject: bug#11361: 24.1.50; doc string of `yank-pop-change-selection'
Date: Fri, 27 Apr 2012 08:34:36 -0700	[thread overview]
Message-ID: <21F070A715AC494CAD3EBDD54749EE53@us.oracle.com> (raw)
In-Reply-To: <83sjfpgr1m.fsf@gnu.org>

> I don't know why you decided to go after this doc string.  It is quite
> clear.

The manual is clear.  The doc string is worthless.

> > What on earth does "changes the window system selection"?
> 
> Would "sets the window system's primary selection" be clearer?

No.

> > What is "the window system selection"?
> 
> "Selection" is widely accepted terminology.  From the Emacs Glossary:

None of that helps.  And none of it mentions "window system selection".  And
none of it says anything close to what the Emacs manual says about this option.

> > How is it changed by setting this to non-nil?  What do nil 
> > and non-nil actually do?
> 
> The nil value doesn't do anything, obviously.

There must be a difference in behavior between nil and non-nil (or else it is
not only the doc string that is problematic).  That's the point.

Instead of playing word games, try clarifying what this option does.  What does
it mean to "change" the "window system selection"?  Change it how?  Change it to
what?  From what?

As I said:
> See the Emacs manual, node Clipboard (presumably) for inspiration.

There you will see that this option is apparently about _rotating_ the _kill
ring_ (perhaps among other things?):

 "if you change `yank-pop-change-selection' to `t', then `M-y'
  saves the new yank to the clipboard."

That at least is clear.  Whether that is all that the option does, I do not
know.  If so, then its doc string should speak in terms of rotating the kill
ring.  If it is more complicated than that, then additional explanation is
needed.






  reply	other threads:[~2012-04-27 15:34 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-27 14:18 bug#11361: 24.1.50; doc string of `yank-pop-change-selection' Drew Adams
2012-04-27 14:58 ` Eli Zaretskii
2012-04-27 15:34   ` Drew Adams [this message]
2012-04-27 15:46     ` Eli Zaretskii
2012-07-07  9:35       ` Chong Yidong
2012-07-07 15:35         ` Drew Adams

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=21F070A715AC494CAD3EBDD54749EE53@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=11361@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.