Gnus development mailing list
 help / color / mirror / Atom feed
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


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