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): only clean up the Buffer data in CLEAN...



details:   https://anonhg.NetBSD.org/src/rev/cdd81f782553
branches:  trunk
changeset: 959056:cdd81f782553
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sat Jan 30 20:59:29 2021 +0000

description:
make(1): only clean up the Buffer data in CLEANUP mode

Cleaning up the members is only useful during debugging but not during
use in production.

diffstat:

 usr.bin/make/buf.c |  15 ++++++++++-----
 1 files changed, 10 insertions(+), 5 deletions(-)

diffs (65 lines):

diff -r 8c77da26bcb4 -r cdd81f782553 usr.bin/make/buf.c
--- a/usr.bin/make/buf.c        Sat Jan 30 20:53:29 2021 +0000
+++ b/usr.bin/make/buf.c        Sat Jan 30 20:59:29 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: buf.c,v 1.48 2021/01/30 20:53:29 rillig Exp $  */
+/*     $NetBSD: buf.c,v 1.49 2021/01/30 20:59:29 rillig Exp $  */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -75,7 +75,7 @@
 #include "make.h"
 
 /*     "@(#)buf.c      8.1 (Berkeley) 6/6/93"  */
-MAKE_RCSID("$NetBSD: buf.c,v 1.48 2021/01/30 20:53:29 rillig Exp $");
+MAKE_RCSID("$NetBSD: buf.c,v 1.49 2021/01/30 20:59:29 rillig Exp $");
 
 /* Make space in the buffer for adding at least 16 more bytes. */
 void
@@ -176,30 +176,34 @@
 
 /*
  * Free the data from the buffer.
- * The buffer is left in an indeterminate state.
+ * Leave the buffer itself in an indeterminate state.
  */
 void
 Buf_Done(Buffer *buf)
 {
        free(buf->data);
 
+#ifdef CLEANUP
        buf->cap = 0;
        buf->len = 0;
        buf->data = NULL;
+#endif
 }
 
 /*
  * Return the data from the buffer.
- * The buffer is left in an indeterminate state.
+ * Leave the buffer itself in an indeterminate state.
  */
 char *
 Buf_DoneData(Buffer *buf)
 {
        char *data = buf->data;
 
+#ifdef CLEANUP
        buf->cap = 0;
        buf->len = 0;
        buf->data = NULL;
+#endif
 
        return data;
 }
@@ -209,7 +213,8 @@
 #endif
 
 /*
- * Reset the buffer and return its data.
+ * Return the data from the buffer.
+ * Leave the buffer itself in an indeterminate state.
  *
  * If the buffer size is much greater than its content,
  * a new buffer will be allocated and the old one freed.



Home | Main Index | Thread Index | Old Index