[src/trunk]: src/usr.bin/xlint/lint1 lint: split complete_tag into separate f...

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

details:   https://anonhg.NetBSD.org/src/rev/55177424c95d
branches:  trunk
changeset: 1017511:55177424c95d
user:      rillig <rillig%NetBSD.org@localhost>
date:      Tue Dec 29 17:29:31 2020 +0000

description:
lint: split complete_tag into separate functions

diffstat:

 usr.bin/xlint/lint1/cgram.y    |  12 ++--
 usr.bin/xlint/lint1/decl.c     |  86 ++++++++++++++++++++++-------------------
 usr.bin/xlint/lint1/externs1.h |   5 +-
 3 files changed, 55 insertions(+), 48 deletions(-)

diffs (189 lines):

diff -r b631d4daaf19 -r 55177424c95d usr.bin/xlint/lint1/cgram.y
--- a/usr.bin/xlint/lint1/cgram.y       Tue Dec 29 17:17:14 2020 +0000
+++ b/usr.bin/xlint/lint1/cgram.y       Tue Dec 29 17:29:31 2020 +0000
@@ -1,5 +1,5 @@
 %{
-/* $NetBSD: cgram.y,v 1.115 2020/12/29 16:48:53 rillig Exp $ */
+/* $NetBSD: cgram.y,v 1.116 2020/12/29 17:29:31 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.115 2020/12/29 16:48:53 rillig Exp $");
+__RCSID("$NetBSD: cgram.y,v 1.116 2020/12/29 17:29:31 rillig Exp $");
 #endif
 
 #include <limits.h>
@@ -705,12 +705,12 @@
        | struct struct_tag {
                dcs->d_tagtyp = mktag($2, $1, 1, 0);
          } struct_declaration {
-               $$ = complete_tag(dcs->d_tagtyp, $4);
+               $$ = complete_tag_struct_or_union(dcs->d_tagtyp, $4);
          }
        | struct {
                dcs->d_tagtyp = mktag(NULL, $1, 1, 0);
          } struct_declaration {
-               $$ = complete_tag(dcs->d_tagtyp, $3);
+               $$ = complete_tag_struct_or_union(dcs->d_tagtyp, $3);
          }
        | struct error {
                symtyp = FVFT;
@@ -901,12 +901,12 @@
        | enum enum_tag {
                dcs->d_tagtyp = mktag($2, ENUM, 1, 0);
          } enum_declaration {
-               $$ = complete_tag(dcs->d_tagtyp, $4);
+               $$ = complete_tag_enum(dcs->d_tagtyp, $4);
          }
        | enum {
                dcs->d_tagtyp = mktag(NULL, ENUM, 1, 0);
          } enum_declaration {
-               $$ = complete_tag(dcs->d_tagtyp, $3);
+               $$ = complete_tag_enum(dcs->d_tagtyp, $3);
          }
        | enum error {
                symtyp = FVFT;
diff -r b631d4daaf19 -r 55177424c95d usr.bin/xlint/lint1/decl.c
--- a/usr.bin/xlint/lint1/decl.c        Tue Dec 29 17:17:14 2020 +0000
+++ b/usr.bin/xlint/lint1/decl.c        Tue Dec 29 17:29:31 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: decl.c,v 1.77 2020/12/29 13:33:03 rillig Exp $ */
+/* $NetBSD: decl.c,v 1.78 2020/12/29 17:29:31 rillig Exp $ */
 
 /*
  * Copyright (c) 1996 Christopher G. Demetriou.  All Rights Reserved.
@@ -38,7 +38,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: decl.c,v 1.77 2020/12/29 13:33:03 rillig Exp $");
+__RCSID("$NetBSD: decl.c,v 1.78 2020/12/29 17:29:31 rillig Exp $");
 #endif
 
 #include <sys/param.h>
@@ -1773,10 +1773,10 @@
 }
 
 /*
- * tp points to the type of the tag, fmem to the list of members/enums.
+ * tp points to the type of the tag, fmem to the list of members.
  */
 type_t *
-complete_tag(type_t *tp, sym_t *fmem)
+complete_tag_struct_or_union(type_t *tp, sym_t *fmem)
 {
        tspec_t t;
        str_t   *sp;
@@ -1785,47 +1785,53 @@
 
        setcomplete(tp, 1);
 
-       if ((t = tp->t_tspec) != ENUM) {
-               align(dcs->d_stralign, 0);
-               sp = tp->t_str;
-               sp->align = dcs->d_stralign;
-               sp->memb = fmem;
-               if (tp->t_ispacked)
-                       setpackedsize(tp);
-               else
-                       sp->size = dcs->d_offset;
-
-               if (sp->size == 0) {
-                       /* zero sized %s */
-                       (void)c99ism(47, ttab[t].tt_name);
+       t = tp->t_tspec;
+       align(dcs->d_stralign, 0);
+       sp = tp->t_str;
+       sp->align = dcs->d_stralign;
+       sp->memb = fmem;
+       if (tp->t_ispacked)
+               setpackedsize(tp);
+       else
+               sp->size = dcs->d_offset;
+
+       if (sp->size == 0) {
+               /* zero sized %s */
+               (void)c99ism(47, ttab[t].tt_name);
+       }
+
+       n = 0;
+       for (mem = fmem; mem != NULL; mem = mem->s_nxt) {
+               /* bind anonymous members to the structure */
+               if (mem->s_styp == NULL) {
+                       mem->s_styp = sp;
+                       if (mem->s_type->t_isfield) {
+                               sp->size += bitfieldsize(&mem);
+                               if (mem == NULL)
+                                       break;
+                       }
+                       sp->size += tsize(mem->s_type);
                }
-
-               n = 0;
-               for (mem = fmem; mem != NULL; mem = mem->s_nxt) {
-                       /* bind anonymous members to the structure */
-                       if (mem->s_styp == NULL) {
-                               mem->s_styp = sp;
-                               if (mem->s_type->t_isfield) {
-                                       sp->size += bitfieldsize(&mem);
-                                       if (mem == NULL)
-                                               break;
-                               }
-                               sp->size += tsize(mem->s_type);
-                       }
-                       if (mem->s_name != unnamed)
-                               n++;
-               }
-
-               if (n == 0 && sp->size != 0) {
-                       /* %s has no named members */
-                       warning(65, t == STRUCT ? "structure" : "union");
-               }
-       } else {
-               tp->t_enum->elem = fmem;
+               if (mem->s_name != unnamed)
+                       n++;
+       }
+
+       if (n == 0 && sp->size != 0) {
+               /* %s has no named members */
+               warning(65, t == STRUCT ? "structure" : "union");
        }
        return tp;
 }
 
+type_t *
+complete_tag_enum(type_t *tp, sym_t *fmem)
+{
+
+       setcomplete(tp, 1);
+       tp->t_enum->elem = fmem;
+       return tp;
+}
+
 /*
  * Processes the name of an enumerator in an enum declaration.
  *
diff -r b631d4daaf19 -r 55177424c95d usr.bin/xlint/lint1/externs1.h
--- a/usr.bin/xlint/lint1/externs1.h    Tue Dec 29 17:17:14 2020 +0000
+++ b/usr.bin/xlint/lint1/externs1.h    Tue Dec 29 17:29:31 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: externs1.h,v 1.39 2020/12/29 16:48:53 rillig Exp $     */
+/*     $NetBSD: externs1.h,v 1.40 2020/12/29 17:29:31 rillig Exp $     */
 
 /*
  * Copyright (c) 1994, 1995 Jochen Pohl
@@ -166,7 +166,8 @@
 extern sym_t   *old_style_function_name(sym_t *);
 extern type_t  *mktag(sym_t *, tspec_t, int, int);
 extern const   char *storage_class_name(scl_t);
-extern type_t  *complete_tag(type_t *, sym_t *);
+extern type_t  *complete_tag_struct_or_union(type_t *, sym_t *);
+extern type_t  *complete_tag_enum(type_t *, sym_t *);
 extern sym_t   *ename(sym_t *, int, int);
 extern void    decl1ext(sym_t *, int);
 extern void    copy_usage_info(sym_t *, sym_t *);