Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/xlint/lint1 lint: inline variable in initialization_...



details:   https://anonhg.NetBSD.org/src/rev/d8e097aacbfc
branches:  trunk
changeset: 954072:d8e097aacbfc
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Mar 28 18:33:27 2021 +0000

description:
lint: inline variable in initialization_init_array_using_string

No functional change.

diffstat:

 usr.bin/xlint/lint1/init.c |  12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

diffs (49 lines):

diff -r 61c48c3ac415 -r d8e097aacbfc usr.bin/xlint/lint1/init.c
--- a/usr.bin/xlint/lint1/init.c        Sun Mar 28 18:28:22 2021 +0000
+++ b/usr.bin/xlint/lint1/init.c        Sun Mar 28 18:33:27 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: init.c,v 1.170 2021/03/28 18:28:22 rillig Exp $        */
+/*     $NetBSD: init.c,v 1.171 2021/03/28 18:33:27 rillig Exp $        */
 
 /*
  * Copyright (c) 1994, 1995 Jochen Pohl
@@ -37,7 +37,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: init.c,v 1.170 2021/03/28 18:28:22 rillig Exp $");
+__RCSID("$NetBSD: init.c,v 1.171 2021/03/28 18:33:27 rillig Exp $");
 #endif
 
 #include <stdlib.h>
@@ -1179,7 +1179,6 @@
 initialization_init_array_using_string(struct initialization *in, tnode_t *tn)
 {
        struct brace_level *level;
-       int     len;
        strg_t  *strg;
 
        if (tn->tn_op != STRING)
@@ -1213,12 +1212,9 @@
        } else
                goto nope;
 
-       /* Get length without trailing NUL character. */
-       len = strg->st_len;
-
        if (level->bl_array_of_unknown_size) {
                level->bl_array_of_unknown_size = false;
-               level->bl_type->t_dim = len + 1;
+               level->bl_type->t_dim = (int)(strg->st_len + 1);
                setcomplete(level->bl_type, true);
        } else {
                /*
@@ -1226,7 +1222,7 @@
                 * initialized
                 */
                /* XXX: double-check for off-by-one error */
-               if (level->bl_type->t_dim < len) {
+               if (level->bl_type->t_dim < (int)strg->st_len) {
                        /* non-null byte ignored in string initializer */
                        warning(187);
                }



Home | Main Index | Thread Index | Old Index