bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#4531: marked as done (23.1; browse-url with unknown func in browse-u


From: Emacs bug Tracking System
Subject: bug#4531: marked as done (23.1; browse-url with unknown func in browse-url-browser-function)
Date: Fri, 02 Oct 2009 07:20:06 +0000

Your message dated Fri, 02 Oct 2009 03:11:17 -0400
with message-id <address@hidden>
and subject line Re: bug#4531: 23.1; browse-url with unknown func in 
browse-url-browser-function
has caused the Emacs bug report #4531,
regarding 23.1; browse-url with unknown func in browse-url-browser-function
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact address@hidden
immediately.)


-- 
4531: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=4531
Emacs Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 23.1; browse-url with unknown func in browse-url-browser-function Date: Wed, 23 Sep 2009 11:22:37 +1000 User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.1 (gnu/linux)
If browse-url-browser-function is an unknown symbol the error from
browse-url isn't very helpful.  Eg.

    M-: (setq browse-url-browser-function 'FOOBAR-browse-url)
    M-x browse-url http://www.gnu.org

    => Wrong type argument: listp, FOOBAR-browse-url

I hoped it would say the problem is the symbol is unbound, not that it's
supposed to be a list.  (The value can be a list of course, but if the
value is a symbol then it's surely meant to be the name of a function.)
Sending symbols down the `apply' leg of `browse-url' would give

    => Symbol's function definition is void: FOOBAR-browse-url

which I think is a much better.


2009-09-23  Kevin Ryde  <address@hidden>

        * net/browse-url.el (browse-url): Pass any symbol in
        browse-url-browser-function to `apply', since if you've mistakenly
        put an unbound symbol then the error from apply is much clearer
        that the function you wanted is not available.

--- browse-url.el.~1.80.~       2009-09-23 11:08:37.000000000 +1000
+++ browse-url.el       2009-09-23 11:16:22.000000000 +1000
@@ -778,7 +778,11 @@
     ;; which may not even exist any more.
     (if (stringp (frame-parameter (selected-frame) 'display))
         (setenv "DISPLAY" (frame-parameter (selected-frame) 'display)))
-    (if (functionp browse-url-browser-function)
+    ;; send any symbol to `apply', not just fboundp ones, since
+    ;; void-function from apply is clearer than wrong-type-argument
+    ;; from dolist
+    (if (or (symbolp browse-url-browser-function)
+            (functionp browse-url-browser-function))
         (apply browse-url-browser-function url args)
       ;; The `function' can be an alist; look down it for first match
       ;; and apply the function (which might be a lambda).

In GNU Emacs 23.1.1 (i486-pc-linux-gnu, GTK+ Version 2.16.5)
 of 2009-08-03 on raven, modified by Debian
configured using `configure  '--build=i486-linux-gnu' '--host=i486-linux-gnu' 
'--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' 
'--localstatedir=/var/lib' '--infodir=/usr/share/info' 
'--mandir=/usr/share/man' '--with-pop=yes' 
'--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.1/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.1/leim'
 '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 
'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g 
-O2' 'LDFLAGS=-g' 'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_AU
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default-enable-multibyte-characters: t

--- End Message ---
--- Begin Message --- Subject: Re: bug#4531: 23.1; browse-url with unknown func in browse-url-browser-function Date: Fri, 02 Oct 2009 03:11:17 -0400 User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
Thanks; applied.

--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]