PR/57993 CVS commit: src/usr.sbin/certctl

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR bin/57993; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57993 CVS commit: src/usr.sbin/certctl
Date: Sun, 3 Mar 2024 15:53:55 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sun Mar  3 15:53:55 UTC 2024
 
 Modified Files:
 	src/usr.sbin/certctl: certctl.sh
 
 Log Message:
 certctl(8): Avoid basename(1).
 
 Saves some time running subprocesses.  Since this is only used for
 non-directories (i.e., there's never trailing / on the inputs), it
 suffices to delete the longest prefix matching glob `*/' with shell
 parameter expansion -- much cheaper than spawning a subprocess.
 
 Shaves off about 1/3 of the time spent in `certctl list' on an
 aarch64 VM in qemu.
 
 PR bin/57993
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.5 -r1.6 src/usr.sbin/certctl/certctl.sh
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.