From: Simon Josefsson <jas@pdc.kth.se>
Cc: ding@gnus.org
Subject: Re: browse-url button following mangles commas in URLs
Date: 25 Sep 1999 13:57:49 +0200 [thread overview]
Message-ID: <iluemfnursy.fsf@badis.pdc.kth.se> (raw)
In-Reply-To: Neil Crellin's message of "24 Sep 1999 10:02:16 -0700"
Neil Crellin <neilc@wallaby.cc> writes:
> When presented with a buttonized url in gnus such as the following:
>
> <URL:http://www.zdnet.com/pcweek/stories/news/0,4153,2340538,00.html>
>
> if I click on the button, the URL is mangled with every comma replaced
> by 2c like so:
>
> <URL:http://www.zdnet.com/pcweek/stories/news/02c41532c23405382c00.html>
>
> Now %2c in place of "," would be fine, but just 2c is no good.
>
> Can anyone see where this is occuring, I found this too convoluted to
> debug properly, sorry.
The bug is fixed in Emacs 20.4. I believe the patch below would work
(stolen from gnu.emacs.bug). It's not a gnus problem.
*** browse-url.el Mon Sep 13 10:54:56 1999
--- browse-url.el~ Fri Jun 19 18:02:11 1998
***************
*** 700,706 ****
;; include at least commas; presumably also close parens.
(while (string-match "[,)]" url)
(setq url (replace-match
! (format "%%%x" (string-to-char (match-string 0 url))) t t url)))
(let* ((process-environment (browse-url-process-environment))
(process (apply 'start-process
(concat "netscape " url) nil
--- 700,706 ----
;; include at least commas; presumably also close parens.
(while (string-match "[,)]" url)
(setq url (replace-match
! (format "%x" (string-to-char (match-string 0 url))) t t url)))
(let* ((process-environment (browse-url-process-environment))
(process (apply 'start-process
(concat "netscape " url) nil
prev parent reply other threads:[~1999-09-25 11:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
1999-09-24 17:02 Neil Crellin
1999-09-25 10:09 ` Lars Magne Ingebrigtsen
1999-09-25 11:57 ` Simon Josefsson [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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=iluemfnursy.fsf@badis.pdc.kth.se \
--to=jas@pdc.kth.se \
--cc=ding@gnus.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.
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 NNTP newsgroup(s).