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: rename push_member and pop_member



details:   https://anonhg.NetBSD.org/src/rev/1f7f091a7a45
branches:  trunk
changeset: 960427:1f7f091a7a45
user:      rillig <rillig%NetBSD.org@localhost>
date:      Fri Mar 19 00:55:02 2021 +0000

description:
lint: rename push_member and pop_member

These two functions are supposed to model the designator that is used
for initializing structs and arrays.  The implementation is still buggy
and does not work at all for C99 designators with multiple names, see
d_init_pop_member.c.

For now, just rename the functions to head in the right direction.

No functional change.

diffstat:

 usr.bin/xlint/lint1/cgram.y    |   8 ++++----
 usr.bin/xlint/lint1/externs1.h |   4 ++--
 usr.bin/xlint/lint1/init.c     |  14 +++++++-------
 3 files changed, 13 insertions(+), 13 deletions(-)

diffs (118 lines):

diff -r 308426e2cff2 -r 1f7f091a7a45 usr.bin/xlint/lint1/cgram.y
--- a/usr.bin/xlint/lint1/cgram.y       Fri Mar 19 00:44:09 2021 +0000
+++ b/usr.bin/xlint/lint1/cgram.y       Fri Mar 19 00:55:02 2021 +0000
@@ -1,5 +1,5 @@
 %{
-/* $NetBSD: cgram.y,v 1.176 2021/03/17 15:45:30 rillig Exp $ */
+/* $NetBSD: cgram.y,v 1.177 2021/03/19 00:55:02 rillig Exp $ */
 
 /*
  * Copyright (c) 1996 Christopher G. Demetriou.  All Rights Reserved.
@@ -35,7 +35,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: cgram.y,v 1.176 2021/03/17 15:45:30 rillig Exp $");
+__RCSID("$NetBSD: cgram.y,v 1.177 2021/03/19 00:55:02 rillig Exp $");
 #endif
 
 #include <limits.h>
@@ -1368,7 +1368,7 @@
                if (!Sflag)
                        /* struct or union member name in initializer is ... */
                        warning(313);
-               push_member($2);
+               designator_push_name($2);
          }
        ;
 
@@ -1382,7 +1382,7 @@
        | identifier T_COLON {
                /* GCC style struct or union member name in initializer */
                gnuism(315);
-               push_member($1);
+               designator_push_name($1);
          }
        ;
 
diff -r 308426e2cff2 -r 1f7f091a7a45 usr.bin/xlint/lint1/externs1.h
--- a/usr.bin/xlint/lint1/externs1.h    Fri Mar 19 00:44:09 2021 +0000
+++ b/usr.bin/xlint/lint1/externs1.h    Fri Mar 19 00:55:02 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: externs1.h,v 1.76 2021/03/17 15:45:30 rillig Exp $     */
+/*     $NetBSD: externs1.h,v 1.77 2021/03/19 00:55:02 rillig Exp $     */
 
 /*
  * Copyright (c) 1994, 1995 Jochen Pohl
@@ -299,7 +299,7 @@
 extern void    init_rbrace(void);
 extern void    init_lbrace(void);
 extern void    init_using_expr(tnode_t *);
-extern void    push_member(sbuf_t *);
+extern void    designator_push_name(sbuf_t *);
 extern void    designator_push_subscript(range_t);
 
 /*
diff -r 308426e2cff2 -r 1f7f091a7a45 usr.bin/xlint/lint1/init.c
--- a/usr.bin/xlint/lint1/init.c        Fri Mar 19 00:44:09 2021 +0000
+++ b/usr.bin/xlint/lint1/init.c        Fri Mar 19 00:55:02 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: init.c,v 1.102 2021/03/19 00:39:17 rillig Exp $        */
+/*     $NetBSD: init.c,v 1.103 2021/03/19 00:55:02 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.102 2021/03/19 00:39:17 rillig Exp $");
+__RCSID("$NetBSD: init.c,v 1.103 2021/03/19 00:55:02 rillig Exp $");
 #endif
 
 #include <stdlib.h>
@@ -338,7 +338,7 @@
 #endif
 
 void
-push_member(sbuf_t *sb)
+designator_push_name(sbuf_t *sb)
 {
        namlist_t *nam = xcalloc(1, sizeof (namlist_t));
        nam->n_name = sb->sb_name;
@@ -378,7 +378,7 @@
 }
 
 static void
-pop_member(void)
+designator_pop_name(void)
 {
        debug_step("%s: %s %p", __func__, namedmem->n_name, namedmem);
        if (namedmem->n_next == namedmem) {
@@ -450,7 +450,7 @@
                        /* XXX: why ++? */
                        istk->i_remaining++;
                        /* XXX: why is i_seen_named_member not set? */
-                       pop_member();
+                       designator_pop_name();
                        return;
                }
        }
@@ -458,7 +458,7 @@
        /* undefined struct/union member: %s */
        error(101, namedmem->n_name);
 
-       pop_member();
+       designator_pop_name();
        istk->i_seen_named_member = true;
 }
 
@@ -650,7 +650,7 @@
                istk->i_subt = m->s_type;
                istk->i_seen_named_member = true;
                debug_step("named member '%s'", namedmem->n_name);
-               pop_member();
+               designator_pop_name();
                cnt = istk->i_type->t_tspec == STRUCT ? 2 : 1;
        }
        istk->i_brace = true;



Home | Main Index | Thread Index | Old Index