Module Name:    src
Committed By:   rillig
Date:           Sun Nov 29 09:42:54 UTC 2020

Modified Files:
        src/usr.bin/make: dir.c

Log Message:
make(1): remove comment about returning const

The callers modify the reference count of the CachedDir, therefore it
must be modifiable.


To generate a diff of this commit:
cvs rdiff -u -r1.230 -r1.231 src/usr.bin/make/dir.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.bin/make/dir.c
diff -u src/usr.bin/make/dir.c:1.230 src/usr.bin/make/dir.c:1.231
--- src/usr.bin/make/dir.c:1.230	Sun Nov 29 09:38:04 2020
+++ src/usr.bin/make/dir.c	Sun Nov 29 09:42:54 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: dir.c,v 1.230 2020/11/29 09:38:04 rillig Exp $	*/
+/*	$NetBSD: dir.c,v 1.231 2020/11/29 09:42:54 rillig Exp $	*/
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -136,7 +136,7 @@
 #include "job.h"
 
 /*	"@(#)dir.c	8.2 (Berkeley) 1/2/94"	*/
-MAKE_RCSID("$NetBSD: dir.c,v 1.230 2020/11/29 09:38:04 rillig Exp $");
+MAKE_RCSID("$NetBSD: dir.c,v 1.231 2020/11/29 09:42:54 rillig Exp $");
 
 #define DIR_DEBUG0(text) DEBUG0(DIR, text)
 #define DIR_DEBUG1(fmt, arg1) DEBUG1(DIR, fmt, arg1)
@@ -1435,10 +1435,6 @@ Dir_UpdateMTime(GNode *gn, Boolean reche
  */
 CachedDir *
 Dir_AddDir(SearchPath *path, const char *name)
-/*
- * XXX: Maybe return const CachedDir, as a hint that the return value must
- * not be freed since it is owned by openDirs.
- */
 {
 	CachedDir *dir = NULL;	/* the added directory */
 	DIR *d;
@@ -1554,12 +1550,7 @@ SearchPath_ToFlags(const char *flag, Sea
 	return Buf_Destroy(&buf, FALSE);
 }
 
-/* Nuke a directory descriptor, if possible. Callback procedure for the
- * suffixes module when destroying a search path.
- *
- * Input:
- *	dirp		The directory descriptor to nuke
- */
+/* Nuke a directory descriptor, if it is no longer used. */
 static void
 CachedDir_Destroy(CachedDir *dir)
 {

Reply via email to