Source-Changes-HG archive

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

[src/trunk]: src/tests/lib/libcurses/slave tests/libcurses: remove redundant ...



details:   https://anonhg.NetBSD.org/src/rev/863a6eba5cd4
branches:  trunk
changeset: 1021701:863a6eba5cd4
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Jun 13 18:11:44 2021 +0000

description:
tests/libcurses: remove redundant argument numbers

Since all arguments are processed in increasing order, there is no need
to add the redundant argument numbers.  Most of the curses functions
have less than 5 arguments, which makes it easy enough to count the ARG
macros.

Changes to curses_commands.c:

* replace ^(\tARG_\w+\()\d(?:, |) with $1
* replace (define ARG_\w+\()i,\s with $1
* replace args\[i\] with *args++
* replace \(i\) with ()
* replace \(void\)0 with args++

The wrong argument count in cmd_mvwget_wch is still detected by
lint.lua, as it was before.  There is no test yet that covers this
function.

diffstat:

 tests/lib/libcurses/slave/curses_commands.c |  1502 +++++++++++++-------------
 tests/lib/libcurses/slave/lint.lua          |    19 +-
 2 files changed, 758 insertions(+), 763 deletions(-)

diffs (truncated from 3763 to 300 lines):

diff -r 4751252947b3 -r 863a6eba5cd4 tests/lib/libcurses/slave/curses_commands.c
--- a/tests/lib/libcurses/slave/curses_commands.c       Sun Jun 13 17:02:14 2021 +0000
+++ b/tests/lib/libcurses/slave/curses_commands.c       Sun Jun 13 18:11:44 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: curses_commands.c,v 1.25 2021/04/04 09:49:13 rin Exp $ */
+/*     $NetBSD: curses_commands.c,v 1.26 2021/06/13 18:11:44 rillig Exp $      */
 
 /*-
  * Copyright 2009 Brett Lymn <blymn%NetBSD.org@localhost>
@@ -102,68 +102,68 @@
        if (check_arg_count(nargs, n) == 1)                             \
                return
 
-#define ARG_SHORT(i, arg) \
+#define ARG_SHORT(arg) \
        short arg;                                                      \
-       if (set_short(args[i], &arg) != 0)                              \
+       if (set_short(*args++, &arg) != 0)                              \
                return
 
-#define ARG_INT(i, arg) \
+#define ARG_INT(arg) \
        int arg;                                                        \
-       if (set_int(args[i], &arg) != 0)                                \
+       if (set_int(*args++, &arg) != 0)                                \
                return
 
-#define ARG_UINT(i, arg) \
+#define ARG_UINT(arg) \
        unsigned int arg;                                               \
-       if (set_uint(args[i], &arg) != 0)                               \
+       if (set_uint(*args++, &arg) != 0)                               \
                return
 
-#define ARG_CHTYPE(i, arg) \
-       chtype arg = ((const chtype *)args[i])[0]
-
-#define ARG_WCHAR(i, arg) \
-       wchar_t arg = ((const wchar_t *)args[i])[0]
-
-#define ARG_STRING(i, arg) \
-       const char *arg = args[i]
+#define ARG_CHTYPE(arg) \
+       chtype arg = ((const chtype *)*args++)[0]
+
+#define ARG_WCHAR(arg) \
+       wchar_t arg = ((const wchar_t *)*args++)[0]
+
+#define ARG_STRING(arg) \
+       const char *arg = *args++
 
 /* Only used for legacy interfaces that are missing the 'const'. */
-#define ARG_MODIFIABLE_STRING(i, arg) \
-       char *arg = args[i]
-
-#define ARG_CHTYPE_STRING(i, arg) \
-       const chtype *arg = (const chtype *)args[i]
-
-#define ARG_CCHAR_STRING(i, arg) \
-       const cchar_t *arg = (const cchar_t *)args[i]
-
-#define ARG_WCHAR_STRING(i, arg) \
-       wchar_t *arg = (wchar_t *)args[i]
-
-#define ARG_WINDOW(i, arg) \
+#define ARG_MODIFIABLE_STRING(arg) \
+       char *arg = *args++
+
+#define ARG_CHTYPE_STRING(arg) \
+       const chtype *arg = (const chtype *)*args++
+
+#define ARG_CCHAR_STRING(arg) \
+       const cchar_t *arg = (const cchar_t *)*args++
+
+#define ARG_WCHAR_STRING(arg) \
+       wchar_t *arg = (wchar_t *)*args++
+
+#define ARG_WINDOW(arg) \
        WINDOW *arg;                                                    \
-       if (set_win(args[i], &arg) != 0)                                \
+       if (set_win(*args++, &arg) != 0)                                \
                return
 
-#define ARG_SCREEN(i, arg) \
+#define ARG_SCREEN(arg) \
        SCREEN *arg;                                                    \
-       if (set_scrn(args[i], &arg) != 0)                               \
+       if (set_scrn(*args++, &arg) != 0)                               \
                return
 
 /*
  * Required by the API, intended for future extensions, but this
  * implementation does not support the extension.
  */
-#define ARG_NULL(i) \
-       (void)0
-
-#define ARG_IGNORE(i) \
-       (void)0
+#define ARG_NULL() \
+       args++
+
+#define ARG_IGNORE() \
+       args++
 
 void
 cmd_DRAIN(int nargs, char **args)
 {
        ARGC(1);
-       ARG_WINDOW(0, win);
+       ARG_WINDOW(win);
 
        while (wgetch(win) != ERR);
        report_count(1);
@@ -174,8 +174,8 @@
 cmd_addbytes(int nargs, char **args)
 {
        ARGC(2);
-       ARG_STRING(0, str);
-       ARG_INT(1, count);
+       ARG_STRING(str);
+       ARG_INT(count);
 
        report_count(1);
        report_return(addbytes(str, count));
@@ -186,7 +186,7 @@
 cmd_addch(int nargs, char **args)
 {
        ARGC(1);
-       ARG_CHTYPE(0, ch);
+       ARG_CHTYPE(ch);
 
        report_count(1);
        report_return(addch(ch));
@@ -197,8 +197,8 @@
 cmd_addchnstr(int nargs, char **args)
 {
        ARGC(2);
-       ARG_CHTYPE_STRING(0, chstr);
-       ARG_INT(1, count);
+       ARG_CHTYPE_STRING(chstr);
+       ARG_INT(count);
 
        report_count(1);
        report_return(addchnstr(chstr, count));
@@ -209,7 +209,7 @@
 cmd_addchstr(int nargs, char **args)
 {
        ARGC(1);
-       ARG_CHTYPE_STRING(0, chstr);
+       ARG_CHTYPE_STRING(chstr);
 
        report_count(1);
        report_return(addchstr(chstr));
@@ -220,8 +220,8 @@
 cmd_addnstr(int nargs, char **args)
 {
        ARGC(2);
-       ARG_STRING(0, str);
-       ARG_INT(1, count);
+       ARG_STRING(str);
+       ARG_INT(count);
 
        report_count(1);
        report_return(addnstr(str, count));
@@ -232,7 +232,7 @@
 cmd_addstr(int nargs, char **args)
 {
        ARGC(1);
-       ARG_STRING(0, str);
+       ARG_STRING(str);
 
        report_count(1);
        report_return(addstr(str));
@@ -261,7 +261,7 @@
 cmd_attr_off(int nargs, char **args)
 {
        ARGC(1);
-       ARG_INT(0, attrib);
+       ARG_INT(attrib);
 
        report_count(1);
        report_return(attr_off(attrib, NULL));
@@ -272,7 +272,7 @@
 cmd_attr_on(int nargs, char **args)
 {
        ARGC(1);
-       ARG_INT(0, attrib);
+       ARG_INT(attrib);
 
        report_count(1);
        report_return(attr_on(attrib, NULL));
@@ -283,8 +283,8 @@
 cmd_attr_set(int nargs, char **args)
 {
        ARGC(2);
-       ARG_INT(0, attrib);
-       ARG_SHORT(1, pair);
+       ARG_INT(attrib);
+       ARG_SHORT(pair);
 
        report_count(1);
        report_return(attr_set(attrib, pair, NULL));
@@ -295,7 +295,7 @@
 cmd_attroff(int nargs, char **args)
 {
        ARGC(1);
-       ARG_INT(0, attrib);
+       ARG_INT(attrib);
 
        report_count(1);
        report_return(attroff(attrib));
@@ -306,7 +306,7 @@
 cmd_attron(int nargs, char **args)
 {
        ARGC(1);
-       ARG_INT(0, attrib);
+       ARG_INT(attrib);
 
        report_count(1);
        report_return(attron(attrib));
@@ -317,7 +317,7 @@
 cmd_attrset(int nargs, char **args)
 {
        ARGC(1);
-       ARG_INT(0, attrib);
+       ARG_INT(attrib);
 
        report_count(1);
        report_return(attrset(attrib));
@@ -328,7 +328,7 @@
 cmd_bkgd(int nargs, char **args)
 {
        ARGC(1);
-       ARG_CHTYPE(0, ch);
+       ARG_CHTYPE(ch);
 
        report_count(1);
        report_return(bkgd(ch));
@@ -339,7 +339,7 @@
 cmd_bkgdset(int nargs, char **args)
 {
        ARGC(1);
-       ARG_CHTYPE(0, ch);
+       ARG_CHTYPE(ch);
 
        bkgdset(ch);            /* returns void */
        report_count(1);
@@ -351,14 +351,14 @@
 cmd_border(int nargs, char **args)
 {
        ARGC(8);
-       ARG_INT(0, ls);
-       ARG_INT(1, rs);
-       ARG_INT(2, ts);
-       ARG_INT(3, bs);
-       ARG_INT(4, tl);
-       ARG_INT(5, tr);
-       ARG_INT(6, bl);
-       ARG_INT(7, br);
+       ARG_INT(ls);
+       ARG_INT(rs);
+       ARG_INT(ts);
+       ARG_INT(bs);
+       ARG_INT(tl);
+       ARG_INT(tr);
+       ARG_INT(bl);
+       ARG_INT(br);
 
        report_count(1);
        report_return(border(ls, rs, ts, bs, tl, tr, bl, br));
@@ -399,8 +399,8 @@
 cmd_color_set(int nargs, char **args)
 {
        ARGC(2);
-       ARG_SHORT(0, colour_pair);
-       ARG_NULL(1);
+       ARG_SHORT(colour_pair);
+       ARG_NULL();
 
        report_count(1);
        report_return(color_set(colour_pair, NULL));
@@ -431,7 +431,7 @@
 cmd_echochar(int nargs, char **args)
 {
        ARGC(1);
-       ARG_CHTYPE(0, ch);
+       ARG_CHTYPE(ch);
 
        /* XXX causes refresh */
        report_count(1);
@@ -466,7 +466,7 @@
        char *string;
 
        ARGC(1);
-       ARG_INT(0, limit);
+       ARG_INT(limit);
 
        if ((string = malloc(limit + 1)) == NULL) {
                report_count(1);



Home | Main Index | Thread Index | Old Index