]> git-server-git.apps.pok.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
lib: Fix non-ANSI function declarations
authorEric Sandeen <sandeen@redhat.com>
Mon, 10 Mar 2025 18:29:05 +0000 (13:29 -0500)
committerZorro Lang <zlang@kernel.org>
Tue, 11 Mar 2025 20:02:50 +0000 (04:02 +0800)
lib/ was full of non-ANSI function declarations, fix them to make
sparse happier.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Zorro Lang <zlang@kernel.org>
lib/dataascii.c
lib/databin.c
lib/datapid.c
lib/file_lock.c
lib/forker.c
lib/pattern.c
lib/random_range.c
lib/str_to_bytes.c
lib/tlibio.c
lib/write_log.c

index e2509f8d5bc094ca29a38762bcf5b1a7cd8f2b54..d11609eed949c778ebb2488ae360ac5cb5a3d9a7 100644 (file)
 static char Errmsg[80];
 
 int
-dataasciigen(listofchars, buffer, bsize, offset)
-char *listofchars;     /* a null terminated list of characters */
-char *buffer;
-int bsize;
-int offset;
+dataasciigen(
+       char *listofchars,      /* a null terminated list of characters */
+       char *buffer,
+       int bsize,
+       int offset)
 {
-   int cnt;
-   int total;
-   int ind;    /* index into CHARS array */
-   char *chr;
-   int chars_size;
-   char *charlist;
+       int cnt;
+       int total;
+       int ind;        /* index into CHARS array */
+       char *chr;
+       int chars_size;
+       char *charlist;
 
        chr=buffer;
        total=offset+bsize;
@@ -52,19 +52,19 @@ int offset;
 }      /* end of dataasciigen */
 
 int
