Module Name:    src
Committed By:   rillig
Date:           Wed Feb  9 21:32:38 UTC 2022

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

Log Message:
make: simplify control flow in ModifyWord_SysVSubst

No functional change.


To generate a diff of this commit:
cvs rdiff -u -r1.1010 -r1.1011 src/usr.bin/make/var.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/var.c
diff -u src/usr.bin/make/var.c:1.1010 src/usr.bin/make/var.c:1.1011
--- src/usr.bin/make/var.c:1.1010	Wed Feb  9 21:03:13 2022
+++ src/usr.bin/make/var.c	Wed Feb  9 21:32:38 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: var.c,v 1.1010 2022/02/09 21:03:13 rillig Exp $	*/
+/*	$NetBSD: var.c,v 1.1011 2022/02/09 21:32:38 rillig Exp $	*/
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -139,7 +139,7 @@
 #include "metachar.h"
 
 /*	"@(#)var.c	8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: var.c,v 1.1010 2022/02/09 21:03:13 rillig Exp $");
+MAKE_RCSID("$NetBSD: var.c,v 1.1011 2022/02/09 21:32:38 rillig Exp $");
 
 /*
  * Variables are defined using one of the VAR=value assignments.  Their
@@ -1430,10 +1430,11 @@ ModifyWord_SysVSubst(Substring word, Sep
 	if (Substring_IsEmpty(word))
 		return;
 
-	if (!Substring_HasPrefix(word, args->lhsPrefix))
-		goto no_match;
-	if (!Substring_HasSuffix(word, args->lhsSuffix))
-		goto no_match;
+	if (!Substring_HasPrefix(word, args->lhsPrefix) ||
+	    !Substring_HasSuffix(word, args->lhsSuffix)) {
+		SepBuf_AddSubstring(buf, word);
+		return;
+	}
 
 	rhs = FStr_InitRefer(args->rhs);
 	Var_Expand(&rhs, args->scope, VARE_WANTRES);
@@ -1449,10 +1450,6 @@ ModifyWord_SysVSubst(Substring word, Sep
 	SepBuf_AddStr(buf, percent != NULL ? percent + 1 : rhs.str);
 
 	FStr_Done(&rhs);
-	return;
-
-no_match:
-	SepBuf_AddSubstring(buf, word);
 }
 #endif
 

Reply via email to