unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Drew Adams <drew.adams@oracle.com>
Cc: 19682@debbugs.gnu.org
Subject: bug#19682: 25.0.50; `other-window' doc string should say that by default it selects win	in same frame
Date: Sun, 25 Jan 2015 18:05:36 +0200	[thread overview]
Message-ID: <83y4oqrgkv.fsf@gnu.org> (raw)
In-Reply-To: <733fcda9-19ad-4096-aa8a-7f0f47062ac2@default>

> Date: Sat, 24 Jan 2015 19:33:46 -0800 (PST)
> From: Drew Adams <drew.adams@oracle.com>
> 
> The doc string does not describe the default behavior: select another
> window in the same frame, if there is one.
> 
> The doc string refers to the doc for `next-window', for the reader to
> find out what ALL-FRAMES does - even for the default, nil, case.
> 
> It is fine to refer to `next-window' for ALL-FRAMES (e.g. for full
> understanding and for non-interactive use).  But surely the default
> interactive behavior should be described.

The doc string says:

  This function uses `next-window' for finding the window to
  select.  The argument ALL-FRAMES has the same meaning as in
  `next-window', but the MINIBUF argument of `next-window' is
  always effectively nil.

This is a very clear invitation to read the documentation of
'next-window', if the reader cares about which window will be
selected.  If the reader indeed goes there, she will find there the
details of what you wanted her to know.

OTOH, many uses of 'other-window' don't really care about the order
and the candidates, in which case that argument is of no importance
for them.

So I see no reason to think this doc strings lacks something
important.





  reply	other threads:[~2015-01-25 16:05 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-25  3:33 bug#19682: 25.0.50; `other-window' doc string should say that by default it selects win in same frame Drew Adams
2015-01-25 16:05 ` Eli Zaretskii [this message]
     [not found] <<733fcda9-19ad-4096-aa8a-7f0f47062ac2@default>
     [not found] ` <<83y4oqrgkv.fsf@gnu.org>
2015-01-25 17:12   ` Drew Adams
2015-01-25 17:48     ` Eli Zaretskii
2016-04-30 16:44       ` Lars Ingebrigtsen
     [not found] <<c78c5328-c0eb-4b91-afa1-2dbae2d03280@default>
     [not found] ` <<83ppa2rbta.fsf@gnu.org>
2015-01-25 19:05   ` Drew Adams
2015-01-25 19:26     ` Eli Zaretskii
     [not found] <<b0accd37-8912-4873-ad3c-2816b55cdaaa@default>
     [not found] ` <<83iofur7aj.fsf@gnu.org>
2015-01-25 23:40   ` Drew Adams
2015-01-26  6:30     ` 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

  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=83y4oqrgkv.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=19682@debbugs.gnu.org \
    --cc=drew.adams@oracle.com \
    /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).