unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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


      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).