Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/xlint/lint2 lint: constify lint2 checking functions
details: https://anonhg.NetBSD.org/src/rev/8e10ffd713fd
branches: trunk
changeset: 985384:8e10ffd713fd
user: rillig <rillig%NetBSD.org@localhost>
date: Sun Aug 22 04:43:44 2021 +0000
description:
lint: constify lint2 checking functions
No functional change.
diffstat:
usr.bin/xlint/lint2/chk.c | 70 +++++++++++++++++++++---------------------
usr.bin/xlint/lint2/externs2.h | 4 +-
usr.bin/xlint/lint2/main2.c | 12 +++++-
3 files changed, 46 insertions(+), 40 deletions(-)
diffs (269 lines):
diff -r 5b3fd2bb1b0d -r 8e10ffd713fd usr.bin/xlint/lint2/chk.c
--- a/usr.bin/xlint/lint2/chk.c Sat Aug 21 23:00:30 2021 +0000
+++ b/usr.bin/xlint/lint2/chk.c Sun Aug 22 04:43:44 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: chk.c,v 1.43 2021/08/08 11:56:35 rillig Exp $ */
+/* $NetBSD: chk.c,v 1.44 2021/08/22 04:43:44 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: chk.c,v 1.43 2021/08/08 11:56:35 rillig Exp $");
+__RCSID("$NetBSD: chk.c,v 1.44 2021/08/22 04:43:44 rillig Exp $");
#endif
#include <ctype.h>
@@ -48,23 +48,23 @@
#include "lint2.h"
-static void chkund(hte_t *);
-static void chkdnu(hte_t *);
-static void chkdnud(hte_t *);
-static void chkmd(hte_t *);
-static void chkvtui(hte_t *, sym_t *, sym_t *);
-static void chkvtdi(hte_t *, sym_t *, sym_t *);
-static void chkfaui(hte_t *, sym_t *, sym_t *);
-static void chkau(hte_t *, int, sym_t *, sym_t *, pos_t *,
+static void chkund(const hte_t *);
+static void chkdnu(const hte_t *);
+static void chkdnud(const hte_t *);
+static void chkmd(const hte_t *);
+static void chkvtui(const hte_t *, sym_t *, sym_t *);
+static void chkvtdi(const hte_t *, sym_t *, sym_t *);
+static void chkfaui(const hte_t *, sym_t *, sym_t *);
+static void chkau(const hte_t *, int, sym_t *, sym_t *, pos_t *,
fcall_t *, fcall_t *, type_t *, type_t *);
-static void chkrvu(hte_t *, sym_t *);
-static void chkadecl(hte_t *, sym_t *, sym_t *);
-static void printflike(hte_t *,fcall_t *, int, const char *, type_t **);
-static void scanflike(hte_t *, fcall_t *, int, const char *, type_t **);
-static void badfmt(hte_t *, fcall_t *);
-static void inconarg(hte_t *, fcall_t *, int);
-static void tofewarg(hte_t *, fcall_t *);
-static void tomanyarg(hte_t *, fcall_t *);
+static void chkrvu(const hte_t *, sym_t *);
+static void chkadecl(const hte_t *, sym_t *, sym_t *);
+static void printflike(const hte_t *, fcall_t *, int, const char *, type_t **);
+static void scanflike(const hte_t *, fcall_t *, int, const char *, type_t **);
+static void badfmt(const hte_t *, fcall_t *);
+static void inconarg(const hte_t *, fcall_t *, int);
+static void tofewarg(const hte_t *, fcall_t *);
+static void tomanyarg(const hte_t *, fcall_t *);
static bool eqtype(type_t *, type_t *, bool, bool, bool, bool *);
static bool eqargs(type_t *, type_t *, bool *);
static bool mnoarg(type_t *, bool *);
@@ -86,7 +86,7 @@
* Performs all tests for a single name
*/
void
-chkname(hte_t *hte)
+chkname(const hte_t *hte)
{
sym_t *sym, *def, *pdecl, *decl;
@@ -131,7 +131,7 @@
* Print a warning if the name has been used, but not defined.
*/
static void
-chkund(hte_t *hte)
+chkund(const hte_t *hte)
{
fcall_t *fcall;
usym_t *usym;
@@ -152,7 +152,7 @@
* Print a warning if the name has been defined, but never used.
*/
static void
-chkdnu(hte_t *hte)
+chkdnu(const hte_t *hte)
{
sym_t *sym;
@@ -173,7 +173,7 @@
* or defined.
*/
static void
-chkdnud(hte_t *hte)
+chkdnud(const hte_t *hte)
{
sym_t *sym;
@@ -195,7 +195,7 @@
* this name.
*/
static void
-chkmd(hte_t *hte)
+chkmd(const hte_t *hte)
{
sym_t *sym, *def1;
char *pos1;
@@ -234,7 +234,7 @@
* call as it's done for function arguments.
*/
static void
-chkvtui(hte_t *hte, sym_t *def, sym_t *decl)
+chkvtui(const hte_t *hte, sym_t *def, sym_t *decl)
{
fcall_t *call;
char *pos1;
@@ -295,7 +295,7 @@
* types of return values are tested.
*/
static void
-chkvtdi(hte_t *hte, sym_t *def, sym_t *decl)
+chkvtdi(const hte_t *hte, sym_t *def, sym_t *decl)
{
sym_t *sym;
type_t *tp1, *tp2;
@@ -337,7 +337,7 @@
* of the same function.
*/
static void
-chkfaui(hte_t *hte, sym_t *def, sym_t *decl)
+chkfaui(const hte_t *hte, sym_t *def, sym_t *decl)
{
type_t *tp1, *tp2, **ap1, **ap2;
pos_t *pos1p = NULL;
@@ -445,7 +445,7 @@
*
*/
static void
-chkau(hte_t *hte, int n, sym_t *def, sym_t *decl, pos_t *pos1p,
+chkau(const hte_t *hte, int n, sym_t *def, sym_t *decl, pos_t *pos1p,
fcall_t *call1, fcall_t *call, type_t *arg1, type_t *arg2)
{
bool promote, asgn, dowarn;
@@ -600,7 +600,7 @@
* string fmt.
*/
static void
-printflike(hte_t *hte, fcall_t *call, int n, const char *fmt, type_t **ap)
+printflike(const hte_t *hte, fcall_t *call, int n, const char *fmt, type_t **ap)
{
const char *fp;
int fc;
@@ -825,7 +825,7 @@
* string fmt.
*/
static void
-scanflike(hte_t *hte, fcall_t *call, int n, const char *fmt, type_t **ap)
+scanflike(const hte_t *hte, fcall_t *call, int n, const char *fmt, type_t **ap)
{
const char *fp;
int fc;
@@ -1016,7 +1016,7 @@
}
static void
-badfmt(hte_t *hte, fcall_t *call)
+badfmt(const hte_t *hte, fcall_t *call)
{
/* %s: malformed format string \t%s */
@@ -1024,7 +1024,7 @@
}
static void
-inconarg(hte_t *hte, fcall_t *call, int n)
+inconarg(const hte_t *hte, fcall_t *call, int n)
{
/* %s, arg %d inconsistent with format \t%s */
@@ -1032,7 +1032,7 @@
}
static void
-tofewarg(hte_t *hte, fcall_t *call)
+tofewarg(const hte_t *hte, fcall_t *call)
{
/* %s: too few args for format \t%s */
@@ -1040,7 +1040,7 @@
}
static void
-tomanyarg(hte_t *hte, fcall_t *call)
+tomanyarg(const hte_t *hte, fcall_t *call)
{
/* %s: too many args for format \t%s */
@@ -1066,7 +1066,7 @@
* or return values which are always or sometimes ignored.
*/
static void
-chkrvu(hte_t *hte, sym_t *def)
+chkrvu(const hte_t *hte, sym_t *def)
{
fcall_t *call;
bool used, ignored;
@@ -1119,7 +1119,7 @@
* Print warnings for inconsistent argument declarations.
*/
static void
-chkadecl(hte_t *hte, sym_t *def, sym_t *decl)
+chkadecl(const hte_t *hte, sym_t *def, sym_t *decl)
{
bool osdef, eq, dowarn;
int n;
diff -r 5b3fd2bb1b0d -r 8e10ffd713fd usr.bin/xlint/lint2/externs2.h
--- a/usr.bin/xlint/lint2/externs2.h Sat Aug 21 23:00:30 2021 +0000
+++ b/usr.bin/xlint/lint2/externs2.h Sun Aug 22 04:43:44 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: externs2.h,v 1.10 2021/08/10 17:31:44 rillig Exp $ */
+/* $NetBSD: externs2.h,v 1.11 2021/08/22 04:43:44 rillig Exp $ */
/*
* Copyright (c) 1996 Christopher G. Demetriou. All Rights Reserved.
@@ -78,7 +78,7 @@
*/
extern void inittyp(void);
extern void mainused(void);
-extern void chkname(hte_t *);
+extern void chkname(const hte_t *);
/*
* msg.c
diff -r 5b3fd2bb1b0d -r 8e10ffd713fd usr.bin/xlint/lint2/main2.c
--- a/usr.bin/xlint/lint2/main2.c Sat Aug 21 23:00:30 2021 +0000
+++ b/usr.bin/xlint/lint2/main2.c Sun Aug 22 04:43:44 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: main2.c,v 1.18 2021/08/11 05:37:45 rillig Exp $ */
+/* $NetBSD: main2.c,v 1.19 2021/08/22 04:43:44 rillig Exp $ */
/*
* Copyright (c) 1994, 1995 Jochen Pohl
@@ -37,7 +37,7 @@
#include <sys/cdefs.h>
#if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: main2.c,v 1.18 2021/08/11 05:37:45 rillig Exp $");
+__RCSID("$NetBSD: main2.c,v 1.19 2021/08/22 04:43:44 rillig Exp $");
#endif
#include <stdio.h>
@@ -91,6 +91,12 @@
static void usage(void) __attribute__((noreturn));
+static void
+check_name(hte_t *hte)
+{
+ chkname(hte);
+}
+
int
main(int argc, char *argv[])
{
@@ -182,7 +188,7 @@
/* perform all tests */
/* TODO: sort the names; hashcode order looks chaotic. */
- forall(chkname);
+ forall(check_name);
exit(0);
/* NOTREACHED */
Home |
Main Index |
Thread Index |
Old Index