Subject: Re: CVS commit: src/bin/sh
To: Jaromir Dolecek <jdolecek@netbsd.org>
From: Luke Mewburn <lukem@NetBSD.org>
List: source-changes
Date: 12/22/2003 01:59:26
--xs+9IvWevLaxKUtW
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Dec 21, 2003 at 08:32:39AM +0000, Jaromir Dolecek wrote:
  |=20
  | Module Name:	src
  | Committed By:	jdolecek
  | Date:		Sun Dec 21 08:32:39 UTC 2003
  |=20
  | Modified Files:
  | 	src/bin/sh: expand.c
  |=20
  | Log Message:
  | minor optimization in evalvar()
  | change sent in bin/23813 by VaX#n8
  |=20
  |=20
  | To generate a diff of this commit:
  | cvs rdiff -r1.59 -r1.60 src/bin/sh/expand.c


Did you check the compiler output before & after this change?
I suspect that gcc generates the same code in either case.
If so, this was a gratitious change with no real benefit
for performance _or_ code readability.

--xs+9IvWevLaxKUtW
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (NetBSD)

iD8DBQE/5bVOpBhtmn8zJHIRAv80AKDHLmAtpQsvx2PKbgx9IIipUFO97QCdFrOz
SU2oSB7Cai2F5pTPZicRsjs=
=juSS
-----END PGP SIGNATURE-----

--xs+9IvWevLaxKUtW--