diff options
| author | Ingo Schwarze <schwarze@openbsd.org> | 2020-06-14 22:49:36 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2020-06-14 22:49:36 +0000 |
| commit | bb6266ad99efe5f97667072fdd52437234e94768 (patch) | |
| tree | 68ea36e5f0ca8795488aae9ea6557ecbdd9044b1 | |
| parent | 84bf69644549389532a13e44d2351672fa2fca9f (diff) | |
| download | mandoc-bb6266ad99efe5f97667072fdd52437234e94768.tar.gz | |
Give the fts_compar struct member a real prototype.
This adds clarity and avoids compiler warnings.
| -rw-r--r-- | compat_fts.c | 5 | ||||
| -rw-r--r-- | compat_fts.h | 3 |
2 files changed, 6 insertions, 2 deletions
diff --git a/compat_fts.c b/compat_fts.c index 87847927..01577f50 100644 --- a/compat_fts.c +++ b/compat_fts.c @@ -62,6 +62,8 @@ static int fts_palloc(FTS *, size_t); static FTSENT *fts_sort(FTS *, FTSENT *, int); static unsigned short fts_stat(FTS *, FTSENT *); +typedef int (*qsort_compar_proto)(const void *, const void *); + #define ISDOT(a) (a[0] == '.' && (!a[1] || (a[1] == '.' && !a[2]))) #ifndef O_CLOEXEC #define O_CLOEXEC 0 @@ -585,7 +587,8 @@ fts_sort(FTS *sp, FTSENT *head, int nitems) } for (ap = sp->fts_array, p = head; p; p = p->fts_link) *ap++ = p; - qsort(sp->fts_array, nitems, sizeof(FTSENT *), sp->fts_compar); + qsort(sp->fts_array, nitems, sizeof(FTSENT *), + (qsort_compar_proto)sp->fts_compar); for (head = *(ap = sp->fts_array); --nitems; ++ap) ap[0]->fts_link = ap[1]; ap[0]->fts_link = NULL; diff --git a/compat_fts.h b/compat_fts.h index f4a97a4c..ca3f3a7e 100644 --- a/compat_fts.h +++ b/compat_fts.h @@ -43,7 +43,8 @@ typedef struct { char *fts_path; /* path for this descent */ size_t fts_pathlen; /* sizeof(path) */ int fts_nitems; /* elements in the sort array */ - int (*fts_compar)(); /* compare function */ + int (*fts_compar)(const struct _ftsent **, const struct _ftsent **); + /* compare function */ #define FTS_NOCHDIR 0x0004 /* don't change directories */ #define FTS_PHYSICAL 0x0010 /* physical walk */ |