From owner-freebsd-cvsweb@FreeBSD.ORG  Fri May  6 16:39:06 2005
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 7E49916A4D4
	for <freebsd-cvsweb@freebsd.org>;
	Fri,  6 May 2005 16:39:06 +0000 (GMT)
Received: from wproxy.gmail.com (wproxy.gmail.com [64.233.184.201])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 279E643D41
	for <freebsd-cvsweb@freebsd.org>;
	Fri,  6 May 2005 16:39:06 +0000 (GMT)
	(envelope-from jpnairn@gmail.com)
Received: by wproxy.gmail.com with SMTP id 71so809752wra
	for <freebsd-cvsweb@freebsd.org>;
	Fri, 06 May 2005 09:39:02 -0700 (PDT)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
	s=beta; d=gmail.com;
	h=received:message-id:date:from:reply-to:to:subject:mime-version:content-type:content-transfer-encoding:content-disposition;
	b=Q95nVw/TH/aFz4FBJ6Sx5ZH2PyrpZwfBwo742VBBPPnVePV6n0q670I4G/NjmgzC+CqQm8PgjUAVsRXi74DtzK9YIze83u7aXaeOXjllsiyv2AKWp1xWtrZYYCkW+JysdUnG9Vdx+ltoqpnv0aqxnE+Dj1WFRG9qbjRy9tVZkEM=
Received: by 10.54.36.13 with SMTP id j13mr968919wrj;
        Fri, 06 May 2005 09:39:02 -0700 (PDT)
Received: by 10.54.153.4 with HTTP; Fri, 6 May 2005 09:39:02 -0700 (PDT)
Message-ID: <3bccd73d050506093958f6c0d2@mail.gmail.com>
Date: Fri, 6 May 2005 09:39:02 -0700
From: Jerry Nairn <jpnairn@gmail.com>
To: freebsd-cvsweb@freebsd.org
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
Subject: Re: Log view diff line
X-BeenThere: freebsd-cvsweb@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
Reply-To: Jerry Nairn <jpnairn@gmail.com>
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: Fri, 06 May 2005 16:39:06 -0000

> Thoughts about the attached patch?  It uses a semicolon for separating
> the diff link "categories", moves the "to" to be only at the beginning
> of the line, and adds parens around the displayed revision numbers.  I'm
> not sure if the parens add any value though.

If it were my choice, I would not add the parens around the revision
number, but it's not anything I feel strongly about.
I like the other changes.

Jerry Nairn