[src/trunk]: src/usr.bin/make make(1): remove redundant null check in SuffFin...

[rillig <rillig%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/8baf36f2dc38
branches:  trunk
changeset: 957239:8baf36f2dc38
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sat Nov 21 23:25:29 2020 +0000

description:
make(1): remove redundant null check in SuffFindCmds

Src.suff is never null.  It is initialized in the constructor and then
never modified again.  It cannot be marked as const though because memory
allocation and initialization are separated in SrcNew.

diffstat:

 usr.bin/make/suff.c |  8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diffs (29 lines):

diff -r a9b44ee9af9e -r 8baf36f2dc38 usr.bin/make/suff.c
--- a/usr.bin/make/suff.c       Sat Nov 21 23:09:07 2020 +0000
+++ b/usr.bin/make/suff.c       Sat Nov 21 23:25:29 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: suff.c,v 1.279 2020/11/21 23:09:07 rillig Exp $        */
+/*     $NetBSD: suff.c,v 1.280 2020/11/21 23:25:29 rillig Exp $        */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -114,7 +114,7 @@
 #include "dir.h"
 
 /*     "@(#)suff.c     8.4 (Berkeley) 3/21/94" */
-MAKE_RCSID("$NetBSD: suff.c,v 1.279 2020/11/21 23:09:07 rillig Exp $");
+MAKE_RCSID("$NetBSD: suff.c,v 1.280 2020/11/21 23:25:29 rillig Exp $");
 
 #define SUFF_DEBUG0(text) DEBUG0(SUFF, text)
 #define SUFF_DEBUG1(fmt, arg1) DEBUG1(SUFF, fmt, arg1)
@@ -1087,9 +1087,7 @@
         * XXX: Handle multi-stage transformations here, too.
         */
 
-       /* XXX: Can targ->suff be NULL here? */
-       if (targ->suff != NULL &&
-           Lst_FindDatum(suff->parents, targ->suff) != NULL)
+       if (Lst_FindDatum(suff->parents, targ->suff) != NULL)
            break;
     }