-dataasciichk(listofchars, buffer, bsize, offset, errmsg)
-char *listofchars;     /* a null terminated list of characters */
-char *buffer;
-int bsize;
-int offset;
-char **errmsg;
+dataasciichk(
+       char *listofchars,      /* a null terminated list of characters */
+       char *buffer,
+       int bsize,
+       int offset,
+       char **errmsg)
 {
-   int cnt;
-   int total;
-   int ind;    /* index into CHARS array */
-   char *chr;
-   int chars_size;
-   char *charlist;
+       int cnt;
+       int total;
+       int ind;        /* index into CHARS array */
+       char *chr;
+       int chars_size;
+       char *charlist;
 
        chr=buffer;
        total=offset+bsize;
@@ -104,15 +104,12 @@ char **errmsg;
  * main for doing unit testing
  ***********************************************************************/
 int
-main(ac, ag)
-int ac;
-char **ag;
+main(int ac, char **ag)
 {
-
-int size=1023;
-char *buffer;
-int ret;
-char *errmsg;
+    int size=1023;
+    char *buffer;
+    int ret;
+    char *errmsg;
 
     if ((buffer=(char *)malloc(size)) == NULL ) {
         perror("malloc");
index 8a36dff31efbfeb45e196ccb049d4062d53d2f28..000d0d1a84529ae3355bf8cfb1da5e91f98179a8 100644 (file)
 static char Errmsg[80];
 
 void
-databingen (mode, buffer, bsize, offset)
-int mode;      /* either a, c, r, o, z or C */
-unsigned char *buffer; /* buffer pointer */
-int bsize;     /* size of buffer */
-int offset;    /* offset into the file where buffer starts */
+databingen(
+       int mode,       /* either a, c, r, o, z or C */
+       unsigned char *buffer,  /* buffer pointer */
+       int bsize,      /* size of buffer */
+       int offset)     /* offset into the file where buffer starts */
 {
-int ind;
+       int ind;
 
         switch (mode)
         {
@@ -63,12 +63,12 @@ int ind;
  *      < 0  : no error
  ***********************************************************************/
 int
-databinchk(mode, buffer, bsize, offset, errmsg)
-int mode;      /* either a, c, r, z, o, or C */
-unsigned char *buffer; /* buffer pointer */
-int bsize;     /* size of buffer */
-int offset;    /* offset into the file where buffer starts */
-char **errmsg;
+databinchk(
+       int mode,       /* either a, c, r, z, o, or C */
+       unsigned char *buffer,  /* buffer pointer */
+       int bsize,      /* size of buffer */
+       int offset,     /* offset into the file where buffer starts */
+       char **errmsg)
 {
        int cnt;
        unsigned char *chr;
@@ -138,9 +138,7 @@ char **errmsg;
  * main for doing unit testing
  ***********************************************************************/
 int
-main(ac, ag)
-int ac;
-char **ag;
+main(int ac, char **ag)
 {
 
     int size=1023;
index 15af887167e45ef2a968126102880a113a437884..6786323d337398f61f220201670142f7b1d1bc70 100644 (file)
@@ -57,15 +57,13 @@ static char Errmsg[80];
  * Thus, offset 8 is in middle of word 1
  ***********************************************************************/
 int
-datapidgen(pid, buffer, bsize, offset)
-int pid;
-char *buffer;
-int bsize;
-int offset;
+datapidgen(
+       int pid,
+       char *buffer,
+       int bsize,
+       int offset)
 {
        return -1;      /* not support on non-64 bits word machines  */
-
-
 } 
 
 /***********************************************************************
@@ -73,12 +71,7 @@ int offset;
  *
  ***********************************************************************/
 int
-datapidchk(pid, buffer, bsize, offset, errmsg)
-int pid;
-char *buffer;
-int bsize;
-int offset;
-char **errmsg;
+datapidchk(int pid, char *buffer, int bsize, int offset, char **errmsg)
 {
     if ( errmsg != NULL ) {
         *errmsg = Errmsg;
@@ -94,15 +87,12 @@ char **errmsg;
  * main for doing unit testing
  ***********************************************************************/
 int
-main(ac, ag)
-int ac;
-char **ag;
+main( int ac, char **ag)
 {
-
-int size=1234;
-char *buffer;
-int ret;
-char *errmsg;
+    int size=1234;
+    char *buffer;
+    int ret;
+    char *errmsg;
 
     if ((buffer=(char *)malloc(size)) == NULL ) {
         perror("malloc");
index f0791489c77186aa02e95777ce436d86ca4a8054..6d87e2813389e28d3b0ad75e738f133441eb483a 100644 (file)
@@ -34,10 +34,7 @@ static char errmsg[256];
  * It will loop if the LOCK_NB flags is NOT set.
  ***********************************************************************/
 int
-file_lock(fd, flags, errormsg)
-int fd;
-int flags;
-char **errormsg;
+file_lock(int fd, int flags, char **errormsg)
 {
         register int cmd, ret;
         struct flock flocks;
@@ -109,12 +106,7 @@ char **errormsg;
  * It will loop if the LOCK_NB flags is NOT set.
  ***********************************************************************/
 int
-record_lock(fd, flags, start, len, errormsg)
-int fd;
-int flags;
-int start;
-int len;
-char **errormsg;
+record_lock(int fd, int flags, int start, int len, char **errormsg)
 {
         register int cmd, ret;
         struct flock flocks;
index 63d8fcdba8d0da5a133d9e5166b32ba3242dc729..10920ddb0a409deb9634708716e43ab76fabbfc3 100644 (file)
@@ -105,8 +105,7 @@ int Forker_npids=0;             /* number of entries in Forker_pids */
  *  !0 : if fork failed, the return value will be the errno.
  ***********************************************************************/
 int
-background(prefix)
-char *prefix;
+background(char *prefix)
 {
   switch (fork()) {
   case -1:
@@ -131,12 +130,12 @@ char *prefix;
  * 
  ***********************************************************************/
 int
-forker(ncopies, mode, prefix)
-int ncopies;
-int mode;      /* 0 - all childern of parent, 1 - only 1 direct child */
-char *prefix;   /* if ! NULL, an message will be printed to stderr */
-               /* if fork fails.  The prefix (program name) will */
-               /* preceed the message */
+forker(
+    int ncopies,
+    int mode,          /* 0: all childern of parent, 1: only 1 direct child */
+    char *prefix)      /* if ! NULL, an message will be printed to stderr */
+                       /* if fork fails.  The prefix (program name) will */
+                       /* preceed the message */
 {
     int cnt;
     int pid;
@@ -210,9 +209,7 @@ char *prefix;   /* if ! NULL, an message will be printed to stderr */
  */
 
 int
-main(argc, argv)
-int argc;
-char **argv;
+main(int argc, char **argv)
 {
     int ncopies=1;
     int mode=0;
index d622b93552f41d194fac758411ee65e1715195cc..20bbdc977a671dd809de9b3257159b7d81bb299f 100644 (file)
  */
 
 int
-pattern_check(buf, buflen, pat, patlen, patshift)
-char   *buf;
-int    buflen;
-char   *pat;
-int    patlen;
-int    patshift;
+pattern_check(char *buf, int buflen, char *pat, int patlen, int patshift)
 {
     int                nb, ncmp, nleft;
     char       *cp;
@@ -79,12 +74,7 @@ int  patshift;
 }
 
 int
-pattern_fill(buf, buflen, pat, patlen, patshift)
-char   *buf;
-int    buflen;
-char   *pat;
-int    patlen;
-int    patshift;
+pattern_fill(char *buf, int buflen, char *pat, int patlen, int patshift)
 {
     int                trans, ncopied, nleft;
     char       *cp;
index 3fa01f0ddffda9ddb6c4db30321877e12a624266..680bf71ca12121fce85fa844d818befe455bcafd 100644 (file)
@@ -73,14 +73,14 @@ static int       str_to_int();
 static long long divider(long long, long long, long long, long long);
 
 int
-parse_ranges(str, defmin, defmax, defmult, parse_func, rangeptr, errptr)
-char   *str;
-int    defmin;
-int    defmax;
-int    defmult;
-int    (*parse_func)();
-char   **rangeptr;
-char   **errptr;
+parse_ranges(
+       char *str,
+       int defmin,
+       int defmax,
+       int defmult,
+       int (*parse_func)(),
+       char **rangeptr,
+       char **errptr)
 {
        int             ncommas;
        char            *tmpstr, *cp, *tok, *n1str, *n2str, *multstr;
@@ -194,9 +194,7 @@ char        **errptr;
  */
 
 static int
-str_to_int(str, ip)
-char   *str;
-int    *ip;
+str_to_int(char *str, int *ip)
 {
        char    c;
 
@@ -214,25 +212,19 @@ int       *ip;
  */
 
 int
-range_min(rbuf, r)
-char   *rbuf;
-int    r;
+range_min(char *rbuf, int r)
 {
        return ((struct range *)rbuf)[r].min;
 }
 
 int
-range_max(rbuf, r)
-char   *rbuf;
-int    r;
+range_max(char *rbuf, int r)
 {
        return ((struct range *)rbuf)[r].max;
 }
 
 int
-range_mult(rbuf, r)
-char   *rbuf;
-int    r;
+range_mult(char *rbuf, int r)
 {
        return ((struct range *)rbuf)[r].mult;
 }
@@ -263,11 +255,7 @@ int        r;
  *****************************************************************************/
 
 long
-random_range(min, max, mult, errp)
-int    min;
-int    max;
-int    mult;
-char   **errp;
+random_range(int min, int max, int mult, char **errp)
 {
        int             r, nmults, orig_min, orig_max, orig_mult, tmp;
        extern long     lrand48();
@@ -333,11 +321,7 @@ char       **errp;
  * Just like random_range, but all values are longs.
  */
 long
-random_rangel(min, max, mult, errp)
-long   min;
-long   max;
-long   mult;
-char   **errp;
+random_range1(long min, long max, long mult, char **errp)
 {
        long            r, nmults, orig_min, orig_max, orig_mult, tmp;
        extern long     lrand48();
@@ -424,11 +408,7 @@ char       **errp;
  *  Attempts to be just like random_range, but everything is long long (64 bit)
  */
 long long
-random_rangell(min, max, mult, errp)
-long long      min;
-long long      max;
-long long      mult;
-char           **errp;
+random_rangell(long long min, long long max, long long mult, char **errp)
 {
        long long       r, nmults, orig_min, orig_max, orig_mult, tmp;
         long long      randnum;
@@ -588,8 +568,7 @@ printf("   diff = %lld, half = %lld,   med = %lld\n", diff, half, med);
  *****************************************************************************/
 
 void
-random_range_seed(s)
-long    s;
+random_range_seed(long s)
 {
     extern void srand48();
 
@@ -652,9 +631,7 @@ random_bit(long mask)
 /*
  *  The following is a unit test main function for random_bit().
  */
-main(argc, argv)
-int argc;
-char **argv;
+main(int argc, char **argv)
 {
     int ind;
     int cnt, iter;
@@ -695,9 +672,7 @@ char **argv;
 #define MEG  1024*1024*1024
 #define GIG 1073741824
 int
-main(argc, argv)
-int argc;
-char **argv;
+main(int argc, char **argv)
 {
     int ind;
     int cnt, iter=10;
index 89bc7d31ef8f9a640c2d285b7ed3a1c602b8e844..2586b0064ceb4b6abcea17146ca18e9cc1dcc809 100644 (file)
@@ -42,8 +42,7 @@
 #define T_MULT 1099511627776   /* tera or 2^40 */
 
 int
-str_to_bytes(s)
-char    *s;
+str_to_bytes(char *s)
 {
     char    mult, junk;
     int            nconv;
@@ -77,8 +76,7 @@ char    *s;
 }
 
 long
-str_to_lbytes(s)
-char    *s;
+str_to_lbytes(char *s)
 {
     char    mult, junk;
     long    nconv;
@@ -117,8 +115,7 @@ char    *s;
  */
 
 long long
-str_to_llbytes(s)
-char    *s;
+str_to_llbytes(char *s)
 {
     char    mult, junk;
     long    nconv;
index ae936f41c5a79caf9bcc12925ebe8a3784c50164..17ab34ee1f30168380ee6393d01e38ef78e57a78 100644 (file)
@@ -143,13 +143,13 @@ static int Debug_level = 0;
  ***********************************************************************/
 
 int
-stride_bounds(offset, stride, nstrides, bytes_per_stride, min, max)
-int    offset;
-int    stride;
-int    nstrides;
-int    bytes_per_stride;
-int    *min;
-int    *max;
+stride_bounds(
+       int offset,
+       int stride,
+       int nstrides,
+       int bytes_per_stride,
+       int *min,
+       int *max)
 {
        int     nbytes, min_byte, max_byte;
 
@@ -443,14 +443,14 @@ lio_random_methods(long curr_mask)
  * (rrl 04/96)
  ***********************************************************************/
 int
-lio_write_buffer(fd, method, buffer, size, sig, errmsg, wrd)
-int fd;                /* open file descriptor */
-int method;    /* contains io type and wait method bitmask */
-char *buffer;  /* pointer to buffer */
-int size;      /* the size of the io */
-int sig;       /* signal to use if async io */
-char **errmsg; /* char pointer that will be updated to point to err message */
-long wrd;      /* to allow future features, use zero for now */
+lio_write_buffer(
+    int fd,            /* open file descriptor */
+    int method,                /* contains io type and wait method bitmask */
+    char *buffer,      /* pointer to buffer */
+    int size,          /* the size of the io */
+    int sig,           /* signal to use if async io */
+    char **errmsg,     /* char pointer that will be updated to point to err message */
+    long wrd)          /* to allow future features, use zero for now */
 {
     int ret = 0;       /* syscall return or used to get random method */
 #ifndef __linux__
@@ -640,14 +640,14 @@ long wrd; /* to allow future features, use zero for now */
  * (rrl 04/96)
  ***********************************************************************/
 int
-lio_read_buffer(fd, method, buffer, size, sig, errmsg, wrd)
-int fd;                /* open file descriptor */
-int method;    /* contains io type and wait method bitmask */
-char *buffer;  /* pointer to buffer */
-int size;      /* the size of the io */
-int sig;       /* signal to use if async io */
-char **errmsg; /* char pointer that will be updated to point to err message */
-long wrd;      /* to allow future features, use zero for now */
+lio_read_buffer(
+    int fd,            /* open file descriptor */
+    int method,                /* contains io type and wait method bitmask */
+    char *buffer,      /* pointer to buffer */
+    int size,          /* the size of the io */
+    int sig,           /* signal to use if async io */
+    char **errmsg,     /* char pointer that will be updated to point to err message */
+    long wrd)          /* to allow future features, use zero for now */
 {
     int ret = 0;       /* syscall return or used to get random method */
 #ifndef __linux__
@@ -1031,9 +1031,7 @@ struct unit_info_t {
 };
 
 int
-main(argc, argv)
-int argc;
-char **argv;
+main(int argc, char **argv)
 {
     extern char *optarg;
     extern int optind;
index 7c58a95f0038bc39ada84534a58a45cfd5808c44..32567a1c6ac47031dff508799fa22d0b44f04c28 100644 (file)
@@ -87,10 +87,7 @@ static int   wlog_rec_unpack();
  */
 
 int
-wlog_open(wfile, trunc, mode)
-struct wlog_file       *wfile;
-int                    trunc;
-int                    mode;
+wlog_open(struct wlog_file *wfile, int trunc, int mode)
 {
        int     omask, oflags;
 
@@ -138,8 +135,7 @@ int                 mode;
  */
 
 int
-wlog_close(wfile)
-struct wlog_file       *wfile;
+wlog_close(struct wlog_file *wfile)
 {
        close(wfile->w_afd);
        close(wfile->w_rfd);
@@ -173,10 +169,7 @@ struct wlog_file   *wfile;
  */
 
 int
-wlog_record_write(wfile, wrec, offset)
-struct wlog_file       *wfile;
-struct wlog_rec                *wrec;
-long                   offset;
+wlog_record_write(struct wlog_file *wfile, struct wlog_rec *wrec, long offset)
 {
     int                reclen;
     char       wbuf[WLOG_REC_MAX_SIZE + 2];
@@ -221,11 +214,11 @@ long                      offset;
  */
 
 int
-wlog_scan_backward(wfile, nrecs, func, data)
-struct wlog_file       *wfile;
-int                    nrecs;
-int                    (*func)();
-long                   data;
+wlog_scan_backward(
+       struct wlog_file        *wfile,
+       int                     nrecs,
+       int                     (*func)(),
+       long                    data)
 {
        int                     fd, leftover, nbytes, offset, recnum, reclen;
        char                    buf[BSIZE*32], *bufend, *cp, *bufstart;
@@ -351,10 +344,7 @@ long                       data;
  */
 
 static int
-wlog_rec_pack(wrec, buf, flag)
-struct wlog_rec        *wrec;
-char           *buf;
-int             flag;
+wlog_rec_pack(struct wlog_rec *wrec, char *buf, int flag)
 {
        char                    *file, *host, *pattern;
        struct wlog_rec_disk    *wrecd;
@@ -400,9 +390,7 @@ int             flag;
 }
 
 static int
-wlog_rec_unpack(wrec, buf)
-struct wlog_rec        *wrec;
-char           *buf;
+wlog_rec_unpack(struct wlog_rec *wrec, char *buf)
 {
        char                    *file, *host, *pattern;
        struct wlog_rec_disk    *wrecd;