emacs-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] eww improvements


From: Rüdiger Sonderfeld
Subject: Re: [PATCH] eww improvements
Date: Sun, 23 Jun 2013 23:48:21 +0200
User-agent: KMail/4.10.3 (Linux/3.8.0-23-generic; KDE/4.10.3; x86_64; ; )

On Sunday 23 June 2013 20:29:55 Lars Magne Ingebrigtsen wrote:
> Ivan Kanis <address@hidden> writes:
> > I have attached 3 patches for eww.
> 
> Thanks; all applied.

http://bzr.savannah.gnu.org/lh/emacs/trunk/revision/113144

I don't think

+  (cond ((eq system-type 'windows-nt)
+        'browse-url-default-windows-browser)
+       ((eq system-type 'darwin)
+        'browse-url-default-macosx-browser)
+       (t
+        'browse-url-netscape))

is a good choice for `eww-external-browser'.  Who still uses netscape? And why 
duplicate all the effort to find the default browser when this is already done 
in browse-url.el?

I think using `browse-url-default-browser' as default value makes more sense.

We could also try to use `browse-url-browser-function', unless it is set to 
eww itself:

  (if (memq browse-url-browser-function '(eww-browse-url eww))
        #'browse-url-default-browser
      browse-url-browser-function)

Regards,
Rüdiger




reply via email to

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