From: Paul Eggert <eggert@cs.ucla.edu>
To: "Stephen J. Turnbull" <stephen@xemacs.org>
Cc: Glenn Morris <rgm@gnu.org>, emacs-devel@gnu.org
Subject: Re: master f9e354e: Spelling and wording fixes
Date: Tue, 17 Mar 2015 21:00:53 -0700 [thread overview]
Message-ID: <5508F875.7050407@cs.ucla.edu> (raw)
In-Reply-To: <87bnjrrofq.fsf@uwakimon.sk.tsukuba.ac.jp>
[-- Attachment #1: Type: text/plain, Size: 372 bytes --]
Stephen J. Turnbull wrote:
> semantically your phrasing seems quite awkward to me
True. Let's omit that sentence. First, the point doesn't appear to be that
important. Second, it's typically more confusing to document a procedure
*doesn't* do than to document what the procedure *does* do. So I installed the
attached. Undoubtedly this could be further improved.
[-- Attachment #2: 0001-lisp-net-browse-url.el-Omit-confusing-documentation.patch --]
[-- Type: text/x-patch, Size: 1445 bytes --]
From 94a90c867853f83d5f833dfd910919af9209c71b Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Tue, 17 Mar 2015 20:55:02 -0700
Subject: [PATCH] * lisp/net/browse-url.el: Omit confusing documentation.
---
lisp/net/browse-url.el | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/lisp/net/browse-url.el b/lisp/net/browse-url.el
index dff587e..b44bb71 100644
--- a/lisp/net/browse-url.el
+++ b/lisp/net/browse-url.el
@@ -862,8 +862,7 @@ first, if that exists."
;;;###autoload
(defun browse-url-at-point (&optional arg)
"Ask a WWW browser to load the URL at or before point.
-Don't let you edit the URL like `browse-url'. Variable
-`browse-url-browser-function' says which browser to use."
+Variable `browse-url-browser-function' says which browser to use."
(interactive "P")
(let ((url (browse-url-url-at-point)))
(if url
@@ -876,9 +875,8 @@ Don't let you edit the URL like `browse-url'. Variable
(defun browse-url-at-mouse (event)
"Ask a WWW browser to load a URL clicked with the mouse.
The URL is the one around or before the position of the mouse click
-but point is not changed. Don't let you edit the URL like
-`browse-url'. Variable `browse-url-browser-function' says which browser
-to use."
+but point is not changed. Variable `browse-url-browser-function'
+says which browser to use."
(interactive "e")
(save-excursion
(mouse-set-point event)
--
2.1.0
prev parent reply other threads:[~2015-03-18 4:00 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20150318001119.1234.13177@vcs.savannah.gnu.org>
[not found] ` <E1YY1Zv-0000KY-VT@vcs.savannah.gnu.org>
2015-03-18 0:51 ` master f9e354e: Spelling and wording fixes Glenn Morris
2015-03-18 0:52 ` Paul Eggert
2015-03-18 3:10 ` Stephen J. Turnbull
2015-03-18 4:00 ` Paul Eggert [this message]
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=5508F875.7050407@cs.ucla.edu \
--to=eggert@cs.ucla.edu \
--cc=emacs-devel@gnu.org \
--cc=rgm@gnu.org \
--cc=stephen@xemacs.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).