Module Name: src
Committed By: abhinav
Date: Sun Apr 30 15:27:24 UTC 2017
Modified Files:
src/usr.sbin/makemandb: apropos-utils.c
Log Message:
Instead of dereferencing the pointer passed in as function argument, use a
temporary local buffer. Saves the cost of pointer dereferencing at so many
places.
To generate a diff of this commit:
cvs rdiff -u -r1.34 -r1.35 src/usr.sbin/makemandb/apropos-utils.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/usr.sbin/makemandb/apropos-utils.c
diff -u src/usr.sbin/makemandb/apropos-utils.c:1.34 src/usr.sbin/makemandb/apropos-utils.c:1.35
--- src/usr.sbin/makemandb/apropos-utils.c:1.34 Sun Apr 30 14:53:58 2017
+++ src/usr.sbin/makemandb/apropos-utils.c Sun Apr 30 15:27:24 2017
@@ -1,4 +1,4 @@
-/* $NetBSD: apropos-utils.c,v 1.34 2017/04/30 14:53:58 abhinav Exp $ */
+/* $NetBSD: apropos-utils.c,v 1.35 2017/04/30 15:27:24 abhinav Exp $ */
/*-
* Copyright (c) 2011 Abhinav Upadhyay <[email protected]>
* All rights reserved.
@@ -31,7 +31,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: apropos-utils.c,v 1.34 2017/04/30 14:53:58 abhinav Exp $");
+__RCSID("$NetBSD: apropos-utils.c,v 1.35 2017/04/30 15:27:24 abhinav Exp $");
#include <sys/queue.h>
#include <sys/stat.h>
@@ -111,31 +111,34 @@ void
concat2(char **dst, const char *src, size_t srclen)
{
size_t totallen, dstlen;
+ char *mydst = *dst;
assert(src != NULL);
/*
* If destination buffer dst is NULL, then simply
* strdup the source buffer
*/
- if (*dst == NULL) {
- *dst = estrndup(src, srclen);
+ if (mydst == NULL) {
+ mydst = estrndup(src, srclen);
+ *dst = mydst;
return;
}
- dstlen = strlen(*dst);
+ dstlen = strlen(mydst);
/*
* NUL Byte and separator space
*/
totallen = dstlen + srclen + 2;
- *dst = erealloc(*dst, totallen);
+ mydst = erealloc(mydst, totallen);
/* Append a space at the end of dst */
- (*dst)[dstlen++] = ' ';
+ mydst[dstlen++] = ' ';
/* Now, copy src at the end of dst */
- memcpy(*dst + dstlen, src, srclen);
- (*dst)[dstlen + srclen] = '\0';
+ memcpy(mydst + dstlen, src, srclen);
+ mydst[dstlen + srclen] = '\0';
+ *dst = mydst;
}
void
