From owner-freebsd-cvsweb@FreeBSD.ORG  Wed Aug 11 11:10:19 2004
Return-Path: <owner-freebsd-cvsweb@FreeBSD.ORG>
Delivered-To: freebsd-cvsweb@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 5133816A4CF
	for <freebsd-cvsweb@freebsd.org>;
	Wed, 11 Aug 2004 11:10:19 +0000 (GMT)
Received: from smtp2.pp.htv.fi (smtp2.pp.htv.fi [213.243.153.35])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 18BB143D2D
	for <freebsd-cvsweb@freebsd.org>;
	Wed, 11 Aug 2004 11:10:19 +0000 (GMT)
	(envelope-from scop@FreeBSD.org)
Received: from [62.78.130.106] (cs78130106.pp.htv.fi [62.78.130.106])
	by smtp2.pp.htv.fi (Postfix) with ESMTP id BD34F29727C
	for <freebsd-cvsweb@freebsd.org>;
	Wed, 11 Aug 2004 14:10:17 +0300 (EEST)
From: Ville =?ISO-8859-1?Q?Skytt=E4?= <scop@FreeBSD.org>
To: "'freebsd-cvsweb@freebsd.org'" <freebsd-cvsweb@freebsd.org>
In-Reply-To: <762A181C0D966E459A2EB3B810142C7707B2199A@nmrusdunsx12.nmrlan.net>
References: 
	 <762A181C0D966E459A2EB3B810142C7707B2199A@nmrusdunsx12.nmrlan.net>
Content-Type: text/plain
Organization: FreeBSD
Message-Id: <1092222617.2909.614.camel@bobcat.mine.nu>
Mime-Version: 1.0
X-Mailer: Ximian Evolution 1.4.6 (1.4.6-2) 
Date: Wed, 11 Aug 2004 14:10:17 +0300
Content-Transfer-Encoding: 7bit
Subject: Re: Bug in sub hrefquote
X-BeenThere: freebsd-cvsweb@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS Web maintenance mailing list <freebsd-cvsweb.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-cvsweb>,
	<mailto:freebsd-cvsweb-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-cvsweb>
List-Post: <mailto:freebsd-cvsweb@freebsd.org>
List-Help: <mailto:freebsd-cvsweb-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-cvsweb>,
	<mailto:freebsd-cvsweb-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Aug 2004 11:10:19 -0000

> >> 	There's probably other ways to do this (like urlencode), but this
> >> seems simpler.
> 
> >I would like to get rid of hrefquote() altogether.  Will look into it
> >soonish.

Ok, that's done now.  I went through all of the code reviewing URI
escaping and fixed a number of bugs and got rid of urlencode() and
hrefquote() altogether in favour of using stuff from URI::Escape.

Unless something new pops up, I'm going to release 3.0.2 soonish from
what is currently in CVS HEAD.  Testing (and reporting how it works :)
would be welcome.