Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/make make(1): remove trailing Z from buffer functions
details: https://anonhg.NetBSD.org/src/rev/e4e84814df64
branches: trunk
changeset: 974684:e4e84814df64
user: rillig <rillig%NetBSD.org@localhost>
date: Sat Aug 08 18:54:04 2020 +0000
description:
make(1): remove trailing Z from buffer functions
This Z had been useful during the migration from int to size_t. This
migration is finished, at least for the Buffer type, so the Z is no
longer necessary.
diffstat:
usr.bin/make/buf.c | 18 ++++++++--------
usr.bin/make/buf.h | 8 +++---
usr.bin/make/cond.c | 14 ++++++------
usr.bin/make/for.c | 14 ++++++------
usr.bin/make/main.c | 10 ++++----
usr.bin/make/var.c | 58 ++++++++++++++++++++++++++--------------------------
6 files changed, 61 insertions(+), 61 deletions(-)
diffs (truncated from 513 to 300 lines):
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/buf.c
--- a/usr.bin/make/buf.c Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/buf.c Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: buf.c,v 1.31 2020/08/03 20:26:09 rillig Exp $ */
+/* $NetBSD: buf.c,v 1.32 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -70,14 +70,14 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: buf.c,v 1.31 2020/08/03 20:26:09 rillig Exp $";
+static char rcsid[] = "$NetBSD: buf.c,v 1.32 2020/08/08 18:54:04 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
#if 0
static char sccsid[] = "@(#)buf.c 8.1 (Berkeley) 6/6/93";
#else
-__RCSID("$NetBSD: buf.c,v 1.31 2020/08/03 20:26:09 rillig Exp $");
+__RCSID("$NetBSD: buf.c,v 1.32 2020/08/08 18:54:04 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -104,7 +104,7 @@
/* Add the given bytes to the buffer. */
void
-Buf_AddBytesZ(Buffer *bp, const Byte *bytesPtr, size_t numBytes)
+Buf_AddBytes(Buffer *bp, const Byte *bytesPtr, size_t numBytes)
{
size_t count = bp->count;
Byte *ptr;
@@ -124,14 +124,14 @@
void
Buf_AddBytesBetween(Buffer *bp, const char *start, const char *end)
{
- Buf_AddBytesZ(bp, start, (size_t)(end - start));
+ Buf_AddBytes(bp, start, (size_t)(end - start));
}
/* Add the given string to the buffer. */
void
Buf_AddStr(Buffer *bp, const char *str)
{
- Buf_AddBytesZ(bp, str, strlen(str));
+ Buf_AddBytes(bp, str, strlen(str));
}
/* Add the given number to the buffer. */
@@ -147,7 +147,7 @@
char buf[1 + (bits + 2) / 3 + 1];
size_t len = (size_t)snprintf(buf, sizeof buf, "%d", n);
- Buf_AddBytesZ(bp, buf, len);
+ Buf_AddBytes(bp, buf, len);
}
/* Get the data (usually a string) from the buffer.
@@ -157,7 +157,7 @@
* Returns the pointer to the data and optionally the length of the
* data in the buffer. */
Byte *
-Buf_GetAllZ(Buffer *bp, size_t *numBytesPtr)
+Buf_GetAll(Buffer *bp, size_t *numBytesPtr)
{
if (numBytesPtr != NULL)
*numBytesPtr = bp->count;
@@ -175,7 +175,7 @@
/* Initialize a buffer.
* If the given initial size is 0, a reasonable default is used. */
void
-Buf_InitZ(Buffer *bp, size_t size)
+Buf_Init(Buffer *bp, size_t size)
{
if (size <= 0) {
size = BUF_DEF_SIZE;
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/buf.h
--- a/usr.bin/make/buf.h Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/buf.h Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: buf.h,v 1.22 2020/08/01 21:40:49 rillig Exp $ */
+/* $NetBSD: buf.h,v 1.23 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -111,13 +111,13 @@
#define Buf_Size(bp) ((bp)->count)
void Buf_Expand_1(Buffer *);
-void Buf_AddBytesZ(Buffer *, const Byte *, size_t);
+void Buf_AddBytes(Buffer *, const Byte *, size_t);
void Buf_AddBytesBetween(Buffer *, const Byte *, const Byte *);
void Buf_AddStr(Buffer *, const char *);
void Buf_AddInt(Buffer *, int);
-Byte *Buf_GetAllZ(Buffer *, size_t *);
+Byte *Buf_GetAll(Buffer *, size_t *);
void Buf_Empty(Buffer *);
-void Buf_InitZ(Buffer *, size_t);
+void Buf_Init(Buffer *, size_t);
Byte *Buf_Destroy(Buffer *, Boolean);
Byte *Buf_DestroyCompact(Buffer *);
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/cond.c
--- a/usr.bin/make/cond.c Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/cond.c Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: cond.c,v 1.91 2020/08/08 17:03:04 rillig Exp $ */
+/* $NetBSD: cond.c,v 1.92 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -70,14 +70,14 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: cond.c,v 1.91 2020/08/08 17:03:04 rillig Exp $";
+static char rcsid[] = "$NetBSD: cond.c,v 1.92 2020/08/08 18:54:04 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
#if 0
static char sccsid[] = "@(#)cond.c 8.2 (Berkeley) 1/2/94";
#else
-__RCSID("$NetBSD: cond.c,v 1.91 2020/08/08 17:03:04 rillig Exp $");
+__RCSID("$NetBSD: cond.c,v 1.92 2020/08/08 18:54:04 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -221,7 +221,7 @@
* Create a buffer for the argument and start it out at 16 characters
* long. Why 16? Why not?
*/
- Buf_InitZ(&buf, 16);
+ Buf_Init(&buf, 16);
paren_depth = 0;
for (;;) {
@@ -255,7 +255,7 @@
cp++;
}
- *argPtr = Buf_GetAllZ(&buf, &argLen);
+ *argPtr = Buf_GetAll(&buf, &argLen);
Buf_Destroy(&buf, FALSE);
while (*cp == ' ' || *cp == '\t') {
@@ -399,7 +399,7 @@
int qt;
const char *start;
- Buf_InitZ(&buf, 0);
+ Buf_Init(&buf, 0);
str = NULL;
*freeIt = NULL;
*quoted = qt = *condExpr == '"' ? 1 : 0;
@@ -490,7 +490,7 @@
}
}
got_str:
- *freeIt = Buf_GetAllZ(&buf, NULL);
+ *freeIt = Buf_GetAll(&buf, NULL);
str = *freeIt;
cleanup:
Buf_Destroy(&buf, FALSE);
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/for.c
--- a/usr.bin/make/for.c Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/for.c Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: for.c,v 1.61 2020/08/03 20:26:09 rillig Exp $ */
+/* $NetBSD: for.c,v 1.62 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1992, The Regents of the University of California.
@@ -30,14 +30,14 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: for.c,v 1.61 2020/08/03 20:26:09 rillig Exp $";
+static char rcsid[] = "$NetBSD: for.c,v 1.62 2020/08/08 18:54:04 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
#if 0
static char sccsid[] = "@(#)for.c 8.1 (Berkeley) 6/6/93";
#else
-__RCSID("$NetBSD: for.c,v 1.61 2020/08/03 20:26:09 rillig Exp $");
+__RCSID("$NetBSD: for.c,v 1.62 2020/08/08 18:54:04 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -270,7 +270,7 @@
}
}
- Buf_InitZ(&new_for->buf, 0);
+ Buf_Init(&new_for->buf, 0);
accumFor = new_for;
forLevel = 1;
return 1;
@@ -364,7 +364,7 @@
if (ch == '$') {
size_t len = for_var_len(item);
if (len != 0) {
- Buf_AddBytesZ(cmds, item - 1, len + 1);
+ Buf_AddBytes(cmds, item - 1, len + 1);
item += len;
continue;
}
@@ -409,9 +409,9 @@
* to contrive a makefile where an unwanted substitution happens.
*/
- cmd_cp = Buf_GetAllZ(&arg->buf, &cmd_len);
+ cmd_cp = Buf_GetAll(&arg->buf, &cmd_len);
body_end = cmd_cp + cmd_len;
- Buf_InitZ(&cmds, cmd_len + 256);
+ Buf_Init(&cmds, cmd_len + 256);
for (cp = cmd_cp; (cp = strchr(cp, '$')) != NULL;) {
char ech;
ch = *++cp;
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/main.c
--- a/usr.bin/make/main.c Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/main.c Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: main.c,v 1.296 2020/08/03 20:26:09 rillig Exp $ */
+/* $NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -69,7 +69,7 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: main.c,v 1.296 2020/08/03 20:26:09 rillig Exp $";
+static char rcsid[] = "$NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
@@ -81,7 +81,7 @@
#if 0
static char sccsid[] = "@(#)main.c 8.3 (Berkeley) 3/19/94";
#else
-__RCSID("$NetBSD: main.c,v 1.296 2020/08/03 20:26:09 rillig Exp $");
+__RCSID("$NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -1661,14 +1661,14 @@
(void)close(fds[1]);
savederr = 0;
- Buf_InitZ(&buf, 0);
+ Buf_Init(&buf, 0);
/* XXX: split variable cc into 2 */
do {
char result[BUFSIZ];
cc = read(fds[0], result, sizeof(result));
if (cc > 0)
- Buf_AddBytesZ(&buf, result, (size_t)cc);
+ Buf_AddBytes(&buf, result, (size_t)cc);
}
while (cc > 0 || (cc == -1 && errno == EINTR));
if (cc == -1)
diff -r 6bffa7e0c92c -r e4e84814df64 usr.bin/make/var.c
--- a/usr.bin/make/var.c Sat Aug 08 18:50:11 2020 +0000
+++ b/usr.bin/make/var.c Sat Aug 08 18:54:04 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: var.c,v 1.432 2020/08/08 18:50:11 rillig Exp $ */
+/* $NetBSD: var.c,v 1.433 2020/08/08 18:54:04 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -69,14 +69,14 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: var.c,v 1.432 2020/08/08 18:50:11 rillig Exp $";
+static char rcsid[] = "$NetBSD: var.c,v 1.433 2020/08/08 18:54:04 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
#if 0
static char sccsid[] = "@(#)var.c 8.3 (Berkeley) 3/19/94";
#else
-__RCSID("$NetBSD: var.c,v 1.432 2020/08/08 18:50:11 rillig Exp $");
+__RCSID("$NetBSD: var.c,v 1.433 2020/08/08 18:54:04 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -381,8 +381,8 @@
v->name = bmake_strdup(name);
len = strlen(env);
- Buf_InitZ(&v->val, len + 1);
- Buf_AddBytesZ(&v->val, env, len);
+ Buf_Init(&v->val, len + 1);
Home |
Main Index |
Thread Index |
Old Index